Add test for the even handler leaking fix (PowerShell#10768)

daxian-dbw · kilasuit · commit b2e6d60d49f0 · 2019-11-08T23:19:49.000-05:00
Add a test for the even handler leaking fix addressed in PowerShell#10626 PowerShell#10626 The test is added to verify that PowerShell unregister the 'AppDomain.ProcessExit' event handler when a Runspace closes.
diff --git a/test/xUnit/csharp/test_Runspace.cs b/test/xUnit/csharp/test_Runspace.cs
@@ -105,9 +105,6 @@ public void TestRunspaceWithPowerShellAndInitialSessionState()
         [SkippableFact]
         public void TestAppDomainProcessExitEvenHandlerNotLeaking()
         {
-            // Skip this flaky test for now.
-            Skip.IfNot(false);
-
             Skip.IfNot(Platform.IsWindows);
 
             EventHandler eventHandler;

Original file line number	Diff line number	Diff line change
`@@ -105,9 +105,6 @@ public void TestRunspaceWithPowerShellAndInitialSessionState()`
`105`	`105`	`[SkippableFact]`
`106`	`106`	`public void TestAppDomainProcessExitEvenHandlerNotLeaking()`
`107`	`107`	`{`
`108`		`- // Skip this flaky test for now.`
`109`		`- Skip.IfNot(false);`
`110`		`-`
`111`	`108`	`Skip.IfNot(Platform.IsWindows);`
`112`	`109`
`113`	`110`	`EventHandler eventHandler;`