test(config): better describe behavior in resolving circular dependen…

…cies
rime · Sep 21, 2017 · b685d40 · b685d40
1 parent 2e52d54
commit b685d40
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/data/test/config_circular_dependency_test.yaml b/data/test/config_circular_dependency_test.yaml
@@ -1,8 +1,8 @@
 test:
-  __include: sometimes?
+  __patch: sometimes?
   home: excited
   work:
     __include: /test/home
 
 sometimes:
-  work: naive
+  home: naive
diff --git a/test/config_compiler_test.cc b/test/config_compiler_test.cc
@@ -253,8 +253,11 @@ class RimeConfigCircularDependencyTest : public RimeConfigCompilerTestBase {
 
 TEST_F(RimeConfigCircularDependencyTest, BestEffortResolution) {
   const string& prefix = "test/";
-  EXPECT_TRUE(config_->IsNull(prefix + "__include"));
+  EXPECT_TRUE(config_->IsNull(prefix + "__patch"));
   EXPECT_TRUE(config_->IsNull(prefix + "work/__include"));
+  string home;
+  EXPECT_TRUE(config_->GetString(prefix + "home", &home));
+  EXPECT_EQ("naive", home);
   string work;
   EXPECT_TRUE(config_->GetString(prefix + "work", &work));
   EXPECT_EQ("excited", work);