-
Notifications
You must be signed in to change notification settings - Fork 305
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
chore(content-uploader): Migrate UploadsManagerAction (#3604)
* chore(content-uploader): Migrate UploadsManagerAction * chore(content-uploader): comments * chore(content-uploader): update onClick type * chore(content-uploader): replace injectIntl by useIntl --------- Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
- Loading branch information
1 parent
8ad18b8
commit 1746e3c
Showing
6 changed files
with
82 additions
and
80 deletions.
There are no files selected for viewing
34 changes: 16 additions & 18 deletions
34
.../content-uploader/UploadsManagerAction.js → ...ent-uploader/UploadsManagerAction.js.flow
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,3 @@ | ||
.bcu-uploads-manager-action { | ||
margin: 5px -20px 5px 5px; | ||
|
||
.btn-primary { | ||
display: flex; | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
import * as React from 'react'; | ||
import { useIntl } from 'react-intl'; | ||
import { Button } from '@box/blueprint-web'; | ||
|
||
import { RESIN_TAG_TARGET } from '../../common/variables'; | ||
import { STATUS_ERROR } from '../../constants'; | ||
|
||
import messages from '../common/messages'; | ||
import type { UploadStatus } from '../../common/types/upload'; | ||
|
||
import './UploadsManagerAction.scss'; | ||
|
||
export interface UploadsManagerActionProps { | ||
hasMultipleFailedUploads: boolean; | ||
onClick: (status: UploadStatus) => void; | ||
} | ||
|
||
const UploadsManagerAction = ({ hasMultipleFailedUploads, onClick }: UploadsManagerActionProps) => { | ||
const handleResumeClick = event => { | ||
event.stopPropagation(); | ||
onClick(STATUS_ERROR); | ||
}; | ||
const intl = useIntl(); | ||
const resumeMessage = hasMultipleFailedUploads ? messages.resumeAll : messages.resume; | ||
const resin = { [RESIN_TAG_TARGET]: 'uploadresumeheader' }; | ||
|
||
return ( | ||
<div className="bcu-uploads-manager-action"> | ||
<Button onClick={handleResumeClick} size="small" variant="primary" {...resin}> | ||
{intl.formatMessage(resumeMessage)} | ||
</Button> | ||
</div> | ||
); | ||
}; | ||
|
||
export default UploadsManagerAction; |
23 changes: 0 additions & 23 deletions
23
src/elements/content-uploader/__tests__/UploadsManagerAction.test.js
This file was deleted.
Oops, something went wrong.
30 changes: 30 additions & 0 deletions
30
src/elements/content-uploader/__tests__/UploadsManagerAction.test.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
import * as React from 'react'; | ||
import { render, screen, fireEvent } from '../../../test-utils/testing-library'; | ||
|
||
import UploadsManagerAction, { UploadsManagerActionProps } from '../UploadsManagerAction'; | ||
|
||
describe('elements/content-uploader/UploadsManagerAction', () => { | ||
const getWrapper = (props: Partial<UploadsManagerActionProps>) => | ||
render(<UploadsManagerAction hasMultipleFailedUploads={false} onClick={jest.fn()} {...props} />); | ||
|
||
test('should render correctly with hasMultipleFailedUploads as true', () => { | ||
getWrapper({ hasMultipleFailedUploads: true }); | ||
|
||
expect(screen.getByText('Resume All')).toBeInTheDocument(); | ||
}); | ||
|
||
test('should render correctly with hasMultipleFailedUploads as false', () => { | ||
getWrapper({ hasMultipleFailedUploads: false }); | ||
|
||
expect(screen.getByText('Resume')).toBeInTheDocument(); | ||
expect(screen.queryByText('Resume All')).not.toBeInTheDocument(); | ||
}); | ||
|
||
test('should call onClick when resume button is clicked', () => { | ||
const handleResumeClick = jest.fn(); | ||
getWrapper({ onClick: handleResumeClick }); | ||
|
||
fireEvent.click(screen.getByText('Resume')); | ||
expect(handleResumeClick).toHaveBeenCalledTimes(1); | ||
}); | ||
}); |
35 changes: 0 additions & 35 deletions
35
src/elements/content-uploader/__tests__/__snapshots__/UploadsManagerAction.test.js.snap
This file was deleted.
Oops, something went wrong.