Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
418b2a1
Introduce LocationBasedServices command_module
jp94 Feb 23, 2018
8bbcd51
Introduce test cases for LocationBasedServices command_module
jp94 Feb 23, 2018
18d9384
[Refactor] Renamed lbs to locationbasedservices
jp94 Feb 24, 2018
cb738db
[Refactor] Renamed lbs to locationbasedservices
jp94 Feb 24, 2018
b74cb79
[Legal] Add the Preview Terms agreement requirement
jp94 Feb 24, 2018
409b03b
[Test] Add more strict assertion rule for key validation.
jp94 Feb 26, 2018
ed3ca01
[Test] Add test for tags parameter
jp94 Feb 26, 2018
81e4fb5
[Test] Added an additional resource group.
jp94 Feb 26, 2018
0d11a6d
[Test] Init recordings
jp94 Feb 26, 2018
87accc7
Merge remote-tracking branch 'upstream/dev' into dev
jp94 Feb 26, 2018
9e5051f
[Pylint] Specify string format arguments as logging function parameters
jp94 Feb 26, 2018
77a9f22
[Refactor] Update description in _help to match the general template
jp94 Feb 27, 2018
98d973e
[Refactor] General template mismatch fix for 'key'
jp94 Feb 27, 2018
5853331
[Feature] Introduce search by ids
jp94 Feb 28, 2018
4d930f8
[Temporary] Throw an exception on 'account show' command, when nonexist
jp94 Feb 28, 2018
d36abf9
[Test] 'account show' on non-existent account should not return empty.
jp94 Feb 28, 2018
b79cb10
[Feature] Introduce update command
jp94 Mar 1, 2018
75795e6
[Test] Add test for update and search by id
jp94 Mar 1, 2018
100df29
Merge remote-tracking branch 'upstream/dev' into dev
jp94 Mar 20, 2018
1fd7bef
[Refactor] Rename generic client (sync with updated Swagger)
jp94 Mar 21, 2018
1aee6b2
[Refactor] Use default SDK's function
jp94 Mar 21, 2018
35976e7
[Refactor] Remove Preview Terms & docs in custom.py
jp94 Mar 21, 2018
e7784a1
[Maps] Rename LocationBasedServices to Maps
jp94 May 7, 2018
f9ca47d
[Maps] Use Maps SDK from LocationBasedServices SDK
jp94 May 7, 2018
52b9dde
[Maps] Update dependencies
jp94 May 7, 2018
c0c8bb8
[Maps] Resolving build error (doc_source_map)
jp94 May 7, 2018
498ca9c
Merge remote-tracking branch 'upstream/dev' into rename
jp94 May 8, 2018
4176c07
Merge remote-tracking branch 'upstream/dev' into rename
jp94 May 9, 2018
c6c5c02
[Maps] Resolving PR issues.
jp94 May 9, 2018
31f7a3a
[Maps] Revise help text
jp94 May 9, 2018
d49f8ad
[Maps] Adding back help docs for maps account
jp94 May 9, 2018
1f3d2c1
Merge remote-tracking branch 'upstream/dev' into rename
jp94 May 12, 2018
19b2fe9
[Load Test] replace sql with servicefabric
jp94 May 14, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[Refactor] General template mismatch fix for 'key'
  • Loading branch information
jp94 committed Feb 27, 2018
commit 98d973eedf15abf4ef947d514b4496130f2a2d75
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
short-summary: Manage Azure Location Based Services accounts.
"""

helps['locationbasedservices account key'] = """
helps['locationbasedservices account keys'] = """
type: group
short-summary: Manage Azure Location Based Services account keys.
"""
Expand All @@ -42,16 +42,16 @@
short-summary: Delete a Location Based Services account.
"""

helps['locationbasedservices account key list'] = """
helps['locationbasedservices account keys list'] = """
type: command
short-summary: List the keys to use with the Location Based Services APIs.
long-summary: |
List the keys to use with the Location Based Services APIs. A key is used to authenticate and authorize access to the Location Based Services REST APIs. Only one key is needed at a time; two are given to provide seamless key regeneration.
"""

helps['locationbasedservices account key regenerate'] = """
helps['locationbasedservices account keys renew'] = """
type: command
short-summary: Regenerate either the primary or secondary key for use with the Location Based Services APIs.
short-summary: Renew either the primary or secondary key for use with the Location Based Services APIs.
long-summary: |
Regenerate either the primary or secondary key for use with the Location Based Services APIs. The old key will stop working immediately.
Renew either the primary or secondary key for use with the Location Based Services APIs. The old key will stop working immediately.
"""
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ def load_arguments(self, _):
help='You agree to the Preview Terms. Ignore prompt for confirmation.',
action='store_true')

with self.argument_context('locationbasedservices account key regenerate') as c:
with self.argument_context('locationbasedservices account keys regenerate') as c:
c.argument('key_type',
options_list=['--type', '-t'],
options_list=['--key'],
arg_type=get_enum_type(KeyType))
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,6 @@ def load_command_table(self, _):
g.custom_command('create', 'create')
g.command('delete', 'delete')

with self.command_group('locationbasedservices account key', mgmt_type) as g:
g.command('regenerate', 'regenerate_keys')
with self.command_group('locationbasedservices account keys', mgmt_type) as g:
g.command('renew', 'regenerate_keys')
g.command('list', 'list_keys')
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ def test_create_locationbasedservices_account(self, resource_group):

# Test 'az locationbasedservices account key list'.
# Test to list keys for an LocationBasedServices account.
account_key_list = self.cmd('az locationbasedservices account key list -n {name} -g {rg}', checks=[
account_key_list = self.cmd('az locationbasedservices account keys list -n {name} -g {rg}', checks=[
self.check('id', account['id']),
self.check('resourceGroup', '{rg}')
]).get_output_in_json()
Expand All @@ -117,7 +117,7 @@ def test_create_locationbasedservices_account(self, resource_group):
# Test 'az locationbasedservices account key regenerate'.
# Test to change primary and secondary keys for an LocationBasedServices account.
key_regenerated = self.cmd(
'az locationbasedservices account key regenerate -n {name} -g {rg} -t {key_type_primary}', checks=[
'az locationbasedservices account keys renew -n {name} -g {rg} -t {key_type_primary}', checks=[
self.check('id', account['id']),
self.check('resourceGroup', '{rg}')
]).get_output_in_json()
Expand All @@ -129,7 +129,7 @@ def test_create_locationbasedservices_account(self, resource_group):
# Save the new primary key, and regenerate the secondary key.
primary_key_old = key_regenerated['primaryKey']
key_regenerated = self.cmd(
'az locationbasedservices account key regenerate -n {name} -g {rg} -t {key_type_secondary}') \
'az locationbasedservices account keys renew -n {name} -g {rg} -t {key_type_secondary}') \
.get_output_in_json()
self.assertEqual(primary_key_old, key_regenerated['primaryKey'])
self.assertNotEqual(secondary_key_old, key_regenerated['secondaryKey'])
Expand Down