commit 71fb9d59fbe7136e3dd92f691e6ed3fee0b31d6d
author Anton Muhin <antonm@google.com> Fri Nov 21 22:22:39 2014 +0400
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Sat Nov 22 01:41:10 2014 +0000
tree cecfbf00999eb1ee2970b491f608153cb5eae04c
parent 76933fdbf7ec8e05ca0549db1e1f9df9ec3eb9a6

buffet: Provide results definition in command definition.

That allows us to provide typed results for supported commands.

BUG=chromium:435607
TEST=cros_workon_make --test buffet

Change-Id: I61b5a5b294b4d869366c821adf1ef7f6db31c7ea
Reviewed-on: https://chromium-review.googlesource.com/231322
Reviewed-by: Anton Muhin <antonm@chromium.org>
Commit-Queue: Anton Muhin <antonm@chromium.org>
Tested-by: Anton Muhin <antonm@chromium.org>

buffet/commands/dbus_command_proxy_unittest.cc

diff --git a/buffet/commands/dbus_command_proxy_unittest.cc b/buffet/commands/dbus_command_proxy_unittest.cc
index 3cb907d..81d9827 100644
--- a/buffet/commands/dbus_command_proxy_unittest.cc
+++ b/buffet/commands/dbus_command_proxy_unittest.cc
@@ -50,6 +50,7 @@
     EXPECT_CALL(*bus_, AssertOnDBusThread()).Times(AnyNumber());
 
     // Command instance.
+    // TODO(antonm): Test results.
     auto json = CreateDictionaryValue(R"({
       'robot': {
         'jump': {
@@ -63,7 +64,8 @@
               'type': 'string',
               'enum': ['_withAirFlip', '_withSpin', '_withKick']
             }
-          }
+          },
+          'results': {}
         }
       }
     })");