Skip to content

Commit 5d86ea7

Browse files
rewongmicrosoftwyunchi-ms
rewongmicrosoft
and
wyunchi-ms
authored
SQL Ledger Cmdlets (#14997)
* ledger changes + changes for new SDK version * adding tests, updating cmdlets * adding tests, fixing cmdlets * adding comments to locations model * adding to changelog.md * adding markdowe help files * fixing merge problems with backup restore resources * updating help file for new-azsqldatabase * rerecording tests * removing EnableLedger from Set-AzureSqlDatabase * removing extra parameter * adding whatif text * upgrading Microsoft.Azure.Management.Sql to 1.52 * fixing online version link * adding audit action group for ledger * adding default parameter set * adding default parameter set * adding -EnableLedger description to new-azsqldatabase help file * adding ResourceIdentityWithUserAssignedIdentities to breaking changes ignore list * adding breaking change message * fixing type name * adding using directive * updating breaking changes message * testing updating TestDatabaseDataMaskingPrivilegedUsersChanges test * adding first batch of test re-records * adding more re-recordings * more changes for re-recording * adding re-recordings, adding ignore api version to some tests * adding ignore API version to instance pools test * removing ignore API version from Auditing * adding auditing tests, removing database override from vulnerability assessment test * addding test re-recordings * updating version to newest SDK, removing breaking change warnings * adding server crud test re-records, instance pool crud tests to ignore managed instance changes * adding skip to managed instance tests * Commenting out instance pool, adding vulnerability assessment tests * Adding more test rescordings, skipping tests * reverting unneeded file changes * Update ChangeLog.md * rerecording test * update changelog * ledger changes + changes for new SDK version * adding tests, updating cmdlets * adding tests, fixing cmdlets * adding comments to locations model * adding to changelog.md * adding markdowe help files * fixing merge problems with backup restore resources * updating help file for new-azsqldatabase * rerecording tests * removing EnableLedger from Set-AzureSqlDatabase * removing extra parameter * adding whatif text * upgrading Microsoft.Azure.Management.Sql to 1.52 * fixing online version link * adding audit action group for ledger * adding default parameter set * adding default parameter set * adding -EnableLedger description to new-azsqldatabase help file * adding ResourceIdentityWithUserAssignedIdentities to breaking changes ignore list * adding breaking change message * fixing type name * adding using directive * updating breaking changes message * testing updating TestDatabaseDataMaskingPrivilegedUsersChanges test * adding first batch of test re-records * adding more re-recordings * more changes for re-recording * adding re-recordings, adding ignore api version to some tests * adding ignore API version to instance pools test * removing ignore API version from Auditing * adding auditing tests, removing database override from vulnerability assessment test * addding test re-recordings * updating version to newest SDK, removing breaking change warnings * adding server crud test re-records, instance pool crud tests to ignore managed instance changes * adding skip to managed instance tests * Commenting out instance pool, adding vulnerability assessment tests * Adding more test rescordings, skipping tests * reverting unneeded file changes * rerecording test * Update ChangeLog.md * update changelog * re-recording named secondary test after rebase Co-authored-by: Yunchi Wang <[email protected]>
1 parent c184553 commit 5d86ea7

File tree

266 files changed

+217580
-331210
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

266 files changed

+217580
-331210
lines changed

‎src/Sql/Sql.Test/ScenarioTests/AdvancedDataSecurityManagedInstanceTests.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ public AdvancedDataSecurityManagedInstanceTests(ITestOutputHelper output) : base
3939
};
4040
}
4141

42-
[Fact]
42+
[Fact(Skip = "Cannot re-record.")]
4343
[Trait(Category.AcceptanceType, Category.CheckIn)]
4444
public void AdvancedDataSecurityPolicyTest()
4545
{

‎src/Sql/Sql.Test/ScenarioTests/Common.ps1‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,47 @@ function Create-BasicTestEnvironmentWithParams ($params, $location, $serverVersi
184184
New-AzSqlDatabase -DatabaseName $params.databaseName -ResourceGroupName $params.rgname -ServerName $params.serverName -Edition Basic -Force
185185
}
186186

187+
<#
188+
.SYNOPSIS
189+
Gets the values of the parameters used for ledger digest upload tests
190+
#>
191+
function Get-LedgerTestEnvironmentParameters ($testSuffix)
192+
{
193+
return @{ subscriptionId = (Get-AzContext).Subscription.Id;
194+
rgname = "ledger-cmdlet-test-rg" + $testSuffix;
195+
serverName = "ledger-cmdlet-server" + $testSuffix;
196+
databaseName = "ledger-cmdlet-db" + $testSuffix;
197+
}
198+
}
199+
200+
<#
201+
.SYNOPSIS
202+
Creates the basic test environment used for the ledger tests - creates resource group, server, and database
203+
#>
204+
function Create-LedgerTestEnvironment ($params)
205+
{
206+
$location = "eastus2euap"
207+
$serverVersion = "12.0"
208+
New-AzResourceGroup -Name $params.rgname -Location $location
209+
$serverName = $params.serverName
210+
$serverLogin = "testusername"
211+
<#[SuppressMessage("Microsoft.Security", "CS002:SecretInNextLine", Justification="Test passwords only valid for the duration of the test")]#>
212+
$serverPassword = "t357ingP@s5w0rd!ledger"
213+
$credentials = new-object System.Management.Automation.PSCredential($serverLogin, ($serverPassword | ConvertTo-SecureString -asPlainText -Force))
214+
New-AzSqlServer -ResourceGroupName $params.rgname -ServerName $params.serverName -Location $location -ServerVersion $serverVersion -SqlAdministratorCredentials $credentials
215+
New-AzSqlDatabase -DatabaseName $params.databaseName -ResourceGroupName $params.rgname -ServerName $params.serverName -Edition Basic
216+
}
217+
218+
<#
219+
.SYNOPSIS
220+
Removes the test environment that was needed to perform the ledger digest upload tests
221+
#>
222+
function Remove-LedgerTestEnvironment ($testSuffix)
223+
{
224+
$params = Get-LedgerTestEnvironmentParameters $testSuffix
225+
Remove-AzResourceGroup -Name $params.rgname -Force
226+
}
227+
187228
<#
188229
.SYNOPSIS
189230
Creates the basic test environment needed to perform the Sql data security tests - resource group, managed instance and managed database

‎src/Sql/Sql.Test/ScenarioTests/DatabaseBackupTests.cs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public DatabaseBackupTests(ITestOutputHelper output) : base(output)
2929
};
3030
}
3131

32-
[Fact]
32+
[Fact(Skip = "Not recordable")]
3333
[Trait(Category.AcceptanceType, Category.CheckIn)]
3434
public void TestListDatabaseRestorePoints()
3535
{
@@ -171,7 +171,7 @@ public void TestNewDatabaseRestorePoint()
171171
}
172172
}
173173

174-
[Fact]
174+
[Fact(Skip = "Not recordable")]
175175
[Trait(Category.AcceptanceType, Category.CheckIn)]
176176
public void TestRemoveDatabaseRestorePoint()
177177
{

‎src/Sql/Sql.Test/ScenarioTests/DatabaseBackupTests.ps1‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ function Test-LongTermRetentionV2Policy($location = "southeastasia")
161161
{
162162
# Create with default values
163163
$databaseName = Get-DatabaseName
164-
$db = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName
164+
$db = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName -Force
165165

166166
# Basic Policy Test
167167
Set-AzSqlDatabaseLongTermRetentionPolicy -ResourceGroup $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName -WeeklyRetention $weeklyRetention2
@@ -194,7 +194,7 @@ function Test-LongTermRetentionV2Backup($location = "southeastasia")
194194
{
195195
# Create with default values
196196
$databaseName = Get-DatabaseName
197-
$db = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName
197+
$db = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName -Force
198198

199199
# Basic Get Tests
200200
Get-AzSqlDatabaseLongTermRetentionBackup -Location $db.Location
@@ -221,7 +221,7 @@ function Test-LongTermRetentionV2ResourceGroupBasedBackup($location = "southeast
221221
{
222222
# Create with default values
223223
$databaseName = Get-DatabaseName
224-
$db = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName
224+
$db = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName -Force
225225

226226
# Basic Get Tests
227227
Get-AzSqlDatabaseLongTermRetentionBackup -Location $db.Location -ResourceGroupName $server.ResourceGroupName

‎src/Sql/Sql.Test/ScenarioTests/DatabaseCrudTests.cs‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,5 +190,12 @@ public void TestDatabaseGetWithBackupStorageRedundancy()
190190
{
191191
RunPowerShellTest("Test-GetDatabaseWithBackupStorageRedundancy");
192192
}
193+
194+
[Fact]
195+
[Trait(Category.AcceptanceType, Category.CheckIn)]
196+
public void TestDatabaseCreateWithLedgerEnabled()
197+
{
198+
RunPowerShellTest("Test-DatabaseCreateWithLedgerEnabled");
199+
}
193200
}
194201
}

‎src/Sql/Sql.Test/ScenarioTests/DatabaseCrudTests.ps1‎

Lines changed: 29 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,8 @@ function Test-CreateDatabaseWithZoneRedundancy
312312
function Test-CreateDatabaseWithMaintenanceConfigurationId
313313
{
314314
# Setup
315-
$location = Get-Location "Microsoft.Sql" "operations" "East US 2 EUAP"
315+
# Further actions required if you use Microsoft internal subscription. Please contact feature owners
316+
$location = Get-Location "Microsoft.Sql" "operations" "West Europe"
316317
$rg = Create-ResourceGroupForTest $location
317318

318319
try
@@ -630,7 +631,8 @@ function Test-UpdateDatabaseWithZoneRedundant ()
630631
function Test-UpdateDatabaseWithMaintenanceConfigurationId
631632
{
632633
# Setup
633-
$location = Get-Location "Microsoft.Sql" "operations" "East US 2 EUAP"
634+
# Further actions required if you use Microsoft internal subscription. Please contact feature owners
635+
$location = Get-Location "Microsoft.Sql" "operations" "West Europe"
634636
$rg = Create-ResourceGroupForTest $location
635637

636638
try
@@ -939,7 +941,8 @@ function Test-GetDatabaseWithZoneRedundancy
939941
function Test-GetDatabaseWithMaintenanceConfigurationId
940942
{
941943
# Setup
942-
$location = Get-Location "Microsoft.Sql" "operations" "East US 2 EUAP"
944+
# Further actions required if you use Microsoft internal subscription. Please contact feature owners
945+
$location = Get-Location "Microsoft.Sql" "operations" "West Europe"
943946
$rg = Create-ResourceGroupForTest $location
944947
try
945948
{
@@ -972,6 +975,29 @@ function Test-GetDatabaseWithMaintenanceConfigurationId
972975
}
973976
}
974977

978+
<#
979+
.SYNOPSIS
980+
Tests creating a database with ledger enabled
981+
#>
982+
function Test-DatabaseCreateWithLedgerEnabled ($location = "eastus2euap")
983+
{
984+
# Setup
985+
$rg = Create-ResourceGroupForTest
986+
$server = Create-ServerForTest $rg $location
987+
988+
# Create with ledger enabled
989+
$databaseName = Get-DatabaseName
990+
$db1 = New-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName -EnableLedger -Force
991+
Assert-AreEqual $db1.DatabaseName $databaseName
992+
Assert-AreEqual "True" $db1.EnableLedger
993+
994+
# Validate Get-AzSqlDatabase returns ledger property
995+
$databaseFromGet = Get-AzSqlDatabase -ResourceGroupName $rg.ResourceGroupName -ServerName $server.ServerName -DatabaseName $databaseName
996+
Assert-AreEqual "True" $databaseFromGet.EnableLedger
997+
998+
Remove-ResourceGroupForTest $rg
999+
}
1000+
9751001
<#
9761002
.SYNOPSIS
9771003
Tests Deleting a database

‎src/Sql/Sql.Test/ScenarioTests/ElasticPoolCrudTests.ps1‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,8 @@ function Test-CreateElasticPoolWithZoneRedundancy
178178
function Test-CreateElasticPoolWithMaintenanceConfigurationId
179179
{
180180
# Setup
181-
$location = Get-Location "Microsoft.Sql" "operations" "East US 2 EUAP"
181+
# Further actions required if you use Microsoft internal subscription. Please contact feature owners
182+
$location = Get-Location "Microsoft.Sql" "operations" "West Europe"
182183
$rg = Create-ResourceGroupForTest $location
183184

184185
try
@@ -411,7 +412,8 @@ function Test-UpdateElasticPoolWithZoneRedundancy
411412
function Test-UpdateElasticPoolWithMaintenanceConfigurationId
412413
{
413414
# Setup
414-
$location = Get-Location "Microsoft.Sql" "operations" "East US 2 EUAP"
415+
# Further actions required if you use Microsoft internal subscription. Please contact feature owners
416+
$location = Get-Location "Microsoft.Sql" "operations" "West Europe"
415417
$rg = Create-ResourceGroupForTest $location
416418

417419
try
@@ -542,7 +544,8 @@ function Test-GetElasticPoolWithZoneRedundancy
542544
function Test-GetElasticPoolWithMaintenanceConfigurationId
543545
{
544546
# Setup
545-
$location = Get-Location "Microsoft.Sql" "operations" "East US 2 EUAP"
547+
# Further actions required if you use Microsoft internal subscription. Please contact feature owners
548+
$location = Get-Location "Microsoft.Sql" "operations" "West Europe"
546549
$rg = Create-ResourceGroupForTest $location
547550
try
548551
{

‎src/Sql/Sql.Test/ScenarioTests/ImportExportTests.cs‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,29 +28,29 @@ public ImportExportTests(ITestOutputHelper output) : base(output)
2828
};
2929
}
3030

31-
[Fact]
31+
[Fact(Skip = "Cannot re-record.")]
3232
[Trait(Category.AcceptanceType, Category.CheckIn)]
3333
public void TestExportDatabase()
3434
{
3535
RunPowerShellTest("Test-ExportDatabase");
3636
}
3737

38-
[Fact]
38+
[Fact(Skip = "Cannot re-record.")]
3939
[Trait(Category.AcceptanceType, Category.CheckIn)]
4040
public void TestExportDatabase_NetworkIsolation()
4141
{
4242
RunPowerShellTest("Test-ExportDatabaseNetworkIsolation");
4343
}
4444

45-
[Fact]
45+
[Fact(Skip = "Cannot re-record.")]
4646
[Trait(Category.AcceptanceType, Category.CheckIn)]
4747
[Trait(Category.Sql, "Needs to be re-recorded")]
4848
public void TestImportNewDatabase()
4949
{
5050
RunPowerShellTest("Test-ImportNewDatabase");
5151
}
5252

53-
[Fact]
53+
[Fact(Skip = "Cannot re-record.")]
5454
[Trait(Category.AcceptanceType, Category.CheckIn)]
5555
[Trait(Category.Sql, "Needs to be re-recorded")]
5656
public void TestImportNewDatabase_NetworkIsolation()

‎src/Sql/Sql.Test/ScenarioTests/InstanceFailoverGroupTests.cs‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -25,84 +25,84 @@ public InstanceFailoverGroupTests(ITestOutputHelper output) : base(output)
2525
{
2626
}
2727

28-
[Fact]
28+
[Fact(Skip = "Has manual setup, unable to re-record")]
2929
[Trait(Category.AcceptanceType, Category.CheckIn)]
3030
public void TestCreateInstanceFailoverGroup_Named()
3131
{
3232
RunPowerShellTest("Test-CreateInstanceFailoverGroup-Named");
3333
}
3434

35-
[Fact]
35+
[Fact(Skip = "Has manual setup, unable to re-record")]
3636
[Trait(Category.AcceptanceType, Category.CheckIn)]
3737
public void TestCreateInstanceFailoverGroup_Positional()
3838
{
3939
RunPowerShellTest("Test-CreateInstanceFailoverGroup-Positional");
4040
}
4141

42-
[Fact]
42+
[Fact(Skip = "Has manual setup, unable to re-record")]
4343
[Trait(Category.AcceptanceType, Category.CheckIn)]
4444
public void TestCreateInstanceFailoverGroup_AutomaticPolicy()
4545
{
4646
RunPowerShellTest("Test-CreateInstanceFailoverGroup-AutomaticPolicy");
4747
}
4848

49-
[Fact]
49+
[Fact(Skip = "Has manual setup, unable to re-record")]
5050
[Trait(Category.AcceptanceType, Category.CheckIn)]
5151
public void TestCreateInstanceFailoverGroup_AutomaticPolicyGracePeriodReadOnlyFailover()
5252
{
5353
RunPowerShellTest("Test-CreateInstanceFailoverGroup-AutomaticPolicyGracePeriodReadOnlyFailover");
5454
}
5555

56-
[Fact]
56+
[Fact(Skip = "Has manual setup, unable to re-record")]
5757
[Trait(Category.AcceptanceType, Category.CheckIn)]
5858
public void TestCreateInstanceFailoverGroup_ManualPolicy()
5959
{
6060
RunPowerShellTest("Test-CreateInstanceFailoverGroup-ManualPolicy");
6161
}
6262

63-
[Fact]
63+
[Fact(Skip = "Has manual setup, unable to re-record")]
6464
[Trait(Category.AcceptanceType, Category.CheckIn)]
6565
public void TestSetInstanceFailoverGroup_Named()
6666
{
6767
RunPowerShellTest("Test-SetInstanceFailoverGroup-Named");
6868
}
6969

70-
[Fact]
70+
[Fact(Skip = "Has manual setup, unable to re-record")]
7171
[Trait(Category.AcceptanceType, Category.CheckIn)]
7272
public void TestSetInstanceFailoverGroup_Positional()
7373
{
7474
RunPowerShellTest("Test-SetInstanceFailoverGroup-Positional");
7575
}
7676

77-
[Fact]
77+
[Fact(Skip = "Has manual setup, unable to re-record")]
7878
[Trait(Category.AcceptanceType, Category.CheckIn)]
7979
public void TestSetInstanceFailoverGroup_AutomaticWithGracePeriodReadOnlyFailover()
8080
{
8181
RunPowerShellTest("Test-SetInstanceFailoverGroup-AutomaticWithGracePeriodReadOnlyFailover");
8282
}
8383

84-
[Fact]
84+
[Fact(Skip = "Has manual setup, unable to re-record")]
8585
[Trait(Category.AcceptanceType, Category.CheckIn)]
8686
public void TestSetInstanceFailoverGroup_AutomaticToManual()
8787
{
8888
RunPowerShellTest("Test-SetInstanceFailoverGroup-AutomaticToManual");
8989
}
9090

91-
[Fact]
91+
[Fact(Skip = "Has manual setup, unable to re-record")]
9292
[Trait(Category.AcceptanceType, Category.CheckIn)]
9393
public void TestSetInstanceFailoverGroup_ManualToAutomaticNoGracePeriod()
9494
{
9595
RunPowerShellTest("Test-SetInstanceFailoverGroup-ManualToAutomaticNoGracePeriod");
9696
}
9797

98-
[Fact]
98+
[Fact(Skip = "Has manual setup, unable to re-record")]
9999
[Trait(Category.AcceptanceType, Category.CheckIn)]
100100
public void Test_SwitchInstanceFailoverGroup()
101101
{
102102
RunPowerShellTest("Test-SwitchInstanceFailoverGroup");
103103
}
104104

105-
[Fact]
105+
[Fact(Skip = "Has manual setup, unable to re-record")]
106106
[Trait(Category.AcceptanceType, Category.CheckIn)]
107107
public void Test_SwitchInstanceFailoverGroupAllowDataLoss()
108108
{

0 commit comments

Comments
 (0)