Don't call afterEach within beforeEach (microsoft#46963)

Otherwise, a new afterEach handler is added for each test case and the
number of handlers run grows quadratically.

Author: amcasey

src/testRunner/unittests/createMapShim.ts

@@ -130,9 +130,9 @@ namespace ts {
             }
 
             MapShim = ShimCollections.createMapShim(getIterator);
-            afterEach(() => {
-                MapShim = undefined!;
-            });
+        });
+        afterEach(() => {
+            MapShim = undefined!;
         });
 
         it("iterates values in insertion order and handles changes with string keys", () => {

src/testRunner/unittests/createSetShim.ts

@@ -128,9 +128,9 @@ namespace ts {
             }
 
             SetShim = ShimCollections.createSetShim(getIterator);
-            afterEach(() => {
-                SetShim = undefined!;
-            });
+        });
+        afterEach(() => {
+            SetShim = undefined!;
         });
 
         it("iterates values in insertion order and handles changes with string keys", () => {

src/testRunner/unittests/debugDeprecation.ts

@@ -1,10 +1,12 @@
 namespace ts {
     describe("unittests:: debugDeprecation", () => {
+        let loggingHost: LoggingHost | undefined;
         beforeEach(() => {
-            const loggingHost = Debug.loggingHost;
-            afterEach(() => {
-                Debug.loggingHost = loggingHost;
-            });
+            loggingHost = Debug.loggingHost;
+        });
+        afterEach(() => {
+            Debug.loggingHost = loggingHost;
+            loggingHost = undefined;
         });
         describe("deprecateFunction", () => {
             it("silent deprecation", () => {