Skip to content

Commit ed224cc

Browse files
committed
Moved BrowserUtils test cleanup into 'afterEach'
1 parent 93dbb0a commit ed224cc

1 file changed

Lines changed: 39 additions & 37 deletions

File tree

Lines changed: 39 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -1,54 +1,56 @@
11
import { BrowserUtils } from "./browser-utils";
22

33
describe("BrowserUtils", () => {
4-
describe("isIE", () => {
5-
it("when wondow.document.documentMode is undefined, then returns false", () => {
6-
// Arrange
7-
window.document.documentMode = undefined;
4+
afterEach(() => {
5+
// Clean up
6+
window.document.documentMode = undefined;
7+
});
88

9-
// Act
10-
const result = BrowserUtils.isIE();
9+
describe("isIE", () => {
10+
it("when wondow.document.documentMode is undefined, then returns false", () => {
11+
// Arrange
12+
window.document.documentMode = undefined;
1113

12-
// Assert
13-
expect(result).toBeFalse();
14-
});
14+
// Act
15+
const result = BrowserUtils.isIE();
16+
17+
// Assert
18+
expect(result).toBeFalse();
19+
});
1520

16-
it("when window.document.documentMode is truthy, then returns true", () => {
17-
// Arrange
18-
window.document.documentMode = true;
21+
it("when window.document.documentMode is truthy, then returns true", () => {
22+
// Arrange
23+
window.document.documentMode = true;
1924

20-
// Act
21-
const result = BrowserUtils.isIE();
25+
// Act
26+
const result = BrowserUtils.isIE();
2227

23-
// Assert
24-
expect(result).toBeTrue();
28+
// Assert
29+
expect(result).toBeTrue();
30+
});
2531
});
26-
});
2732

28-
describe("isNotIE", () => {
29-
it("when wondow.document.documentMode is undefined, then returns true", () => {
30-
// Arrange
31-
window.document.documentMode = undefined;
33+
describe("isNotIE", () => {
34+
it("when wondow.document.documentMode is undefined, then returns true", () => {
35+
// Arrange
36+
window.document.documentMode = undefined;
3237

33-
// Act
34-
const result = BrowserUtils.isNotIE();
38+
// Act
39+
const result = BrowserUtils.isNotIE();
3540

36-
// Assert
37-
expect(result).toBeTrue();
38-
});
41+
// Assert
42+
expect(result).toBeTrue();
43+
});
3944

40-
it("when window.document.documentMode is truthy, then returns false", () => {
41-
// Arrange
42-
window.document.documentMode = true;
45+
it("when window.document.documentMode is truthy, then returns false", () => {
46+
// Arrange
47+
window.document.documentMode = true;
4348

44-
// Act
45-
const result = BrowserUtils.isNotIE();
49+
// Act
50+
const result = BrowserUtils.isNotIE();
4651

47-
// Assert
48-
expect(result).toBeFalse();
52+
// Assert
53+
expect(result).toBeFalse();
54+
});
4955
});
50-
});
51-
52-
// Clean up
53-
window.document.documentMode = undefined;
5456
});

0 commit comments

Comments
 (0)