Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

test: introduce vitest #284

Merged
merged 10 commits into from Jul 26, 2022
Merged

test: introduce vitest #284

merged 10 commits into from Jul 26, 2022

Conversation

rot1024
Copy link
Member

@rot1024 rot1024 commented Jul 25, 2022

  • Use Vitest and get ultra fast tests
    • Fix problem that jest-yaml-transform reports an error that node version of engine field in package.json is not met
    • Use react-inlinesvg instead of react-svg to avoid errors with Vitest
  • Refactor some unit tests
  • Add role prop to Flex and Icon component to improve accessibility

@rot1024 rot1024 requested a review from KaWaite as a code owner July 25, 2022 11:26
@netlify
Copy link

netlify bot commented Jul 25, 2022

Deploy Preview for reearth-web ready!

Name Link
🔨 Latest commit f78a0ac
🔍 Latest deploy log https://app.netlify.com/sites/reearth-web/deploys/62df912a1bd1720008151008
😎 Deploy Preview https://deploy-preview-284--reearth-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@codecov
Copy link

codecov bot commented Jul 25, 2022

Codecov Report

Merging #284 (f78a0ac) into main (b3570ab) will increase coverage by 5.20%.
The diff coverage is 95.60%.

Impacted file tree graph

@@            Coverage Diff            @@
##            main     #284      +/-   ##
=========================================
+ Coverage   6.12%   11.33%   +5.20%     
=========================================
  Files        439      480      +41     
  Lines       8797    45674   +36877     
  Branches    2240      384    -1856     
=========================================
+ Hits         539     5178    +4639     
- Misses      8181    40482   +32301     
+ Partials      77       14      -63     
Impacted Files Coverage Δ
...ts/molecules/Common/AssetModal/AssetCard/index.tsx 0.00% <0.00%> (ø)
...olecules/Common/AssetModal/AssetListItem/index.tsx 0.00% <0.00%> (ø)
...ents/molecules/Visualizer/Block/Location/index.tsx 0.00% <0.00%> (ø)
src/components/atoms/Icon/index.tsx 79.76% <65.71%> (+15.05%) ⬆️
src/components/atoms/Accordion/index.test.tsx 100.00% <100.00%> (ø)
src/components/atoms/AutoComplete/index.test.tsx 100.00% <100.00%> (ø)
src/components/atoms/Flex/index.tsx 100.00% <100.00%> (ø)
src/components/atoms/Notification/index.test.tsx 100.00% <100.00%> (ø)
src/components/atoms/Notification/index.tsx 90.42% <100.00%> (+23.75%) ⬆️
src/components/atoms/Plugin/IFrame/index.test.tsx 100.00% <100.00%> (ø)
... and 498 more

@rot1024 rot1024 merged commit 2152e00 into main Jul 26, 2022
@rot1024 rot1024 deleted the vitest branch July 26, 2022 08:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants