commit 6fed0531f3df3d7ca15972f635f946f19868c496
author Vitaly Buka <vitalybuka@chromium.org> Thu May 14 16:57:23 2015 -0700
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Mon Jun 01 06:20:14 2015 +0000
tree 7eff5a77b4374fb7989448c3a149cb3fda511ee2
parent 6b028ae747f94bd9201f71690a9aa7c0ad585f5f

buffet: Added minimal role into command definition

Possible values "viewer", "user", "owner", "manager".
Default minimal role is set to "user".
Role check is not yet implemented.

BUG=brillo:808
TEST=`FEATURES=test emerge-gizmo buffet`

Change-Id: I74d16c24618b50de1e7a2ffa37d7aef36978a38a
Reviewed-on: https://chromium-review.googlesource.com/274117
Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>
Tested-by: Vitaly Buka <vitalybuka@chromium.org>

buffet/device_registration_info_unittest.cc

diff --git a/buffet/device_registration_info_unittest.cc b/buffet/device_registration_info_unittest.cc
index c3088c4..c791355 100644
--- a/buffet/device_registration_info_unittest.cc
+++ b/buffet/device_registration_info_unittest.cc
@@ -373,7 +373,8 @@
               'minimum': 10,
               'type': 'integer'
             }
-          }
+          },
+          'minimalRole': 'user'
         }
       },
       'robot': {
@@ -382,7 +383,8 @@
             '_height': {
               'type': 'integer'
             }
-          }
+          },
+          'minimalRole': 'user'
         }
       }
     })";
@@ -406,10 +408,12 @@
     'base': {
       'reboot': {
         'parameters': {'delay': 'integer'},
+        'minimalRole': 'user',
         'results': {}
       },
       'shutdown': {
         'parameters': {},
+        'minimalRole': 'user',
         'results': {}
       }
     }
@@ -419,12 +423,14 @@
     'base': {
       'reboot': {
         'parameters': {'delay': {'minimum': 10}},
+        'minimalRole': 'user',
         'results': {}
       }
     },
     'robot': {
       '_jump': {
         'parameters': {'_height': 'integer'},
+        'minimalRole': 'user',
         'results': {}
       }
     }
@@ -481,7 +487,8 @@
     'robot': {
       '_jump': {
         'parameters': {'_height': 'integer'},
-        'results': {'status': 'string'}
+        'results': {'status': 'string'},
+        'minimalRole': 'user'
       }
     }
   })");
@@ -492,7 +499,8 @@
   auto commands_json = unittests::CreateValue(R"([{
     'name':'robot._jump',
     'id':'1234',
-    'parameters': {'_height': 100}
+    'parameters': {'_height': 100},
+    'minimalRole': 'user'
   }])");
   ASSERT_NE(nullptr, commands_json.get());
   const base::ListValue* command_list = nullptr;