This is an automated email from the ASF dual-hosted git repository.

yhcast0 pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/kylin5 by this push:
     new 847482c709 [MINOR] Fix ut on test resource json (#2339)
847482c709 is described below

commit 847482c709b1f76dd169923ad25ea8903f08f5ff
Author: Yinghao Lin <[email protected]>
AuthorDate: Tue Sep 16 15:29:54 2025 +0800

    [MINOR] Fix ut on test resource json (#2339)
---
 .../src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java | 4 +++-
 .../ut_meta/internal_measure.model_desc/nmodel_test_expected.json     | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
 
b/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
index d29e83308a..55e19c629e 100644
--- 
a/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
+++ 
b/src/modeling-service/src/test/java/org/apache/kylin/rest/service/ModelServiceTest.java
@@ -86,6 +86,7 @@ import org.apache.commons.lang3.RandomStringUtils;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.commons.lang3.tuple.ImmutablePair;
 import org.apache.kylin.common.KylinConfig;
+import org.apache.kylin.common.KylinVersion;
 import org.apache.kylin.common.exception.KylinException;
 import org.apache.kylin.common.msg.MsgPicker;
 import org.apache.kylin.common.persistence.JsonSerializer;
@@ -4668,7 +4669,8 @@ public class ModelServiceTest extends SourceTestCase {
         String expected = FileUtils
                 .readFileToString(
                         new 
File("src/test/resources/ut_meta/internal_measure.model_desc/nmodel_test_expected.json"))
-                .trim();
+                .trim()
+                .replace("%default_version%", 
KylinVersion.getCurrentVersion().toString());
         Assert.assertEquals(expected, dump);
 
         val index = NIndexPlanManager.getInstance(getTestConfig(), 
getProject()).getIndexPlan(saved.getId());
diff --git 
a/src/modeling-service/src/test/resources/ut_meta/internal_measure.model_desc/nmodel_test_expected.json
 
b/src/modeling-service/src/test/resources/ut_meta/internal_measure.model_desc/nmodel_test_expected.json
index 0f92fa5726..cdaae41857 100644
--- 
a/src/modeling-service/src/test/resources/ut_meta/internal_measure.model_desc/nmodel_test_expected.json
+++ 
b/src/modeling-service/src/test/resources/ut_meta/internal_measure.model_desc/nmodel_test_expected.json
@@ -1 +1 @@
-{"uuid":"","last_modified":0,"create_time":0,"version":"5.0.0.0-SNAPSHOT","project":"default","alias":null,"owner":null,"config_last_modifier":null,"config_last_modified":0,"description":"","fact_table":null,"fact_table_alias":null,"management_type":"MODEL_BASED","filter_condition":null,"partition_desc":null,"capacity":"MEDIUM","segment_config":{"auto_merge_enabled":null,"auto_merge_time_ranges":null,"volatile_range":null,"retention_range":null,"create_empty_segment_enabled":false},"data
 [...]
+{"uuid":"","last_modified":0,"create_time":0,"version":"%default_version%","project":"default","alias":null,"owner":null,"config_last_modifier":null,"config_last_modified":0,"description":"","fact_table":null,"fact_table_alias":null,"management_type":"MODEL_BASED","filter_condition":null,"partition_desc":null,"capacity":"MEDIUM","segment_config":{"auto_merge_enabled":null,"auto_merge_time_ranges":null,"volatile_range":null,"retention_range":null,"create_empty_segment_enabled":false},"dat
 [...]

Reply via email to