[Communication] - PhoneNumbers - Require phone number pool in livetes…

…ts (Azure#27093) * Require phone number pool in livetests * Fix conditional statement with the correct value * Fix formatting
Juancpani · Feb 18, 2022 · 8545cde · 8545cde
1 parent 41b11af
commit 8545cde
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 19 deletions.
diff --git a/sdk/communication/Azure.Communication.Common/tests/CommunicationTestEnvironment.cs b/sdk/communication/Azure.Communication.Common/tests/CommunicationTestEnvironment.cs
@@ -31,7 +31,7 @@ public class CommunicationTestEnvironment : TestEnvironment
 
         public string LiveTestStaticAccessKey => Core.ConnectionString.Parse(LiveTestStaticConnectionString).GetRequired("accesskey");
 
-        public string CommunicationTestPhoneNumber => GetPhoneNumberByTestAgent() ?? GetRecordedVariable(AzurePhoneNumber, options => options.IsSecret("+14255550123"));
+        public string DefaultTestPhoneNumber => GetRecordedVariable(AzurePhoneNumber, options => options.IsSecret("+14255550123"));
 
         public string SkipSmsTest => GetOptionalVariable(SkipIntSmsTestEnvironmentVariableName) ?? "False";
 
@@ -41,14 +41,8 @@ public class CommunicationTestEnvironment : TestEnvironment
 
         public bool ShouldIgnorePhoneNumbersTests => bool.Parse(SkipPhoneNumbersTest);
 
-        public string AzureTestAgent => GetOptionalVariable(AzureTestAgentVariableName);
+        public string TestAgentPhoneNumber => GetRecordedVariable($"{AzurePhoneNumber}_{AzureTestAgent}", options => options.IsSecret("+14255550123"));
 
-        private string? GetPhoneNumberByTestAgent()
-        {
-            if (AzureTestAgent == null)
-                return null;
-
-            return GetRecordedOptionalVariable($"{AzurePhoneNumber}_{AzureTestAgent}", options => options.IsSecret("+14255550123"));
-        }
+        private string AzureTestAgent => GetVariable(AzureTestAgentVariableName);
     }
 }
diff --git a/sdk/communication/Azure.Communication.PhoneNumbers/phone-numbers-livetest-matrix.json b/sdk/communication/Azure.Communication.PhoneNumbers/phone-numbers-livetest-matrix.json
@@ -36,24 +36,21 @@
                 "OSVmImage": "macOS-10.15",
                 "Pool": "Azure Pipelines",
                 "TestTargetFramework": "netcoreapp3.1",
-                "AZURE_PHONE_NUMBER_ID": "2",
-                "SKIP_UPDATE_CAPABILITIES_LIVE_TESTS": "FALSE"
+                "SKIP_UPDATE_CAPABILITIES_LIVE_TESTS": "TRUE"
             },
             "macOS-10.15_NET6.0": {
                 "OSVmImage": "macOS-10.15",
                 "Pool": "Azure Pipelines",
                 "TestTargetFramework": "net6.0",
-                "AZURE_TEST_AGENT": "macos_1015_net60"
+                "AZURE_TEST_AGENT": "macos_1015_net60",
+                "SKIP_UPDATE_CAPABILITIES_LIVE_TESTS": "FALSE"
             }
         },
         "AdditionalTestArguments": [
             "/p:UseProjectReferenceToAzureClients=false",
             "/p:UseProjectReferenceToAzureClients=true"
         ],
-        "BuildConfiguration": [
-            "Debug",
-            "Release"
-        ]
+        "BuildConfiguration": ["Debug", "Release"]
     },
     "include": [
         {

diff --git a/...n/Azure.Communication.PhoneNumbers/tests/Infrastructure/PhoneNumbersClientLiveTestBase.cs b/...n/Azure.Communication.PhoneNumbers/tests/Infrastructure/PhoneNumbersClientLiveTestBase.cs
@@ -86,9 +86,13 @@ protected PhoneNumbersClient CreateClient(AuthMethod authMethod = AuthMethod.Con
 
         protected string GetTestPhoneNumber()
         {
-            return TestEnvironment.Mode == RecordedTestMode.Playback
-                ? RecordedTestSanitizer.SanitizeValue
-                : TestEnvironment.CommunicationTestPhoneNumber;
+            if (TestEnvironment.Mode == RecordedTestMode.Playback)
+                return RecordedTestSanitizer.SanitizeValue;
+
+            if (!SkipUpdateCapabilitiesLiveTest)
+                return TestEnvironment.TestAgentPhoneNumber;
+
+            return TestEnvironment.DefaultTestPhoneNumber;
         }
 
         protected void SleepIfNotInPlaybackMode()