Fix TestsManifestGeneration tests (#46658)

* Flush before checking for manifest data * add comments
dotnet · Jan 7, 2021 · 1d0f67f · 1d0f67f
1 parent 604660b
commit 1d0f67f
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/...ries/System.Diagnostics.Tracing/tests/BasicEventSourceTest/TestsManifestGeneration.Etw.cs b/...ries/System.Diagnostics.Tracing/tests/BasicEventSourceTest/TestsManifestGeneration.Etw.cs
@@ -54,6 +54,11 @@ public void Test_EventSource_EtwManifestGeneration()
                     Thread.Sleep(TimeSpan.FromSeconds(5));
 
                     tracesession.Flush();
+
+                    // Sleep after requesting flush to ensure that the manifest payload generated
+                    // is fully written to the etl file.
+                    Thread.Sleep(TimeSpan.FromSeconds(5));
+
                     tracesession.DisableProvider("SimpleEventSource");
                     tracesession.Dispose();
 
@@ -87,6 +92,10 @@ public void Test_EventSource_EtwManifestGenerationRollover()
 
                     tracesession.Flush();
 
+                    // Sleep after requesting flush to ensure that the manifest payload generated
+                    // is fully written to the etl file.
+                    Thread.Sleep(TimeSpan.FromSeconds(5));
+
                     tracesession.SetFileName(rolloverFile);
 
                     Thread.Sleep(TimeSpan.FromSeconds(5));