plugin_control: more descriptive key for 'plugin stop' result

ElementsProject · Dec 25, 2019 · c6b25ea · c6b25ea
1 parent 689dd28
commit c6b25ea
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lightningd/plugin_control.c b/lightningd/plugin_control.c
@@ -206,7 +206,7 @@ plugin_dynamic_stop(struct command *cmd, const char *plugin_name)
 			plugin_kill(p, "%s stopped by lightningd via RPC", plugin_name);
 			tal_free(p);
 			response = json_stream_success(cmd);
-			json_add_string(response, "",
+			json_add_string(response, "result",
 			                take(tal_fmt(NULL, "Successfully stopped %s.",
 			                             plugin_name)));
 			return command_success(cmd, response);

diff --git a/tests/test_plugin.py b/tests/test_plugin.py
@@ -127,14 +127,16 @@ def test_plugin_command(node_factory):
     n.rpc.plugin_list()
 
     # Make sure the plugin behaves normally after stop and restart
-    assert("Successfully stopped helloworld.py." == n.rpc.plugin_stop(plugin="helloworld.py")[''])
+    assert("Successfully stopped helloworld.py."
+           == n.rpc.plugin_stop(plugin="helloworld.py")["result"])
     n.daemon.wait_for_log(r"Killing plugin: helloworld.py")
     n.rpc.plugin_start(plugin=os.path.join(os.getcwd(), "contrib/plugins/helloworld.py"))
     n.daemon.wait_for_log(r"Plugin helloworld.py initialized")
     assert("Hello world" == n.rpc.call(method="hello"))
 
     # Now stop the helloworld plugin
-    assert("Successfully stopped helloworld.py." == n.rpc.plugin_stop(plugin="helloworld.py")[''])
+    assert("Successfully stopped helloworld.py."
+           == n.rpc.plugin_stop(plugin="helloworld.py")["result"])
     n.daemon.wait_for_log(r"Killing plugin: helloworld.py")
     # Make sure that the 'hello' command from the helloworld.py plugin
     # is not available anymore.