Skip to content

[SQL] Upgrade azure-mgmt-sql to 0.26.0 #16618

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
evelyn-ys merged 27 commits into from
Jan 27, 2021
Merged

[SQL] Upgrade azure-mgmt-sql to 0.26.0 #16618

Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
5d752a9
Add PublicNetworkAccess to Create and Update Server
emgu-ms Feb 28, 2020
9c0c2c7
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
emgu-ms Feb 28, 2020
99f273f
Merge branch 'dev' of https://github.com/Azure/azure-cli into dev
emgu-ms Mar 5, 2020
75a7998
Merge pull request #1 from Azure/dev
OlegSternbergIBM Oct 1, 2020
f530b90
Merge pull request #2 from Azure/dev
OlegSternbergIBM Oct 9, 2020
e2e5cff
Merge pull request #4 from Azure/dev
OlegSternbergIBM Dec 16, 2020
2c39056
Merge pull request #5 from Azure/dev
OlegSternbergIBM Jan 14, 2021
71a2d1b
Merge from upstream
emgu-ms Jan 21, 2021
a3235c1
rerun test
Juliehzl Jan 21, 2021
dc08175
Fix test_sql_server_public_network_access_update_mgmt test
emgu-ms Jan 22, 2021
68682b7
Remove left in testing
emgu-ms Jan 22, 2021
2988242
revert failed test recordings
Juliehzl Jan 22, 2021
6c3565f
Merge branch 'emgu/FixPublicNetworkAccessTest' into sql
Juliehzl Jan 22, 2021
6c2d033
fix recorded tests
Jan 22, 2021
bfe06b8
Merge branch 'fix_recorded_tests_sql_0_26_0' into sql
Juliehzl Jan 22, 2021
e2bd0bf
fix additional tests
Jan 22, 2021
9b48372
fix additional tests
Jan 22, 2021
7640716
use highAvailabilityReplicaCount
Jan 22, 2021
e9b1155
fixed additional recorded tests
Jan 23, 2021
2df75f9
Merge branch 'fix_recorded_tests_sql_0_26_0' into sql
Juliehzl Jan 25, 2021
b405f53
fixed test_sql_db_import_export_mgmt
Jan 25, 2021
2d8a050
* move 'db_import' under correct group
Jan 25, 2021
96b2f4e
Merge branch 'fix_recorded_tests_sql_0_26_0' into sql
Juliehzl Jan 26, 2021
07f1b23
Fix read_replica_count
bradrich-msft Jan 25, 2021
e1eccf8
apply Oleg's change
Juliehzl Jan 26, 2021
128fa3e
fixed test_sql_db_restore.yaml
Jan 26, 2021
0b8e264
test_sql_db_mgmt fix
bradrich-msft Jan 26, 2021
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
Remove left in testing
  • Loading branch information
@emgu-ms
emgu-ms committed Jan 22, 2021
commit 68682b709281fb10b7588699108e495a145de698
170 changes: 85 additions & 85 deletions src/azure-cli/azure/cli/command_modules/sql/tests/latest/test_sql_commands.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,91 +74,91 @@ def _get_resource_group(self, **kwargs):


class SqlServerMgmtScenarioTest(ScenarioTest):
#@ResourceGroupPreparer(parameter_name='resource_group_1', location='westeurope')
#@ResourceGroupPreparer(parameter_name='resource_group_2', location='westeurope')
#def test_sql_server_mgmt(self, resource_group_1, resource_group_2, resource_group_location):
# server_name_1 = self.create_random_name(server_name_prefix, server_name_max_length)
# server_name_2 = self.create_random_name(server_name_prefix, server_name_max_length)
# admin_login = 'admin123'
# admin_passwords = ['SecretPassword123', 'SecretPassword456']

# # test create sql server with minimal required parameters
# server_1 = self.cmd('sql server create -g {} --name {} '
# '--admin-user {} --admin-password {}'
# .format(resource_group_1, server_name_1, admin_login, admin_passwords[0]),
# checks=[
# JMESPathCheck('name', server_name_1),
# JMESPathCheck('location', resource_group_location),
# JMESPathCheck('resourceGroup', resource_group_1),
# JMESPathCheck('administratorLogin', admin_login),
# JMESPathCheck('identity', None)]).get_output_in_json()

# # test list sql server should be 1
# self.cmd('sql server list -g {}'.format(resource_group_1), checks=[JMESPathCheck('length(@)', 1)])

# # test update sql server
# self.cmd('sql server update -g {} --name {} --admin-password {} -i'
# .format(resource_group_1, server_name_1, admin_passwords[1]),
# checks=[
# JMESPathCheck('name', server_name_1),
# JMESPathCheck('resourceGroup', resource_group_1),
# JMESPathCheck('administratorLogin', admin_login),
# JMESPathCheck('identity.type', 'SystemAssigned')])

# # test update without identity parameter, validate identity still exists
# # also use --id instead of -g/-n
# self.cmd('sql server update --id {} --admin-password {}'
# .format(server_1['id'], admin_passwords[0]),
# checks=[
# JMESPathCheck('name', server_name_1),
# JMESPathCheck('resourceGroup', resource_group_1),
# JMESPathCheck('administratorLogin', admin_login),
# JMESPathCheck('identity.type', 'SystemAssigned')])

# # test create another sql server, with identity this time
# self.cmd('sql server create -g {} --name {} -l {} -i '
# '--admin-user {} --admin-password {}'
# .format(resource_group_2, server_name_2, resource_group_location, admin_login, admin_passwords[0]),
# checks=[
# JMESPathCheck('name', server_name_2),
# JMESPathCheck('location', resource_group_location),
# JMESPathCheck('resourceGroup', resource_group_2),
# JMESPathCheck('administratorLogin', admin_login),
# JMESPathCheck('identity.type', 'SystemAssigned')])

# # test list sql server in that group should be 1
# self.cmd('sql server list -g {}'.format(resource_group_2), checks=[JMESPathCheck('length(@)', 1)])

# # test list sql server in the subscription should be at least 2
# self.cmd('sql server list', checks=[JMESPathCheckGreaterThan('length(@)', 1)])

# # test show sql server
# self.cmd('sql server show -g {} --name {}'
# .format(resource_group_1, server_name_1),
# checks=[
# JMESPathCheck('name', server_name_1),
# JMESPathCheck('resourceGroup', resource_group_1),
# JMESPathCheck('administratorLogin', admin_login)])

# self.cmd('sql server show --id {}'
# .format(server_1['id']),
# checks=[
# JMESPathCheck('name', server_name_1),
# JMESPathCheck('resourceGroup', resource_group_1),
# JMESPathCheck('administratorLogin', admin_login)])

# self.cmd('sql server list-usages -g {} -n {}'
# .format(resource_group_1, server_name_1),
# checks=[JMESPathCheck('[0].resourceName', server_name_1)])

# # test delete sql server
# self.cmd('sql server delete --id {} --yes'
# .format(server_1['id']), checks=NoneCheck())
# self.cmd('sql server delete -g {} --name {} --yes'
# .format(resource_group_2, server_name_2), checks=NoneCheck())

# # test list sql server should be 0
# self.cmd('sql server list -g {}'.format(resource_group_1), checks=[NoneCheck()])
@ResourceGroupPreparer(parameter_name='resource_group_1', location='westeurope')
@ResourceGroupPreparer(parameter_name='resource_group_2', location='westeurope')
def test_sql_server_mgmt(self, resource_group_1, resource_group_2, resource_group_location):
server_name_1 = self.create_random_name(server_name_prefix, server_name_max_length)
server_name_2 = self.create_random_name(server_name_prefix, server_name_max_length)
admin_login = 'admin123'
admin_passwords = ['SecretPassword123', 'SecretPassword456']

# test create sql server with minimal required parameters
server_1 = self.cmd('sql server create -g {} --name {} '
'--admin-user {} --admin-password {}'
.format(resource_group_1, server_name_1, admin_login, admin_passwords[0]),
checks=[
JMESPathCheck('name', server_name_1),
JMESPathCheck('location', resource_group_location),
JMESPathCheck('resourceGroup', resource_group_1),
JMESPathCheck('administratorLogin', admin_login),
JMESPathCheck('identity', None)]).get_output_in_json()

# test list sql server should be 1
self.cmd('sql server list -g {}'.format(resource_group_1), checks=[JMESPathCheck('length(@)', 1)])

# test update sql server
self.cmd('sql server update -g {} --name {} --admin-password {} -i'
.format(resource_group_1, server_name_1, admin_passwords[1]),
checks=[
JMESPathCheck('name', server_name_1),
JMESPathCheck('resourceGroup', resource_group_1),
JMESPathCheck('administratorLogin', admin_login),
JMESPathCheck('identity.type', 'SystemAssigned')])

# test update without identity parameter, validate identity still exists
# also use --id instead of -g/-n
self.cmd('sql server update --id {} --admin-password {}'
.format(server_1['id'], admin_passwords[0]),
checks=[
JMESPathCheck('name', server_name_1),
JMESPathCheck('resourceGroup', resource_group_1),
JMESPathCheck('administratorLogin', admin_login),
JMESPathCheck('identity.type', 'SystemAssigned')])

# test create another sql server, with identity this time
self.cmd('sql server create -g {} --name {} -l {} -i '
'--admin-user {} --admin-password {}'
.format(resource_group_2, server_name_2, resource_group_location, admin_login, admin_passwords[0]),
checks=[
JMESPathCheck('name', server_name_2),
JMESPathCheck('location', resource_group_location),
JMESPathCheck('resourceGroup', resource_group_2),
JMESPathCheck('administratorLogin', admin_login),
JMESPathCheck('identity.type', 'SystemAssigned')])

# test list sql server in that group should be 1
self.cmd('sql server list -g {}'.format(resource_group_2), checks=[JMESPathCheck('length(@)', 1)])

# test list sql server in the subscription should be at least 2
self.cmd('sql server list', checks=[JMESPathCheckGreaterThan('length(@)', 1)])

# test show sql server
self.cmd('sql server show -g {} --name {}'
.format(resource_group_1, server_name_1),
checks=[
JMESPathCheck('name', server_name_1),
JMESPathCheck('resourceGroup', resource_group_1),
JMESPathCheck('administratorLogin', admin_login)])

self.cmd('sql server show --id {}'
.format(server_1['id']),
checks=[
JMESPathCheck('name', server_name_1),
JMESPathCheck('resourceGroup', resource_group_1),
JMESPathCheck('administratorLogin', admin_login)])

self.cmd('sql server list-usages -g {} -n {}'
.format(resource_group_1, server_name_1),
checks=[JMESPathCheck('[0].resourceName', server_name_1)])

# test delete sql server
self.cmd('sql server delete --id {} --yes'
.format(server_1['id']), checks=NoneCheck())
self.cmd('sql server delete -g {} --name {} --yes'
.format(resource_group_2, server_name_2), checks=NoneCheck())

# test list sql server should be 0
self.cmd('sql server list -g {}'.format(resource_group_1), checks=[NoneCheck()])

@ResourceGroupPreparer(parameter_name='resource_group_1', location='westeurope')
def test_sql_server_public_network_access_create_mgmt(self, resource_group_1, resource_group_location):
Expand Down