fix bug to support commands that don't take arguments; modify testing accordingly

Author: tepickering

saomsg/client.py

@@ -91,8 +91,12 @@ async def run(self, command, *pars):
         if command not in self.server_info['registered']:
             errmsg = f"{command} not registered by MSG server {self.server_info['name']}"
             raise ValueError(errmsg)
-        params = " ".join(str(x) for x in pars)
-        msg = f"1 {command} {params}\n"
+        if len(pars) > 0:
+            params = " ".join(str(x) for x in pars)
+            msg = f"1 {command} {params}\n"
+        else:
+            params = "<None>"
+            msg = f"1 {command}\n"
         data = await self._writemsg(msg)
         if int(data[0]) == 1 and data[1] == "ack":
             value = True

saomsg/tests/test_client.py

@@ -28,9 +28,12 @@ async def test_get():
 async def test_cmd():
     c = MSGClient()
     await c.open()
-    await c.run("multiply", 4, 5)
+    status = await c.run("multiply", 4, 5)
+    assert(status)
     foo = await c.get("foo")
     assert(int(foo) == 20)
+    status = await c.run("blurb")
+    assert(status)
     await c.close()
 
 

scripts/msg_test.tcl

@@ -23,5 +23,10 @@ proc TESTSRV.multiply { s sock msgid cmd x y } {
     msg_ack $sock $msgid
 }
 
+msg_register TESTSRV blurb
+proc TESTSRV.blurb { s sock msgid cmd } {
+    msg_ack $sock $msgid
+}
+
 msg_up TESTSRV
 vwait forever