Merge pull request #301 from WeDataSphere/dev-1.1.16-fix-conf-error

add configration test
WeDataSphere · Oct 7, 2023 · 42f5837 · 42f5837
2 parents 395ec85 + 8384fe6
commit 42f5837
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/linkis-commons/linkis-common/src/test/resources/linkis.properties b/linkis-commons/linkis-common/src/test/resources/linkis.properties
@@ -13,4 +13,5 @@
 # limitations under the License.
 #
 
-linkis.jobhistory.error.msg.tip=properties支持中文
+linkis.jobhistory.error.msg.tip=properties支持中文
+linkis.test.error.conf=123 
diff --git a/...ommons/linkis-common/src/test/scala/org/apache/linkis/common/conf/ConfigurationTest.scala b/...ommons/linkis-common/src/test/scala/org/apache/linkis/common/conf/ConfigurationTest.scala
@@ -27,4 +27,11 @@ class ConfigurationTest {
     Assertions.assertFalse(Configuration.isAdmin("HaDooop"))
   }
 
+
+  @Test private[conf] def testFormatValue(): Unit = {
+    val confvalue = CommonVars[Int]("linkis.test.error.conf", 456).getValue
+    Assertions.assertTrue(123 == confvalue)
+  }
+
+
 }