Browse Source

[trac931] Send the correct name of module

Michal 'vorner' Vaner 14 years ago
parent
commit
ff008c094a
2 changed files with 8 additions and 1 deletions
  1. 1 1
      src/lib/config/ccsession.cc
  2. 7 0
      src/lib/config/tests/ccsession_unittests.cc

+ 1 - 1
src/lib/config/ccsession.cc

@@ -372,7 +372,7 @@ ModuleCCSession::addRemoteConfig(const std::string& spec_name,
         ConstElementPtr cmd = Element::fromJSON("{ \"command\": ["
                                                 "\"get_module_spec\","
                                                 "{\"module_name\": \"" +
-                                                module_name + "\"} ] }");
+                                                spec_name + "\"} ] }");
         unsigned int seq = session_.group_sendmsg(cmd, "ConfigManager");
         ConstElementPtr env, answer;
         session_.group_recvmsg(env, answer, false, seq);

+ 7 - 0
src/lib/config/tests/ccsession_unittests.cc

@@ -414,6 +414,13 @@ TEST_F(CCSessionTest, remoteConfig) {
 
         EXPECT_NO_THROW(module_name = mccs.addRemoteConfig("Spec2"));
 
+        size_t qsize(session.getMsgQueue()->size());
+        EXPECT_TRUE(session.getMsgQueue()->get(qsize - 2)->equals(*el(
+            "[ \"ConfigManager\", \"*\", { \"command\": ["
+            "\"get_module_spec\", { \"module_name\": \"Spec2\" } ] } ]")));
+        EXPECT_TRUE(session.getMsgQueue()->get(qsize - 1)->equals(*el(
+            "[ \"ConfigManager\", \"*\", { \"command\": [ \"get_config\","
+            "{ \"module_name\": \"Spec2\" } ] } ]")));
         EXPECT_EQ("Spec2", module_name);
         EXPECT_NO_THROW(item1 =
                         mccs.getRemoteConfigValue(module_name,