Skip to content

Commit a0716dc

Browse files
authored
chore: check if checkbox is checked (#63)
1 parent 4d1af82 commit a0716dc

File tree

2 files changed

+16
-6
lines changed

2 files changed

+16
-6
lines changed

assets/demo-todo-app.spec.js

+8-3
Original file line numberDiff line numberDiff line change
@@ -159,12 +159,14 @@ test.describe('Item', () => {
159159

160160
const firstTodo = page.getByTestId('todo-item').nth(0);
161161
const secondTodo = page.getByTestId('todo-item').nth(1);
162-
await firstTodo.getByRole('checkbox').check();
162+
const firstTodoCheckbox = firstTodo.getByRole('checkbox');
163+
164+
await firstTodoCheckbox.check();
163165
await expect(firstTodo).toHaveClass('completed');
164166
await expect(secondTodo).not.toHaveClass('completed');
165167
await checkNumberOfCompletedTodosInLocalStorage(page, 1);
166168

167-
await firstTodo.getByRole('checkbox').uncheck();
169+
await firstTodoCheckbox.uncheck();
168170
await expect(firstTodo).not.toHaveClass('completed');
169171
await expect(secondTodo).not.toHaveClass('completed');
170172
await checkNumberOfCompletedTodosInLocalStorage(page, 0);
@@ -311,8 +313,10 @@ test.describe('Persistence', () => {
311313
}
312314

313315
const todoItems = page.getByTestId('todo-item');
314-
await todoItems.nth(0).getByRole('checkbox').check();
316+
const firstTodoCheck = todoItems.nth(0).getByRole('checkbox');
317+
await firstTodoCheck.check();
315318
await expect(todoItems).toHaveText([TODO_ITEMS[0], TODO_ITEMS[1]]);
319+
await expect(firstTodoCheck).toBeChecked();
316320
await expect(todoItems).toHaveClass(['completed', '']);
317321

318322
// Ensure there is 1 completed item.
@@ -321,6 +325,7 @@ test.describe('Persistence', () => {
321325
// Now reload.
322326
await page.reload();
323327
await expect(todoItems).toHaveText([TODO_ITEMS[0], TODO_ITEMS[1]]);
328+
await expect(firstTodoCheck).toBeChecked();
324329
await expect(todoItems).toHaveClass(['completed', '']);
325330
});
326331
});

assets/demo-todo-app.spec.ts

+8-3
Original file line numberDiff line numberDiff line change
@@ -158,12 +158,14 @@ test.describe('Item', () => {
158158

159159
const firstTodo = page.getByTestId('todo-item').nth(0);
160160
const secondTodo = page.getByTestId('todo-item').nth(1);
161-
await firstTodo.getByRole('checkbox').check();
161+
const firstTodoCheckbox = firstTodo.getByRole('checkbox');
162+
163+
await firstTodoCheckbox.check();
162164
await expect(firstTodo).toHaveClass('completed');
163165
await expect(secondTodo).not.toHaveClass('completed');
164166
await checkNumberOfCompletedTodosInLocalStorage(page, 1);
165167

166-
await firstTodo.getByRole('checkbox').uncheck();
168+
await firstTodoCheckbox.uncheck();
167169
await expect(firstTodo).not.toHaveClass('completed');
168170
await expect(secondTodo).not.toHaveClass('completed');
169171
await checkNumberOfCompletedTodosInLocalStorage(page, 0);
@@ -311,8 +313,10 @@ test.describe('Persistence', () => {
311313
}
312314

313315
const todoItems = page.getByTestId('todo-item');
314-
await todoItems.nth(0).getByRole('checkbox').check();
316+
const firstTodoCheck = todoItems.nth(0).getByRole('checkbox');
317+
await firstTodoCheck.check();
315318
await expect(todoItems).toHaveText([TODO_ITEMS[0], TODO_ITEMS[1]]);
319+
await expect(firstTodoCheck).toBeChecked();
316320
await expect(todoItems).toHaveClass(['completed', '']);
317321

318322
// Ensure there is 1 completed item.
@@ -321,6 +325,7 @@ test.describe('Persistence', () => {
321325
// Now reload.
322326
await page.reload();
323327
await expect(todoItems).toHaveText([TODO_ITEMS[0], TODO_ITEMS[1]]);
328+
await expect(firstTodoCheck).toBeChecked();
324329
await expect(todoItems).toHaveClass(['completed', '']);
325330
});
326331
});

0 commit comments

Comments
 (0)