Merged Pull Request '#57 postindustria-tech:sleep-for-logging-to-happ…

…en->main : 'Failed DataUpdateService_UpdateFromUrl_HttpException'' Failed DataUpdateService_UpdateFromUrl_HttpException
51Degrees · Sep 23, 2023 · fe0242f · fe0242f
2 parents 4a9cbf4 + 4fd2d18
commit fe0242f
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/Tests/FiftyOne.Pipeline.Engines.Tests/Services/DataUpdateServiceTests.cs b/Tests/FiftyOne.Pipeline.Engines.Tests/Services/DataUpdateServiceTests.cs
@@ -53,6 +53,7 @@ public class DataUpdateServiceTests
         private int _ignoreWranings = 0;
         private int _ignoreErrors = 0;
         private const int TEST_TIMEOUT_MS = 3000;
+        private const int LOGGING_TIMEOUT_MS = 75;
 
         private DataUpdateService _dataUpdate;
 
@@ -101,8 +102,18 @@ public void Init()
         [TestCleanup]
         public void Cleanup()
         {
-            _logger.AssertMaxErrors(_ignoreErrors);
-            _logger.AssertMaxWarnings(_ignoreWranings);
+            try
+            {
+                _logger.AssertMaxErrors(_ignoreErrors);
+                _logger.AssertMaxWarnings(_ignoreWranings);
+            }
+            finally
+            {
+                foreach (var entry in _logger.Entries)
+                {
+                    Console.WriteLine($"[LOGGER LOGS] {entry.Key} > {entry.Value}");
+                }
+            }
         }
 
         /// <summary>
@@ -890,6 +901,7 @@ public void DataUpdateService_UpdateFromUrl_HttpException()
                 // Assert
                 Assert.IsTrue(completeFlag.IsSet, "The 'CheckForUpdateComplete' " +
                     "event was never fired");
+                Thread.Sleep(LOGGING_TIMEOUT_MS);
                 _httpHandler.Verify(h => h.Send(It.IsAny<HttpRequestMessage>()), Times.Once());
                 // Make sure engine was not refreshed
                 engine.Verify(e => e.RefreshData(config.Identifier), Times.Never());