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

🔨 Faster tests execution with babel #4338

Merged
merged 12 commits into from Oct 21, 2023
Merged

🔨 Faster tests execution with babel #4338

merged 12 commits into from Oct 21, 2023

Conversation

dubzzz
Copy link
Owner

@dubzzz dubzzz commented Oct 20, 2023

Up-to-now, we used to rely on ts-jest to run our tests against jest. But while powerful, it also imply checking the types which come with a non-negligeable cost. As we already check the types into a dedicated CI tasks, we prefer not spending this time again in the test part.

Let's see what will be the perfromance impact of such change.

Category:

  • ✨ Introduce new features
  • 📝 Add or update documentation
  • ✅ Add or update tests
  • 🐛 Fix a bug
  • 🏷️ Add or update types
  • ⚡️ Improve performance
  • Other(s): ...

Potential impacts:

  • Generated values
  • Shrink values
  • Performance
  • Typings
  • Other(s): ...

Up-to-now, we used to rely on ts-jest to run our tests against jest. But while powerful, it also imply checking the types which come with a non-negligeable cost. As we already check the types into a dedicated CI tasks, we prefer not spending this time again in the test part.

Let's see what will be the perfromance impact of such change.
@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 20, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 593a313:

Sandbox Source
Vanilla Configuration
@fast-check/examples Configuration

@codecov
Copy link

codecov bot commented Oct 21, 2023

Codecov Report

Merging #4338 (593a313) into main (75a6dd0) will decrease coverage by 1.15%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4338      +/-   ##
==========================================
- Coverage   94.71%   93.56%   -1.15%     
==========================================
  Files         210      207       -3     
  Lines        5958     5019     -939     
  Branches     1370     1367       -3     
==========================================
- Hits         5643     4696     -947     
- Misses        299      323      +24     
+ Partials       16        0      -16     
Flag Coverage Δ
unit-tests 93.56% <ø> (-1.15%) ⬇️
unit-tests-16.x-Linux ?
unit-tests-18.x-Linux 93.56% <ø> (-1.15%) ⬇️
unit-tests-latest-Linux 93.56% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 195 files with indirect coverage changes

@dubzzz dubzzz merged commit 183150d into main Oct 21, 2023
49 of 50 checks passed
@dubzzz dubzzz deleted the faster-ci branch October 21, 2023 14:32
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.

None yet

1 participant