Skip to content

Commit 680021e

Browse files
committed
test: add tests for suppressErrorOutput export
1 parent d4b692a commit 680021e

File tree

3 files changed

+51
-9
lines changed

3 files changed

+51
-9
lines changed
Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,29 @@
1+
import { ReactHooksRenderer } from '../../types/react'
2+
13
// This verifies that if pure imports are used
24
// then we DON'T auto-wire up the afterEach for folks
35
describe('error output suppression (pure) tests', () => {
46
const originalConsoleError = console.error
57

8+
let suppressErrorOutput!: ReactHooksRenderer['suppressErrorOutput']
9+
610
beforeAll(() => {
7-
require('../pure')
11+
suppressErrorOutput = (require('../pure') as ReactHooksRenderer).suppressErrorOutput
812
})
913

1014
test('should not patch console.error', () => {
1115
expect(console.error).toBe(originalConsoleError)
1216
})
13-
})
1417

15-
export {}
18+
test('should manually patch console.error', () => {
19+
const restore = suppressErrorOutput()
20+
21+
try {
22+
expect(console.error).not.toBe(originalConsoleError)
23+
} finally {
24+
restore()
25+
}
26+
27+
expect(console.error).toBe(originalConsoleError)
28+
})
29+
})
Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,29 @@
1+
import { ReactHooksRenderer } from '../../types/react'
2+
13
// This verifies that if pure imports are used
24
// then we DON'T auto-wire up the afterEach for folks
35
describe('error output suppression (pure) tests', () => {
46
const originalConsoleError = console.error
57

8+
let suppressErrorOutput!: ReactHooksRenderer['suppressErrorOutput']
9+
610
beforeAll(() => {
7-
require('../pure')
11+
suppressErrorOutput = (require('../pure') as ReactHooksRenderer).suppressErrorOutput
812
})
913

1014
test('should not patch console.error', () => {
1115
expect(console.error).toBe(originalConsoleError)
1216
})
13-
})
1417

15-
export {}
18+
test('should manually patch console.error', () => {
19+
const restore = suppressErrorOutput()
20+
21+
try {
22+
expect(console.error).not.toBe(originalConsoleError)
23+
} finally {
24+
restore()
25+
}
26+
27+
expect(console.error).toBe(originalConsoleError)
28+
})
29+
})
Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,29 @@
1+
import { ReactHooksRenderer } from '../../types/react'
2+
13
// This verifies that if pure imports are used
24
// then we DON'T auto-wire up the afterEach for folks
35
describe('error output suppression (pure) tests', () => {
46
const originalConsoleError = console.error
57

8+
let suppressErrorOutput!: ReactHooksRenderer['suppressErrorOutput']
9+
610
beforeAll(() => {
7-
require('../pure')
11+
suppressErrorOutput = (require('../pure') as ReactHooksRenderer).suppressErrorOutput
812
})
913

1014
test('should not patch console.error', () => {
1115
expect(console.error).toBe(originalConsoleError)
1216
})
13-
})
1417

15-
export {}
18+
test('should manually patch console.error', () => {
19+
const restore = suppressErrorOutput()
20+
21+
try {
22+
expect(console.error).not.toBe(originalConsoleError)
23+
} finally {
24+
restore()
25+
}
26+
27+
expect(console.error).toBe(originalConsoleError)
28+
})
29+
})

0 commit comments

Comments
 (0)