refactor: Add reason for skipped tests

aws · Nov 21, 2024 · 042cf27 · 042cf27
1 parent 5aee938
commit 042cf27
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/sdk/test/IntegrationTests/Tests/General.cs b/sdk/test/IntegrationTests/Tests/General.cs
@@ -680,7 +680,7 @@ public void TestManualClockCorrection()
         // By default it only tests a small subset of services.
         [TestMethod]
         [TestCategory("General")]
-        [Ignore]
+        [Ignore("Skipping flaky test while design for clock skew behavior is defined")]
         public void TestClockSkewCorrection()
         {
             VerifyClockSkewSetting();

diff --git a/sdk/test/NetStandard/IntegrationTests/IntegrationTests/General.cs b/sdk/test/NetStandard/IntegrationTests/IntegrationTests/General.cs
@@ -132,10 +132,10 @@ protected override CredentialsRefreshState GenerateNewCredentials()
             }
         }
 
-        // [Fact]
-        // This is commented out because netstandard uses xunit which runs tests in parallel.
+        // This is ignored because netstandard uses xunit which runs tests in parallel.
         // When this test is run in parallel with other tests it causes them to fail b/c it changes
         // the clockskew while those are running.
+        [Fact(Skip = "Skipping flaky test while design for clock skew behavior is defined")]
         public void TestManualClockCorrection()
         {
             TestClients(TestServiceCallForManualClockCorrection);
@@ -144,10 +144,10 @@ public void TestManualClockCorrection()
         // This test verifies that all service clients are able to
         // correctly handle clock skew errors.
         // By default it only tests a small subset of services.
-        // This is commented out because netstandard uses xunit which runs tests in parallel.
+        // This is ignored because netstandard uses xunit which runs tests in parallel.
         // When this test is run in parallel with other tests it causes them to fail b/c it changes
         // the clockskew while those are running.
-        // [Fact]
+        [Fact(Skip = "Skipping flaky test while design for clock skew behavior is defined")]
         public void TestClockSkewCorrection()
         {
             TestClients(TestServiceCallForClockSkew);