[Synapse] - Fix test failure (#12625)

* Fix test failure * Correct test name * Remove Ignore annotation and use random guid
Azure · Jun 9, 2020 · 48f2e62 · 48f2e62
1 parent 423fa77
commit 48f2e62
Show file tree

Hide file tree

Showing 4 changed files with 46 additions and 47 deletions.
diff --git a/sdk/synapse/Azure.Analytics.Synapse.AccessControl/tests/AccessControlClientLiveTests.cs b/sdk/synapse/Azure.Analytics.Synapse.AccessControl/tests/AccessControlClientLiveTests.cs
@@ -58,11 +58,10 @@ public async Task TestGetRoleAssignment()
         }
 
         [Test]
-        [Ignore("https://github.com/Azure/azure-sdk-for-net/issues/12599")]
         public async Task TestCreateAndDeleteRoleAssignment()
         {
             string sqlAdminRoleId = "7af0c69a-a548-47d6-aea3-d00e69bd83aa";
-            string principalId = Guid.NewGuid().ToString();
+            string principalId = Recording.Random.NewGuid().ToString();
 
             // Create role assignment.
             RoleAssignmentDetails actualRoleAssignment = await AccessControlClient.CreateRoleAssignmentAsync(new RoleAssignmentOptions(roleId:sqlAdminRoleId, principalId: principalId));

diff --git a/.../tests/SessionRecords/AccessControlClientLiveTests/TestCreateAndDeleteRoleAssignment.json b/.../tests/SessionRecords/AccessControlClientLiveTests/TestCreateAndDeleteRoleAssignment.json
diff --git a/...s/SessionRecords/AccessControlClientLiveTests/TestCreateAndDeleteRoleAssignmentAsync.json b/...s/SessionRecords/AccessControlClientLiveTests/TestCreateAndDeleteRoleAssignmentAsync.json
diff --git a/...Azure.Analytics.Synapse.AccessControl/tests/samples/Sample1_AddAndRemoveRoleAssignment.cs b/...Azure.Analytics.Synapse.AccessControl/tests/samples/Sample1_AddAndRemoveRoleAssignment.cs
@@ -18,7 +18,7 @@ namespace Azure.Analytics.Synapse.Samples
     public partial class AddAndRemoveRoleAssignment
     {
         [Test]
-        public void SubmitSparkJobSync()
+        public void AddAndRemoveRoleAssignmentSync()
         {
             // Environment variable with the Synapse workspace endpoint.
             string workspaceUrl = TestEnvironment.WorkspaceUrl;