Skip to content

Commit eaba380

Browse files
test: replacing shallow snapshots with RTL (#2266)
1 parent f3332a2 commit eaba380

File tree

10 files changed

+19
-861
lines changed

10 files changed

+19
-861
lines changed

src/editors/Placeholder.test.jsx

Lines changed: 4 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,36 +1,12 @@
11
import React from 'react';
2-
import { IntlProvider } from '@edx/frontend-platform/i18n';
3-
import TestRenderer from 'react-test-renderer';
4-
import { AppContext } from '@edx/frontend-platform/react';
5-
import { Context as ResponsiveContext } from 'react-responsive';
2+
import { render, initializeMocks, screen } from '@src/testUtils';
63

74
import Placeholder from './Placeholder';
85

96
describe('<Placeholder />', () => {
7+
beforeEach(() => initializeMocks());
108
it('renders correctly', () => {
11-
const component = (
12-
<ResponsiveContext.Provider value={{ width: 1280 }}>
13-
<IntlProvider locale="en" messages={{}}>
14-
<AppContext.Provider
15-
value={{
16-
authenticatedUser: null,
17-
config: {
18-
LMS_BASE_URL: process.env.LMS_BASE_URL,
19-
SITE_NAME: process.env.SITE_NAME,
20-
LOGIN_URL: process.env.LOGIN_URL,
21-
LOGOUT_URL: process.env.LOGOUT_URL,
22-
LOGO_URL: process.env.LOGO_URL,
23-
},
24-
}}
25-
>
26-
<Placeholder />
27-
</AppContext.Provider>
28-
</IntlProvider>
29-
</ResponsiveContext.Provider>
30-
);
31-
32-
const wrapper = TestRenderer.create(component);
33-
34-
expect(wrapper.toJSON()).toMatchSnapshot();
9+
render(<Placeholder />);
10+
expect(screen.getByText(/Under Construction/)).toBeInTheDocument();
3511
});
3612
});

src/editors/__snapshots__/Placeholder.test.jsx.snap

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

src/editors/containers/VideoUploadEditor/VideoUploader.test.jsx

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
import React from 'react';
2-
import { render, fireEvent, act } from '@testing-library/react';
2+
import {
3+
render, fireEvent, act, screen,
4+
} from '@testing-library/react';
35
import { IntlProvider } from '@edx/frontend-platform/i18n';
46
import { initializeMockApp } from '@edx/frontend-platform';
57
import { configureStore } from '@reduxjs/toolkit';
@@ -53,7 +55,10 @@ describe('VideoUploader', () => {
5355
});
5456

5557
it('renders as expected with default behavior', async () => {
56-
expect(await renderComponent(store, setLoadingMock)).toMatchSnapshot();
58+
await renderComponent(store);
59+
expect(screen.getByPlaceholderText('Paste your video ID or URL')).toBeInTheDocument();
60+
expect(screen.getByText('Drag and drop video here or click to upload')).toBeInTheDocument();
61+
expect(screen.getByRole('button', { name: 'Submit' })).toBeInTheDocument();
5762
});
5863

5964
it('calls onURLUpload when URL submit button is clicked', async () => {

0 commit comments

Comments
 (0)