Skip to content

Commit c0032c2

Browse files
committed
Fix issues related to console spies in createReducer.test.ts
1 parent 4d09095 commit c0032c2

File tree

1 file changed

+16
-17
lines changed

1 file changed

+16
-17
lines changed

packages/toolkit/src/tests/createReducer.test.ts

Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,16 @@
1-
import { vi } from 'vitest'
21
import type {
32
CaseReducer,
4-
PayloadAction,
53
Draft,
4+
PayloadAction,
65
Reducer,
76
UnknownAction,
87
} from '@reduxjs/toolkit'
9-
import { isPlainObject } from '@reduxjs/toolkit'
10-
import { createReducer, createAction, createNextState } from '@reduxjs/toolkit'
118
import {
12-
mockConsole,
13-
createConsole,
14-
getLog,
15-
} from 'console-testing-library/pure'
9+
createAction,
10+
createNextState,
11+
createReducer,
12+
isPlainObject,
13+
} from '@reduxjs/toolkit'
1614

1715
interface Todo {
1816
text: string
@@ -42,11 +40,6 @@ type ToggleTodoReducer = CaseReducer<
4240
type CreateReducer = typeof createReducer
4341

4442
describe('createReducer', () => {
45-
let restore: () => void
46-
47-
beforeEach(() => {
48-
restore = mockConsole(createConsole())
49-
})
5043
describe('given impure reducers with immer', () => {
5144
const addTodo: AddTodoReducer = (state, action) => {
5245
const { newTodo } = action.payload
@@ -84,8 +77,11 @@ describe('createReducer', () => {
8477
it('Throws an error if the legacy object notation is used', async () => {
8578
const { createReducer } = await import('../createReducer')
8679
const wrapper = () => {
87-
// @ts-ignore
88-
let dummyReducer = (createReducer as CreateReducer)([] as TodoState, {})
80+
const dummyReducer = (createReducer as CreateReducer)(
81+
[] as TodoState,
82+
// @ts-ignore
83+
{},
84+
)
8985
}
9086

9187
expect(wrapper).toThrowError(
@@ -101,8 +97,11 @@ describe('createReducer', () => {
10197
process.env.NODE_ENV = 'production'
10298
const { createReducer } = await import('../createReducer')
10399
const wrapper = () => {
104-
// @ts-ignore
105-
let dummyReducer = (createReducer as CreateReducer)([] as TodoState, {})
100+
const dummyReducer = (createReducer as CreateReducer)(
101+
[] as TodoState,
102+
// @ts-ignore
103+
{},
104+
)
106105
}
107106

108107
expect(wrapper).toThrowError()

0 commit comments

Comments
 (0)