Skip to content

Commit af2b4dd

Browse files
fix: remove another accidental import of test code from the build
1 parent 7d73945 commit af2b4dd

File tree

5 files changed

+28
-28
lines changed

5 files changed

+28
-28
lines changed

src/editors/containers/EditorContainer/components/TitleHeader/EditConfirmationButtons.jsx

Lines changed: 24 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -3,39 +3,38 @@ import PropTypes from 'prop-types';
33

44
import { IconButtonWithTooltip, ButtonGroup, Icon } from '@openedx/paragon';
55
import { Check, Close } from '@openedx/paragon/icons';
6-
import { injectIntl, intlShape } from '@edx/frontend-platform/i18n';
6+
import { useIntl } from '@edx/frontend-platform/i18n';
77

88
import messages from './messages';
99

1010
const EditConfirmationButtons = ({
1111
updateTitle,
1212
cancelEdit,
13-
// injected
14-
intl,
15-
}) => (
16-
<ButtonGroup>
17-
<IconButtonWithTooltip
18-
tooltipPlacement="left"
19-
tooltipContent={intl.formatMessage(messages.saveTitleEdit)}
20-
src={Check}
21-
iconAs={Icon}
22-
onClick={updateTitle}
23-
/>
24-
<IconButtonWithTooltip
25-
tooltipPlacement="right"
26-
tooltipContent={intl.formatMessage(messages.cancelTitleEdit)}
27-
src={Close}
28-
iconAs={Icon}
29-
onClick={cancelEdit}
30-
/>
31-
</ButtonGroup>
32-
);
13+
}) => {
14+
const intl = useIntl();
15+
return (
16+
<ButtonGroup>
17+
<IconButtonWithTooltip
18+
tooltipPlacement="left"
19+
tooltipContent={intl.formatMessage(messages.saveTitleEdit)}
20+
src={Check}
21+
iconAs={Icon}
22+
onClick={updateTitle}
23+
/>
24+
<IconButtonWithTooltip
25+
tooltipPlacement="right"
26+
tooltipContent={intl.formatMessage(messages.cancelTitleEdit)}
27+
src={Close}
28+
iconAs={Icon}
29+
onClick={cancelEdit}
30+
/>
31+
</ButtonGroup>
32+
);
33+
};
3334

3435
EditConfirmationButtons.propTypes = {
3536
updateTitle: PropTypes.func.isRequired,
3637
cancelEdit: PropTypes.func.isRequired,
37-
// injected
38-
intl: intlShape.isRequired,
3938
};
40-
export const EditConfirmationButtonsInternal = EditConfirmationButtons; // For testing only
41-
export default injectIntl(EditConfirmationButtons);
39+
40+
export default EditConfirmationButtons;

src/editors/containers/EditorContainer/components/TitleHeader/EditConfirmationButtons.test.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import React from 'react';
33
import { shallow } from '@edx/react-unit-test-utils';
44
import { formatMessage } from '../../../../testUtils';
55

6-
import { EditConfirmationButtonsInternal as EditConfirmationButtons } from './EditConfirmationButtons';
6+
import EditConfirmationButtons from './EditConfirmationButtons';
77

88
describe('EditConfirmationButtons', () => {
99
const props = {

src/editors/containers/EditorContainer/components/TitleHeader/EditableHeader.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import React from 'react';
22
import PropTypes from 'prop-types';
33

44
import { Form } from '@openedx/paragon';
5-
import { EditConfirmationButtonsInternal as EditConfirmationButtons } from './EditConfirmationButtons';
5+
import EditConfirmationButtons from './EditConfirmationButtons';
66

77
const EditableHeader = ({
88
handleChange,

src/editors/containers/EditorContainer/components/TitleHeader/EditableHeader.test.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import React from 'react';
33
import { shallow } from '@edx/react-unit-test-utils';
44
import { Form } from '@openedx/paragon';
55
import { EditableHeaderInternal as EditableHeader } from './EditableHeader';
6-
import { EditConfirmationButtonsInternal as EditConfirmationButtons } from './EditConfirmationButtons';
6+
import EditConfirmationButtons from './EditConfirmationButtons';
77

88
describe('EditableHeader', () => {
99
const props = {

src/editors/setupEditorTest.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ jest.mock('@edx/frontend-platform/i18n', () => {
88
const PropTypes = jest.requireActual('prop-types');
99
return {
1010
...i18n,
11+
useIntl: () => ({ formatMessage: (m) => m.defaultMessage }),
1112
intlShape: PropTypes.shape({
1213
formatMessage: PropTypes.func,
1314
}),

0 commit comments

Comments
 (0)