Fix Datagrid row does not appear clickable although it is #10160
Conversation
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
There was a problem hiding this comment.
Because this test is made to please typescript, who complains on the following line that row may be undefined when trying to access the row classList. It works with my test, not with expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
| fireEvent.click(screen.getByText('hello')); | ||
| const cell = screen.getByText('hello'); | ||
| const row = cell.closest('tr'); | ||
| if (!row) { |
There was a problem hiding this comment.
Why not use expect(row).toBeDefined();
Closes #10155