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>
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;