Skip update capabilities live test (Azure#26759)

danielortega-msft · web-flow · commit b4baa9d87021 · 2022-02-08T11:09:07.000-08:00
This prevents the update capabilities test from running in the live test pipeline, which caused an issue with parallel test execution updating the same phone number.
diff --git a/sdk/communication/Azure.Communication.PhoneNumbers/tests/PhoneNumbersClient/PhoneNumbersClientLiveTests.cs b/sdk/communication/Azure.Communication.PhoneNumbers/tests/PhoneNumbersClient/PhoneNumbersClientLiveTests.cs
@@ -178,7 +178,7 @@ public void CreateSearch()
         [AsyncOnly]
         public async Task UpdateCapabilitiesAsync()
         {
-            if (TestEnvironment.ShouldIgnorePhoneNumbersTests) {
+            if (TestEnvironment.ShouldIgnorePhoneNumbersTests || SkipPhoneNumberLiveTests) {
                 Assert.Ignore("Skip phone number live tests flag is on.");
             }
             var number = GetTestPhoneNumber();
@@ -278,7 +278,7 @@ public async Task GetPurchasedPhoneNumbersNextPage()
         [SyncOnly]
         public void UpdateCapabilities()
         {
-            if (TestEnvironment.ShouldIgnorePhoneNumbersTests) {
+            if (TestEnvironment.ShouldIgnorePhoneNumbersTests || SkipPhoneNumberLiveTests) {
                 Assert.Ignore("Skip phone number live tests flag is on.");
             }
             var number = GetTestPhoneNumber();

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ public void CreateSearch()`
`178`	`178`	`[AsyncOnly]`
`179`	`179`	`public async Task UpdateCapabilitiesAsync()`
`180`	`180`	`{`
`181`		`- if (TestEnvironment.ShouldIgnorePhoneNumbersTests) {`
	`181`	`+ if (TestEnvironment.ShouldIgnorePhoneNumbersTests \|\| SkipPhoneNumberLiveTests) {`
`182`	`182`	`Assert.Ignore("Skip phone number live tests flag is on.");`
`183`	`183`	`}`
`184`	`184`	`var number = GetTestPhoneNumber();`
`@@ -278,7 +278,7 @@ public async Task GetPurchasedPhoneNumbersNextPage()`
`278`	`278`	`[SyncOnly]`
`279`	`279`	`public void UpdateCapabilities()`
`280`	`280`	`{`
`281`		`- if (TestEnvironment.ShouldIgnorePhoneNumbersTests) {`
	`281`	`+ if (TestEnvironment.ShouldIgnorePhoneNumbersTests \|\| SkipPhoneNumberLiveTests) {`
`282`	`282`	`Assert.Ignore("Skip phone number live tests flag is on.");`
`283`	`283`	`}`
`284`	`284`	`var number = GetTestPhoneNumber();`