diff --git a/package-lock.json b/package-lock.json index d63ba22a..4710fb22 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1841,6 +1841,15 @@ } } }, + "@loadable/component": { + "version": "5.10.3", + "resolved": "https://registry.npmjs.org/@loadable/component/-/component-5.10.3.tgz", + "integrity": "sha512-/aSO+tXw4vFMwZ6fgLaNQgLuEa7bgTpoBE4PxNzf08/ewAjymrCS3J7v3SbGE7IjGmmKL6vVwkpb7S3cYrk+ag==", + "requires": { + "@babel/runtime": "^7.6.0", + "hoist-non-react-statics": "^3.3.0" + } + }, "@mapbox/hast-util-table-cell-style": { "version": "0.1.3", "resolved": "https://registry.npmjs.org/@mapbox/hast-util-table-cell-style/-/hast-util-table-cell-style-0.1.3.tgz", @@ -2382,6 +2391,15 @@ "resolved": "https://registry.npmjs.org/@types/json-schema/-/json-schema-7.0.3.tgz", "integrity": "sha512-Il2DtDVRGDcqjDtE+rF8iqg1CArehSK84HZJCT7AMITlyXRBpuPhqGLDQMowraqqu1coEaimg4ZOqggt6L6L+A==" }, + "@types/loadable__component": { + "version": "5.10.0", + "resolved": "https://registry.npmjs.org/@types/loadable__component/-/loadable__component-5.10.0.tgz", + "integrity": "sha512-AaDP1VxV3p7CdPOtOTl3ALgQ6ES4AxJKO9UGj9vJonq/w2yERxwdzFiWNQFh9fEDXEzjxujBlM2RmSJtHV1/pA==", + "dev": true, + "requires": { + "@types/react": "*" + } + }, "@types/lunr": { "version": "2.3.2", "resolved": "https://registry.npmjs.org/@types/lunr/-/lunr-2.3.2.tgz", diff --git a/package.json b/package.json index 2024b97a..3de6166a 100644 --- a/package.json +++ b/package.json @@ -29,6 +29,7 @@ "not op_mini all" ], "dependencies": { + "@loadable/component": "^5.10.3", "classnames": "^2.2.6", "date-fns": "^2.8.1", "dms2dec": "^1.1.0", @@ -92,6 +93,7 @@ "@testing-library/react": "^9.3.2", "@types/classnames": "^2.2.9", "@types/jest": "^24.0.21", + "@types/loadable__component": "^5.10.0", "@types/lunr": "^2.3.2", "@types/node": "^12.12.11", "@types/react": "^16.9.11", diff --git a/src/components/atoms/Copy.tsx b/src/components/atoms/Copy.tsx index 2b903838..714f7e81 100644 --- a/src/components/atoms/Copy.tsx +++ b/src/components/atoms/Copy.tsx @@ -1,9 +1,10 @@ import React from 'react' -import Clipboard from 'react-clipboard.js' - +import loadable from '@loadable/component' import styles from './Copy.module.scss' import Icon from './Icon' +const Clipboard = loadable(() => import('react-clipboard.js')) + const onCopySuccess = (e: any) => { e.trigger.classList.add(styles.copied) } diff --git a/src/components/atoms/Qr.test.tsx b/src/components/atoms/Qr.test.tsx index 5666f4da..5b719367 100644 --- a/src/components/atoms/Qr.test.tsx +++ b/src/components/atoms/Qr.test.tsx @@ -1,12 +1,19 @@ -import React from 'react' -import { render, fireEvent } from '@testing-library/react' +import React, { Suspense } from 'react' +import { render, fireEvent, waitForElement } from '@testing-library/react' import Qr from './Qr' describe('Qr', () => { - it('renders without crashing', () => { - const { container } = render() - expect(container.firstChild).toBeInTheDocument() + test('renders lazy', async () => { + const { container } = render( + + + + ) + const lazyElement = await waitForElement(() => + container.querySelector('button') + ) + expect(lazyElement).toBeInTheDocument() fireEvent.click(container.querySelector('button')) }) }) diff --git a/src/components/atoms/Time.test.tsx b/src/components/atoms/Time.test.tsx index 7b1bc4c6..a23d0967 100644 --- a/src/components/atoms/Time.test.tsx +++ b/src/components/atoms/Time.test.tsx @@ -1,9 +1,16 @@ -import React from 'react' -// import { render } from '@testing-library/react' -import testRender from '../../../jest/testRender' +import React, { Suspense } from 'react' +import { render, waitForElement } from '@testing-library/react' import Time from './Time' describe('Time', () => { - testRender(