Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🤖 TEST: Use vitest instead of jest #399

Merged
merged 5 commits into from
Sep 25, 2022
Merged

🤖 TEST: Use vitest instead of jest #399

merged 5 commits into from
Sep 25, 2022

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Sep 25, 2022

No description provided.

@socket-security
Copy link

socket-security bot commented Sep 25, 2022

Socket Security Report

👍 No new dependency issues detected in pull request

Socket.dev scan summary
Issue Status
Did you mean? ✅ no new possible package typos
Install scripts ✅ no new install scripts
Telemetry ✅ no new telemetry
Troll package ✅ no new troll packages
Malware ✅ no new malware
Native code ✅ no new native modules
Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@2.4.2

Ignoring: bufferutil@4.0.6, utf-8-validate@5.0.9, esbuild@0.15.9

Powered by socket.dev

@codecov
Copy link

codecov bot commented Sep 25, 2022

Codecov Report

Merging #399 (0c9b2db) into master (5f0c4a8) will decrease coverage by 1.46%.
The diff coverage is n/a.

❗ Current head 0c9b2db differs from pull request most recent head ab2c674. Consider uploading reports for the commit ab2c674 to get more accurate results

@@            Coverage Diff             @@
##           master     #399      +/-   ##
==========================================
- Coverage   98.67%   97.21%   -1.47%     
==========================================
  Files           4        4              
  Lines         378      753     +375     
  Branches      124      167      +43     
==========================================
+ Hits          373      732     +359     
- Misses          5       21      +16     
Impacted Files Coverage Δ
src/index.ts 94.44% <0.00%> (-5.56%) ⬇️
src/HttpClient.ts 96.82% <0.00%> (-1.73%) ⬇️
src/utils.ts 97.65% <0.00%> (-0.68%) ⬇️
src/HttpAgent.ts 100.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@fengmk2
Copy link
Member Author

fengmk2 commented Sep 25, 2022

@SocketSecurity ignore bufferutil@4.0.6
@SocketSecurity ignore utf-8-validate@5.0.9
@SocketSecurity ignore esbuild@0.15.9

@fengmk2
Copy link
Member Author

fengmk2 commented Sep 25, 2022

@SocketSecurity ignore utf-8-validate@5.0.9 esbuild@0.15.9

@fengmk2
Copy link
Member Author

fengmk2 commented Sep 25, 2022

vitest coverage 在 node >= 16 会 hang 住,先 skip vitest-dev/vitest#2008

"selfsigned": "^2.0.1",
"ts-jest": "28",
"typescript": "4"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jest 需要4个npm包,vitest 需要2个,更新负担减半。

@fengmk2 fengmk2 merged commit 65abb60 into master Sep 25, 2022
@fengmk2 fengmk2 deleted the vitest branch September 25, 2022 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant