Conversation
| }) | ||
|
|
||
| test('должен добавить новую задачу', () => { | ||
| const previousState = {todos: [], activeCount: 0,}; |
There was a problem hiding this comment.
Используй силу beforeEach для инициализации начального стейта, чтобы не писать его в каждом тесте
DRY))
There was a problem hiding this comment.
Там в первом случае начальный стейт отличается от остальных, но ты права, остальные можно выделить в отдельный блок)
There was a problem hiding this comment.
expect(reducer(undefined, { type: undefined })).toEqual({todos: [], activeCount: 0, })
})
в assertion тот же самый prevState :)
const previousState = {todos: [], activeCount: 0,};
There was a problem hiding this comment.
Всё равно не особо соглашусь - начальный стейт используется в 3х тестах из 5, использовать beforeEach не целесообразно. А вот объявить константу имеет смысл)
|
|
||
| test('должен добавить новую задачу', () => { | ||
| const previousState = {todos: [], activeCount: 0,}; | ||
| expect(reducer(previousState, todoAdded({data:'data', id:0, done:false}))).toMatchObject({activeCount: 1, todos:[{data:'data', done:false}]}); |
There was a problem hiding this comment.
toEqual здесь будет лучше, поскольку toMatchObject проверяет содержит ли проверяемый объект те свойства, которые ты ему передаешь, но не гарантирует полную идентичность, в отличие от toEqual
There was a problem hiding this comment.
Тут это сделано намеренно - id будет отличаться.
2 participants
tests added