Merge pull request Azure#3295 from yaakoviyun/dev

Fixing null pointer exception whn Getting Auditing rpolicy from UK resources

Author: cormacpayne

src/ResourceManager/Sql/Commands.Sql.Test/Commands.Sql.Test.csproj

@@ -485,6 +485,9 @@
     </None>
     <None Include="SessionRecords\Microsoft.Azure.Commands.Sql.Test.ScenarioTests.AuditingTests\TestDatatabaseAuditingTypeMigration.json">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
+	<None Include="SessionRecords\Microsoft.Azure.Commands.Sql.Test.ScenarioTests.AuditingTests\TestGetServerAndDatabaseAuditingInUkRegion.json">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
     <None Include="SessionRecords\Microsoft.Azure.Commands.Sql.Test.ScenarioTests.AuditingTests\TestServerAuditingTypeMigration.json">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>

src/ResourceManager/Sql/Commands.Sql.Test/ScenarioTests/AuditingTests.cs

@@ -242,11 +242,18 @@ public void TestServerAuditingTypeMigration()
             RunPowerShellTest("Test-ServerAuditingTypeMigration");
         }
 
-        [Fact (Skip = "Waiting backend validation")]
+        [Fact(Skip = "Waiting backend validation")]
         [Trait(Category.AcceptanceType, Category.CheckIn)]
         public void TestAuditingDatabaseUpdatePolicyWithSameNameStorageOnDifferentRegion()
         {
             RunPowerShellTest("Test-AuditingDatabaseUpdatePolicyWithSameNameStorageOnDifferentRegion");
         }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void TestGetServerAndDatabaseAuditingInUkRegion()
+        {
+            RunPowerShellTest("Test-GetServerAndDatabaseAuditingInUkRegion");
+        }
     }
 }

src/ResourceManager/Sql/Commands.Sql.Test/ScenarioTests/AuditingTests.ps1

@@ -1098,3 +1098,37 @@ function Test-ServerAuditingTypeMigration
 		Remove-TestEnvironment $testSuffix
 	}
 }
+
+<#
+.SYNOPSIS
+Tests that Get Server and Database Auditing on UK passes
+#>
+function Test-GetServerAndDatabaseAuditingInUkRegion
+{
+	# Setup
+	$testSuffix = 684412
+	Create-TestEnvironment $testSuffix "UK South"
+	$params = Get-SqlAuditingTestEnvironmentParameters $testSuffix 
+
+	try
+	{                              
+		# Test
+		$policy = Get-AzureRmSqlServerAuditingPolicy -ResourceGroupName $params.rgname -ServerName $params.serverName
+                                
+		# Assert
+		Assert-AreEqual $policy.AuditState "Disabled"
+		Assert-AreEqual $policy.RetentionInDays 0
+                                
+		# Test
+		$policy = Get-AzureRmSqlDatabaseAuditingPolicy -ResourceGroupName $params.rgname -ServerName $params.serverName -DatabaseName $params.databaseName
+                                
+		# Assert
+		Assert-AreEqual $policy.AuditState "Disabled"
+		Assert-AreEqual $policy.RetentionInDays 0
+    }
+    finally
+    {
+		# Cleanup
+		Remove-TestEnvironment $testSuffix
+    }
+}