SNOW-723810: Add coverage for RestRequest

snowflakedb · Oct 6, 2023 · 3de0c09 · 3de0c09
1 parent ad92e98
commit 3de0c09
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/Snowflake.Data.Tests/UnitTests/SFOktaTest.cs b/Snowflake.Data.Tests/UnitTests/SFOktaTest.cs
@@ -73,11 +73,14 @@ public void TestWrongPostbackUrl()
         [Test]
         public void TestLoginRequestToString()
         {
+            // Arrange
             string expectedOktaAccount = "mockOktaAccount";
             string expectedOktaUser = "mockOktaUser";
             string expectedOktaUrl = "mockOktaUrl";
 
             LoginRequestClientEnv loginRequestClientEnv = new LoginRequestClientEnv();
+
+            // Act
             LoginRequest loginRequest = new LoginRequest()
             {
                 data = new LoginRequestData()
@@ -90,6 +93,7 @@ public void TestLoginRequestToString()
                 }
             };
 
+            // Assert
             Assert.AreEqual($"LoginRequest {{data: LoginRequestData {{ClientAppVersion: {SFEnvironment.DriverVersion},\n " +
                 $"AccountName: {expectedOktaAccount},\n " +
                 $"loginName: {expectedOktaUser},\n " +
@@ -106,7 +110,10 @@ public void TestLoginRequestToString()
         [Test]
         public void TestAuthenticatorRequestToString()
         {
+            // Arrange
             string expectedOktaAccount = "mockOktaAccount";
+
+            // Act
             AuthenticatorRequest authenticatorRequest = new AuthenticatorRequest()
             {
                 Data = new AuthenticatorRequestData()
@@ -115,6 +122,7 @@ public void TestAuthenticatorRequestToString()
                 }
             };
 
+            // Assert
             Assert.AreEqual($"AuthenticatorRequest {{data: AuthenticatorRequestData {{ACCOUNT_NAME: {expectedOktaAccount} }} }}",
                 authenticatorRequest.ToString());
         }