Compare commits

...

8 Commits

Author SHA1 Message Date
curry772 f4e806fa08 test: 전체 테스트 스위트 통과하도록 격리/기대값 수정 (115/115)
전체 스위트 실행 시 115개 중 28개가 실패하던 문제 처리.

1. 클래스별 JVM 격리 (25건)
   TransformEngine/LayoutManager/MessageFactory 는 ApplicationContextProvider
   기반의 JVM 전역 싱글턴이라, 한 JVM 에서 여러 테스트 클래스를 돌리면 마지막에
   뜬 스프링 컨텍스트가 앞선 컨텍스트를 덮어쓴다. 그 결과 뒤 클래스가 앞 클래스의
   레이아웃을 보게 되어 "cannot create message" 로 실패했다.
   (각 클래스는 단독 실행 시에는 모두 통과)
   → build.gradle test 태스크에 forkEvery = 1 추가.
     느려지는 부분은 maxParallelForks 로 상쇄 (7분 → 4분 28초).

2. GetTrinomialFunctionTest 기대값 정정 (2건)
   JEP addStandardConstants() 에는 true/false 가 없고 Parser 가 이를 Boolean 으로
   등록한다(Parser.addBooleanConstants). 테스트 setUp 도 Boolean 을 바인딩해 놓고
   결과를 Number 로 단언하고 있어 자기모순이었음 → Boolean 기대로 정정.

3. LayoutRepositoryTest 시드 ID 수정 (1건)
   시드 SQL 어디에도 없는 TST_TTST00000002_RES 를 조회하고 있었음
   → init_tseaitr07/08 에 존재하는 TST_TOSSTEST1_TGT 로 변경.

운영 코드 변경 없음 (build.gradle 테스트 설정 + 테스트 코드만).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AnVAVJLEqjhZnzeWbNcp5q
2026-08-27 16:25:11 +09:00
curry772 0c048a88fe fix: 변환매핑 item이 하나도 없는 변환매핑 로딩 안되는 문제 처리 2026-08-27 12:03:10 +09:00
curry772 b1e7883d41 fix: target validation 오류 발생 시 item path 정보 출력 안되는 문제 처리 2026-08-27 09:18:49 +09:00
curry772 09ac7f4ca5 printStackTrace() 주석 처리 2026-08-13 15:07:39 +09:00
curry772 90ccf59b25 변환 표현식 내에 true, false가 있을 경우, warning 출력 부분 제거 2026-08-12 17:57:42 +09:00
curry772 051305ac55 숫자형 validation 로직 원복 100.12 -> 5,2 로 설정하는 방식 2026-08-05 16:33:25 +09:00
curry772 375f0e0f15 JSONMessage 불필요한 로직 제거 2026-08-05 13:04:02 +09:00
curry772 027ec4fcea isValid 함수에서 전체 길이는 '.'를 포함하지 않는 정수부 + 소수부 길이로 함 2026-08-04 18:18:44 +09:00
11 changed files with 119 additions and 125 deletions
+10
View File
@@ -85,6 +85,16 @@ dependencies {
test {
useJUnitPlatform()
// TransformEngine/LayoutManager/MessageFactory 는 ApplicationContextProvider 기반의
// JVM 전역 싱글턴이다. 한 JVM 에서 여러 테스트 클래스를 돌리면 마지막에 뜬 스프링
// 컨텍스트가 앞선 컨텍스트를 덮어써서, 뒤 클래스가 앞 클래스의 레이아웃을 보게 된다
// ("cannot create message" 오류). 클래스마다 JVM 을 새로 띄워 격리한다.
forkEvery = 1
// 클래스별 JVM 분리로 느려지는 것을 CPU 코어 수만큼 병렬 실행해 상쇄한다.
// 포크마다 별도 JVM(=별도 H2 인메모리 DB)이라 서로 간섭하지 않는다.
maxParallelForks = Math.max(1, Runtime.runtime.availableProcessors().intdiv(2))
}
publishing {
@@ -1,6 +1,11 @@
package com.eactive.eai.transformer.engine;
import java.io.Serializable;
import java.util.Arrays;
import java.util.Collections;
import java.util.HashSet;
import java.util.LinkedHashSet;
import java.util.Set;
import org.nfunk.jep.JEP;
import org.nfunk.jep.function.PostfixMathCommand;
@@ -26,6 +31,14 @@ public class Parser implements Serializable {
private static Logger logger = Logger.getLogger("transformer");
/**
* 표현식의 변수가 아니라 파서가 미리 심볼테이블에 등록해 두는 예약 심볼.
* getSymbols() 결과에서 제외하지 않으면 Transformer 가 이를 소스전문의 항목경로로 보고
* 조회를 시도해 "레이아웃항목이 존재하지 않음" 경고가 매 변환마다 발생한다.
*/
private static final Set<String> RESERVED_SYMBOLS =
Collections.unmodifiableSet(new HashSet<>(Arrays.asList("true", "false")));
private transient JEP jep = new JEP(); // Create a new parser
public Parser() {
@@ -102,9 +115,9 @@ public class Parser implements Serializable {
@SuppressWarnings("unchecked")
public String[] getSymbols() {
String[] symbols = new String[getJep().getSymbolTable().size()];
symbols = (String[])getJep().getSymbolTable().keySet().toArray(new String[symbols.length]);
return symbols;
Set<String> symbols = new LinkedHashSet<>(getJep().getSymbolTable().keySet());
symbols.removeAll(RESERVED_SYMBOLS);
return symbols.toArray(new String[symbols.size()]);
}
private void addGenericFunctions() throws Exception {
@@ -685,7 +685,7 @@ public class Transformer {
}
}
} catch(InvalidAccessException e) {
e.printStackTrace();
//e.printStackTrace();
errors.add("Transformer] ITEM Error during conversion (source message access) - continue- " + e.getMessage());
value ="";
findSource = false;
@@ -222,6 +222,7 @@ public class Converter {
}
String[] data = normalized.split("[.]");
if (pointLength > 0) {
// length 는 소수점(.) 문자를 포함한 전체 길이 규약. 정수부 허용 자릿수 = length - pointLength - 1
if (data.length == 1) {
if (data[0].length() > length - pointLength - 1) return false;
} else if (data.length == 2) {
@@ -1,7 +1,6 @@
package com.eactive.eai.transformer.message;
import java.math.BigDecimal;
import java.text.DecimalFormat;
import java.util.HashMap;
import java.util.Iterator;
import java.util.Stack;
@@ -338,14 +337,15 @@ public class JSONMessage extends Message {
if( parent instanceof ObjectNode ) {
obj = (ObjectNode)parent;
String val = getStringValue(item.getValue(),item);
// 출력 시점 자릿수 검증 (변환으로 생성된 값은 setData 입력검증을 거치지 않음)
validateDecimal(item.getValue(), item);
// String인 경우에만 Masking 이 가능 - 숫자는 불가능
int itemType = item.getDataTypeId();
if (isMask && "Y".equals(item.getMask()) && (Types.TYPE_STRING == itemType )){
val = maskString2(val, item.getMaskLength(), item.getMaskOffset());
String val = maskString2(getStringValue(item.getValue(),item), item.getMaskLength(), item.getMaskOffset());
item.setValue(val);
}
// Object value = toJsonObjectType(item.getDataTypeId(), val);
String value = val;
if( item.getOccMax() > 1 || item.getOccMax() == -1 ) { // FIELD ARRAY FORMAT "name":["value1", "value2"]
arr = (ArrayNode)obj.get(item.getName());
if (arr == null){
@@ -480,41 +480,22 @@ public class JSONMessage extends Message {
}
}
/**
* item 값을 문자열로 변환한다. 마스킹 대상(TYPE_STRING) 필드에서만 사용된다.
*/
protected String getStringValue(Object value, Item item) throws Exception{
if (value == null) return null;
String val="";
if (value instanceof String){
String data = (String)value;
if (item.getDataTypeId() == Types.TYPE_BIGDECIMAL){
val = getFormat(data,item.getLength(),item.getDecimalPointLength());
}else{
val = data;
}
val = (String)value;
} else if (value instanceof byte[]){
String data = new String((byte[])value);
if (item.getDataTypeId() == Types.TYPE_BIGDECIMAL){
val = getFormat(data,item.getLength(),item.getDecimalPointLength());
}else{
val = data;
}
val = new String((byte[])value);
} else if ( value instanceof BigDecimal) {
String data = ((BigDecimal) value).toPlainString();
if (item.getDataTypeId() == Types.TYPE_BIGDECIMAL){
val = getFormat(data,item.getLength(),item.getDecimalPointLength());
}else{
val = data;
}
val = ((BigDecimal) value).toPlainString();
} else if ( value instanceof Integer) {
String data = String.valueOf(value);
if (item.getDataTypeId() == Types.TYPE_BIGDECIMAL){
val = getFormat(data,item.getLength(),item.getDecimalPointLength());
}else{
val = data;
}
val = String.valueOf(value);
} else if ( value instanceof Boolean) {
String data = String.valueOf(value);
val = data;
val = String.valueOf(value);
}
return val;
}
@@ -591,83 +572,47 @@ public class JSONMessage extends Message {
}
}
private String getFormat(String value,int length,int pointLength) throws Exception {
boolean validation = SystemKeys.isValidationEnabled();
return getFormat(value, length, pointLength, validation) ;
}
private String getFormat(String value, int length, int pointLength, boolean isValidation) throws Exception{
if(isValidation) {
if (!Converter.isValid(value,length,pointLength)){
throw new Exception(String.format("Invalid decimal point format Error value=%s,length=%d,pointLength=%d"
, value,length,pointLength)
);
}
boolean isMinus=false;
String pattern = "";
//음수/양수 check
if (value.indexOf("-")>=0){
isMinus = true;
}
String data = value.replaceAll(" ", "").replaceAll("-", "");
if (pointLength> 0 ){//소수부가 있을 경우
for(int i=isMinus?1:0;i<length-pointLength-1;i++){
pattern = pattern + "#";
}
pattern =pattern+".";
for(int i=0;i<pointLength;i++){
pattern = pattern + "0";
}
}else{//소수부가 없을경우
for(int i=isMinus?1:0;i<length-pointLength;i++){
pattern = pattern + "#";
}
}
if (isMinus) {
pattern="-"+pattern;
}
DecimalFormat format = new DecimalFormat(pattern);
// double num = Double.parseDouble(data);
BigDecimal num = new BigDecimal(data);
return format.format(num);
/**
* BIGDECIMAL 필드의 출력값 검증.
*
* 변환으로 생성된 target 메시지의 값은 setData() 입력검증을 거치지 않으므로
* (Message.setBigDecimal() 에 검증 없음) 출력 시점에 레이아웃 자릿수를 확인한다.
*
* 이전에는 DecimalFormat 으로 자릿수를 맞춘 문자열을 반환하는 getFormat() 이었으나,
* 출력이 setJsonNodeValue() 에서 item.getValue() 를 직접 쓰는 방식으로 바뀌면서
* 반환값이 사용되지 않아 포맷 로직을 제거하고 검증만 남김.
*/
private void validateDecimal(Object value, Item item) throws Exception {
if (value == null || item.getDataTypeId() != Types.TYPE_BIGDECIMAL) {
return;
}
else {
BigDecimal num = new BigDecimal(value);
return num.toPlainString();
String data;
if (value instanceof String) {
data = (String) value;
} else if (value instanceof byte[]) {
data = new String((byte[]) value);
} else if (value instanceof BigDecimal) {
data = ((BigDecimal) value).toPlainString();
} else if (value instanceof Integer) {
data = String.valueOf(value);
} else {
// 그 외 타입은 검증 대상 아님
return;
}
}
private boolean isValid(String value,int length,int pointLength){
String[] data = value.replaceAll(" ", "").split("[.]");
if (pointLength > 0 ){
if (data.length ==1){
if (data[0].length() > length - pointLength - 1 ){ //정수부 오류
return false;
}
}else if (data.length ==2){
if (data[0].length() > length - pointLength - 1 ){ //정수부 오류
return false;
}
if (data[1].length() > pointLength ){ //실수부 오류
return false;
}
}else{
//오류
return false;
}
}else{
if (data.length != 1){ //소수점이 들어와서 오류
return false;
}
if (data[0].length() > length - pointLength - 1){ //정수부 오류
return false;
}
int length = item.getLength();
int pointLength = item.getDecimalPointLength();
if (SystemKeys.isValidationEnabled() && !Converter.isValid(data, length, pointLength)) {
throw new InvalidAccessException(
String.format("Invalid decimal point format Error value=%s,length=%d,pointLength=%d", data, length,
pointLength),
item.getItemPath());
}
return true;
// 숫자형 여부 확인 - 파싱 불가 시 NumberFormatException
new BigDecimal(data.replaceAll(" ", ""));
}
// private Object toJsonObjectType(int itemType, String value) {
// Object jsonValueObject = null;
//
@@ -44,7 +44,7 @@ public class TransformValidator {
try {
if (!transform.getTransformItemList().iterator().hasNext()) {
// throw new TransformerException("source layout does not exist");
throw new TransformerException("전문레이아웃 매핑에 원천 레이아웃이 없습니다.");
throw new TransformerException("전문레이아웃 매핑에 변환 항목(TSEAITR03)이 없습니다.");
}
if (transform.getTargetLayout() == null) {
@@ -202,7 +202,7 @@ public class TransformValidator {
try {
if (!transform.getTransformItemList().iterator().hasNext()) {
// throw new TransformerException("source layout does not exist");
throw new TransformerException("전문레이아웃 매핑에 원천 레이아웃이 없습니다.");
throw new TransformerException("전문레이아웃 매핑에 변환 항목(TSEAITR03)이 없습니다.");
}
if (transform.getTargetLayout() == null) {
@@ -146,17 +146,17 @@ class GetTrinomialFunctionTest {
// ─────────────────────────────────────────────────────────────────
@Test
@DisplayName("실전 패턴 — true/false 결과값 (JEP 상수 1.0/0.0 반환 검증)")
@DisplayName("실전 패턴 — true/false 결과값 (Boolean 타입 반환 검증)")
void testBooleanLiteralReturnValue() {
// JEP addStandardConstants() true=1.0, false=0.0 으로 등록하는지 확인
// → 등록되어 있다면 변수 바인딩 없이 숫자로 반환됨
// → 등록 안 됐다면 파싱 오류 또는 undeclared 변수(NaN)로 처리됨
// JEP addStandardConstants() 에는 true/false 가 없다.
// Parser 가 이를 Boolean 으로 직접 등록하므로(Parser.addBooleanConstants),
// 결과값은 숫자 1.0/0.0 이 아니라 Boolean 이다. setUp 도 동일하게 바인딩한다.
jep.addVariable("intVal", 10.0);
Object result = eval("getTrinomial(intVal, 10, \"=\", true, false)");
assertTrue(result instanceof Number,
"true/false 결과값이 Number 타입이어야 함 (실제: " + result.getClass().getSimpleName() + ")");
assertEquals(1.0, ((Number) result).doubleValue(), 0.0,
"조건 참일 때 true(=1.0) 반환 기대");
assertTrue(result instanceof Boolean,
"true/false 결과값이 Boolean 타입이어야 함 (실제: " + result.getClass().getSimpleName() + ")");
assertEquals(Boolean.TRUE, result,
"조건 참일 때 true 반환 기대");
}
@Test
@@ -197,9 +197,9 @@ class GetTrinomialFunctionTest {
assertFalse(jep.hasError(), "평가 오류: " + jep.getErrorInfo());
}
assertTrue(result instanceof Number,
"결과값이 Number 타입이어야 함 (실제: " + (result == null ? "null" : result.getClass().getSimpleName()) + ")");
assertEquals(1.0, ((Number) result).doubleValue(), 0.0,
"INT_VALUE=10, 비교값=10, 연산자=등호 → 참(true=1.0) 반환 기대");
assertTrue(result instanceof Boolean,
"결과값이 Boolean 타입이어야 함 (실제: " + (result == null ? "null" : result.getClass().getSimpleName()) + ")");
assertEquals(Boolean.TRUE, result,
"INT_VALUE=10, 비교값=10, 연산자=등호 → 참(true) 반환 기대");
}
}
@@ -27,6 +27,7 @@ import com.eactive.eai.transformer.message.MessageSetDataException;
* - INT_VALUE : INT, length=5, decimal=0
* - LONG_VALUE : LONG, length=10, decimal=0
* - BIG_VALUE : BIGDECIMAL, length=10, decimal=5 → 정수부 최대 4자리
* (length 는 소수점(.) 문자를 포함한 전체 길이 규약)
* - BIG15_VALUE : BIGDECIMAL, length=15, decimal=0 → 정수부 최대 15자리
* - STR_VALUE : STRING, length=20
*
@@ -24,7 +24,7 @@ class LayoutRepositoryTest {
@Test
void testFindById() {
LayoutEntity layoutEntity = layoutLoaderRepository.getReferenceById("TST_TTST00000002_RES");
LayoutEntity layoutEntity = layoutLoaderRepository.getReferenceById("TST_TOSSTEST1_TGT");
layoutEntity = layoutEntity.getLayoutItemEntities().stream().findAny().get().getLayoutEntity();
assertNotNull(layoutEntity);
System.out.println(layoutEntity);
@@ -26,7 +26,8 @@ import com.eactive.eai.transformer.message.MessageFactory;
@ContextConfiguration(classes = { TransformJPATestConfiguration.class })
@Sql({ "/com/eactive/eai/transformer/init_tseaitr06.sql", "/com/eactive/eai/transformer/init_tseaitr07.sql",
"/com/eactive/eai/transformer/init_tseaitr08.sql", "/com/eactive/eai/transformer/init_tseaitr01.sql",
"/com/eactive/eai/transformer/init_tseaitr02.sql", "/com/eactive/eai/transformer/init_tseaitr03.sql" })
"/com/eactive/eai/transformer/init_tseaitr02.sql", "/com/eactive/eai/transformer/init_tseaitr03.sql",
"/com/eactive/eai/transformer/init_tseaitr01_noitem.sql" })
@TestInstance(TestInstance.Lifecycle.PER_CLASS)
class TransformManagerTest {
@@ -240,4 +241,18 @@ class TransformManagerTest {
}
/**
* TSEAITR03(변환 항목 매핑) 건도 없더라도 TSEAITR01/02 등록된 변환은 로딩되어야 한다.
* loadAll() inner join fetch 이면 변환은 transformMap 에서 누락되고,
* 호출 "전문레이아웃매핑(변환)정보를 찾을 수 없음" 오류가 발생한다.
*/
@Test
void testTransformWithoutTransformItemIsLoaded() {
Transform transform = transformManager.getTransform("TRANSFORM_TST_NOITEM_REQ");
Assert.assertNotNull("TSEAITR03 매핑이 없는 변환도 로딩되어야 함", transform);
Assert.assertEquals(0, transform.getTransformItemList().size());
Assert.assertEquals("TST_TOSSTEST1_TGT", transform.getTargetLayout().getName());
}
}
@@ -0,0 +1,9 @@
-- 변환 마스터(TSEAITR01) / 레이아웃 지정(TSEAITR02)은 있으나
-- 변환 항목 매핑(TSEAITR03)은 한 건도 없는 변환.
-- loadAll() 이 inner join fetch 를 쓰면 이 변환은 로딩에서 누락된다.
INSERT INTO tseaitr01 (cnvsnname,cnvsndesc,lastamndhms,eaisevrdstcd,modfimgtstusdstcd,useyn,verinfo,author) VALUES
('TRANSFORM_TST_NOITEM_REQ', 'TSEAITR03 매핑 항목 없음', '20241226104652 ','E',' ','1','0',NULL);
INSERT INTO tseaitr02 (cnvsnname,loutname,sourcrsultdstcd) VALUES
('TRANSFORM_TST_NOITEM_REQ', 'TST_TOSSTEST1_TGT', 'TGT_LAYOUT'),
('TRANSFORM_TST_NOITEM_REQ', 'TST_TOSSTEST1_SRC', 'SRC_LAYOUT');