cleanup

taiga-family · Oct 14, 2020 · 126252d · 126252d
1 parent 7d50925
commit 126252d
Show file tree

Hide file tree

Showing 10 changed files with 40 additions and 9 deletions.
diff --git a/projects/cdk/services/test/resize.service.spec.ts b/projects/cdk/services/test/resize.service.spec.ts
@@ -26,6 +26,10 @@ describe('TuiResizeService', () => {
         });
     });
 
+    afterEach(() => {
+        document.body.removeChild(element);
+    });
+
     it('emits default first value', done => {
         const resize$ = TestBed.get(TuiResizeService);
 

diff --git a/projects/cdk/utils/dom/test/get-closest-element.spec.ts b/projects/cdk/utils/dom/test/get-closest-element.spec.ts
@@ -9,6 +9,9 @@ describe('getClosestElement', () => {
         document.body.appendChild(div);
 
         expect(getClosestElement(element, 'div')).toEqual(div);
+
+        document.body.removeChild(element);
+        document.body.removeChild(div);
     });
 
     it('fallback works without closest and matches element', () => {
@@ -25,6 +28,8 @@ describe('getClosestElement', () => {
         expect(getClosestElement(element, 'div')).toEqual(div);
 
         Element.prototype.closest = savedCLosest;
+        document.body.removeChild(element);
+        document.body.removeChild(div);
     });
 
     it('fallback works without closest and with nested element', () => {
@@ -39,5 +44,6 @@ describe('getClosestElement', () => {
         expect(getClosestElement(div, 'img')).toEqual(null);
 
         Element.prototype.closest = savedCLosest;
+        document.body.removeChild(div);
     });
 });
diff --git a/projects/cdk/utils/dom/test/get-document-or-shadow-root.spec.ts b/projects/cdk/utils/dom/test/get-document-or-shadow-root.spec.ts
@@ -17,5 +17,7 @@ describe('getDocumentOrShadowRoot', () => {
         expect(getDocumentOrShadowRoot(element)).toEqual(
             element.getRootNode() as Document,
         );
+
+        document.body.removeChild(element);
     });
 });
diff --git a/projects/cdk/utils/dom/test/is-node-in.spec.ts b/projects/cdk/utils/dom/test/is-node-in.spec.ts
@@ -7,6 +7,8 @@ describe('isNodeIn', () => {
         document.body.appendChild(div);
 
         expect(isNodeIn(div, 'div')).toEqual(true);
+
+        document.body.removeChild(div);
     });
 
     it('class selector', () => {
@@ -17,6 +19,8 @@ describe('isNodeIn', () => {
         document.body.appendChild(div);
 
         expect(isNodeIn(div, '.hello')).toEqual(true);
+
+        document.body.removeChild(div);
     });
 
     it('works with text nodes', () => {
@@ -28,5 +32,7 @@ describe('isNodeIn', () => {
         document.body.appendChild(p);
 
         expect(isNodeIn(textNode, 'p')).toEqual(true);
+
+        document.body.removeChild(p);
     });
 });
diff --git a/projects/cdk/utils/focus/tests/blur-native-focused.spec.ts b/projects/cdk/utils/focus/tests/blur-native-focused.spec.ts
@@ -5,15 +5,15 @@ describe('blurNativeFocused', () => {
     let element: HTMLInputElement;
 
     beforeEach(() => {
-        if (element) {
-            document.body.removeChild(element);
-        }
-
         element = document.createElement('input');
         document.body.appendChild(element);
         setNativeFocused(element, true);
     });
 
+    afterAll(() => {
+        document.body.removeChild(element);
+    });
+
     it('Element is focused', () => {
         expect(document.activeElement).toBe(element);
     });

diff --git a/projects/cdk/utils/focus/tests/get-closest-keyboard-focusable.spec.ts b/projects/cdk/utils/focus/tests/get-closest-keyboard-focusable.spec.ts
@@ -18,6 +18,8 @@ describe('getClosestKeyboardFocusable', () => {
         document.body.appendChild(root);
 
         expect(getClosestKeyboardFocusable(divElement, true, root)).toBe(buttonElement);
+
+        document.body.removeChild(root);
     });
 
     it('returns closest focusable if there is towards', () => {
@@ -30,6 +32,8 @@ describe('getClosestKeyboardFocusable', () => {
         document.body.appendChild(root);
 
         expect(getClosestKeyboardFocusable(divElement, false, root)).toBe(buttonElement);
+
+        document.body.removeChild(root);
     });
 
     it('returns null if there is no focusable elements', () => {
@@ -40,5 +44,7 @@ describe('getClosestKeyboardFocusable', () => {
         document.body.appendChild(root);
 
         expect(getClosestKeyboardFocusable(divElement, undefined, root)).toBe(null);
+
+        document.body.removeChild(root);
     });
 });
diff --git a/projects/cdk/utils/focus/tests/get-native-focused.spec.ts b/projects/cdk/utils/focus/tests/get-native-focused.spec.ts
@@ -8,6 +8,8 @@ describe('getNativeFocused', () => {
         buttonElement.focus();
 
         expect(getNativeFocused(document)).toBe(buttonElement);
+
+        document.body.removeChild(buttonElement);
     });
 
     it('returns element from shadowRoot', () => {
@@ -22,5 +24,7 @@ describe('getNativeFocused', () => {
         buttonElement.focus();
 
         expect(getNativeFocused(document)).toBe(buttonElement);
+
+        document.body.removeChild(div);
     });
 });
diff --git a/projects/cdk/utils/focus/tests/is-native-focused-in.spec.ts b/projects/cdk/utils/focus/tests/is-native-focused-in.spec.ts
@@ -2,7 +2,7 @@ import {isNativeFocusedIn} from '../is-native-focused-in';
 
 describe('isNativeFocusedIn', () => {
     it('element is not focused in', () => {
-        const element = document.createElement('buton');
+        const element = document.createElement('button');
 
         expect(isNativeFocusedIn(element)).toEqual(false);
     });

diff --git a/projects/cdk/utils/focus/tests/move-focus.spec.ts b/projects/cdk/utils/focus/tests/move-focus.spec.ts
@@ -13,5 +13,8 @@ describe('move focus', () => {
         moveFocus(0, arr, 1);
 
         expect(document.activeElement).toEqual(second);
+
+        document.body.removeChild(first);
+        document.body.removeChild(second);
     });
 });
diff --git a/projects/cdk/utils/focus/tests/set-native-focused.spec.ts b/projects/cdk/utils/focus/tests/set-native-focused.spec.ts
@@ -5,17 +5,17 @@ describe('setNativeFocused', () => {
     let top: number;
 
     beforeEach(() => {
-        if (element) {
-            document.body.removeChild(element);
-        }
-
         element = document.createElement('input');
         element.style.marginTop = '200vh';
         document.body.scrollTop = 0;
         document.body.appendChild(element);
         top = element.getBoundingClientRect().top;
     });
 
+    afterEach(() => {
+        document.body.removeChild(element);
+    });
+
     it('Element is not focused', () => {
         expect(document.activeElement).not.toBe(element);
     });