Skip to content

Commit b5dd71e

Browse files
test: remove snapshots and use rtl tests
1 parent e413464 commit b5dd71e

File tree

6 files changed

+18
-1486
lines changed

6 files changed

+18
-1486
lines changed

src/__snapshots__/index.test.jsx.snap

Lines changed: 0 additions & 228 deletions
This file was deleted.

src/course-home/courseware-search/CoursewareSearchEmpty.test.jsx

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import {
55
screen,
66
} from '../../setupTest';
77
import CoursewareSearchEmpty from './CoursewareSearchEmpty';
8+
import messages from './messages';
89

910
function renderComponent() {
1011
const { container } = render(<CoursewareSearchEmpty />);
@@ -16,9 +17,12 @@ describe('CoursewareSearchEmpty', () => {
1617
initializeMockApp();
1718
});
1819

19-
it('should match the snapshot', () => {
20+
it('render empty results text and corresponding classes', () => {
2021
renderComponent();
21-
22-
expect(screen.getByTestId('no-results')).toMatchSnapshot();
22+
const emptyText = screen.getByText(messages.searchResultsNone.defaultMessage);
23+
expect(emptyText).toBeInTheDocument();
24+
expect(emptyText).toHaveClass('courseware-search-results__empty');
25+
expect(emptyText).toHaveAttribute('data-testid', 'no-results');
26+
expect(emptyText.parentElement).toHaveClass('courseware-search-results');
2327
});
2428
});

src/course-home/courseware-search/CoursewareSearchResults.test.jsx

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
import CoursewareSearchResults from './CoursewareSearchResults';
88
import messages from './messages';
99
import searchResultsFactory from './test-data/search-results-factory';
10+
import * as mock from './test-data/mocked-response.json';
1011

1112
jest.mock('react-redux');
1213

@@ -34,8 +35,15 @@ describe('CoursewareSearchResults', () => {
3435
renderComponent({ results });
3536
});
3637

37-
it('should match the snapshot', () => {
38-
expect(screen.getByTestId('search-results')).toMatchSnapshot();
38+
it('should render complete list', () => {
39+
const courses = screen.getAllByRole('link');
40+
expect(courses.length).toBe(mock.results.length);
41+
});
42+
43+
it('should render correct title for first course', () => {
44+
const courses = screen.getAllByRole('link');
45+
const firstCourseTitle = courses[0].querySelector('.courseware-search-results__title span');
46+
expect(firstCourseTitle.innerHTML).toEqual(mock.results[0].data.content.display_name);
3947
});
4048
});
4149
});

src/course-home/courseware-search/__snapshots__/CoursewareSearchEmpty.test.jsx.snap

Lines changed: 0 additions & 10 deletions
This file was deleted.

0 commit comments

Comments
 (0)