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/command_instance_unittest.cc

diff --git a/buffet/commands/command_instance_unittest.cc b/buffet/commands/command_instance_unittest.cc
index b57de25..d5afc70 100644
--- a/buffet/commands/command_instance_unittest.cc
+++ b/buffet/commands/command_instance_unittest.cc
@@ -21,7 +21,8 @@
     auto json = CreateDictionaryValue(R"({
       'base': {
         'reboot': {
-          'parameters': {}
+          'parameters': {},
+          'results': {}
         }
       },
       'robot': {
@@ -36,7 +37,8 @@
               'type': 'string',
               'enum': ['_withAirFlip', '_withSpin', '_withKick']
             }
-          }
+          },
+          'results': {}
         },
         'speak': {
           'parameters': {
@@ -50,7 +52,8 @@
               'minimum': 0,
               'maximum': 10
             }
-          }
+          },
+          'results': {}
         }
       }
     })");
@@ -92,7 +95,8 @@
     'parameters': {
       'height': 53,
       '_jumpType': '_withKick'
-    }
+    },
+    'results': {}
   })");
   auto instance = buffet::CommandInstance::FromJson(json.get(), dict_, nullptr);
   EXPECT_EQ("robot.jump", instance->GetName());