Skip to content

Commit

Permalink
[1.0][Test] Wrap FunComponent in unit test with IntlProvider
Browse files Browse the repository at this point in the history
In the FunctionComponent unit tests, we see many console errors:
`Could not find required intl object. <IntlProvider> needs to
exist in the component ancestry. `

This is because for some unit tests, we mount the FunctionComponent
(with Enzyme's mount()) , which access to the react-intl context by
FormattedMessage without their <IntlProvider /> parent wrapper.

This PR solves 7 out of 8 unit tests with this issue by wrapping the
<IntlProvider /> either through original enzyme_helper functions or
a simple wrapper wrapWithIntl.

Partically Resolved:
opensearch-project#593

Backport PR:
opensearch-project#654

Signed-off-by: Anan Zhuang <[email protected]>
  • Loading branch information
ananzh committed Jul 23, 2021
1 parent 4f0f706 commit f01ed9a
Show file tree
Hide file tree
Showing 11 changed files with 561 additions and 523 deletions.
3 changes: 2 additions & 1 deletion src/core/public/application/ui/app_container.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@
import React from 'react';
import { act } from 'react-dom/test-utils';
import { mount } from 'enzyme';
import { mountWithIntl } from 'test_utils/enzyme_helpers';

import { AppContainer } from './app_container';
import { Mounter, AppMountParameters, AppStatus } from '../types';
Expand Down Expand Up @@ -82,7 +83,7 @@ describe('AppContainer', () => {
const [waitPromise, resolvePromise] = createResolver();
const mounter = createMounter(waitPromise);

const wrapper = mount(
const wrapper = mountWithIntl(
<AppContainer
appPath={`/app/${appId}`}
appId={appId}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,9 @@
*/

import React from 'react';
import { mount } from 'enzyme';
import { ServerStatus } from './server_status';
import { FormattedStatus } from '../lib';
import { mountWithIntl } from 'test_utils/enzyme_helpers';

const getStatus = (parts: Partial<FormattedStatus['state']> = {}): FormattedStatus['state'] => ({
id: 'green',
Expand All @@ -46,7 +46,7 @@ const getStatus = (parts: Partial<FormattedStatus['state']> = {}): FormattedStat
describe('ServerStatus', () => {
it('renders correctly for green state', () => {
const status = getStatus();
const component = mount(<ServerStatus serverState={status} name="My Computer" />);
const component = mountWithIntl(<ServerStatus serverState={status} name="My Computer" />);
expect(component.find('EuiTitle').text()).toMatchInlineSnapshot(
`"OpenSearch Dashboards status is Green"`
);
Expand All @@ -58,18 +58,20 @@ describe('ServerStatus', () => {
id: 'red',
title: 'Red',
});
const component = mount(<ServerStatus serverState={status} name="My Computer" />);
const component = mountWithIntl(<ServerStatus serverState={status} name="My Computer" />);
expect(component.find('EuiTitle').text()).toMatchInlineSnapshot(
`"OpenSearch Dashboards status is Red"`
);
expect(component.find('EuiBadge')).toMatchSnapshot();
});

it('displays the correct `name`', () => {
let component = mount(<ServerStatus serverState={getStatus()} name="Localhost" />);
let component = mountWithIntl(<ServerStatus serverState={getStatus()} name="Localhost" />);
expect(component.find('EuiText').text()).toMatchInlineSnapshot(`"Localhost"`);

component = mount(<ServerStatus serverState={getStatus()} name="OpenSearchDashboards" />);
component = mountWithIntl(
<ServerStatus serverState={getStatus()} name="OpenSearchDashboards" />
);
expect(component.find('EuiText').text()).toMatchInlineSnapshot(`"OpenSearchDashboards"`);
});
});
Loading

0 comments on commit f01ed9a

Please sign in to comment.