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

Replace AVA with bun:test #188

Merged
merged 7 commits into from Sep 28, 2023
Merged

Replace AVA with bun:test #188

merged 7 commits into from Sep 28, 2023

Conversation

fregante
Copy link
Member

@fregante fregante commented Sep 28, 2023

Followup to:

AVA:

❯ time npx ava

  14106 tests passed

________________________________________________________
Executed in    6.91 secs    fish           external
   usr time    7.27 secs   93.00 micros    7.27 secs
   sys time    0.91 secs  495.00 micros    0.91 secs

Bun:

 14106 pass
 0 fail
Ran 14106 tests across 1 files. [518.00ms]

Welp 😅

@fregante
Copy link
Member Author

Additionally, AVA does not run at all on Node 20 and I'm done debugging Node’s stupid errors

(node:73995) ExperimentalWarning: Custom ESM Loaders is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
  Uncaught exception in test.ts

  TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for ./test.ts

  ✘ test.ts exited with a non-zero exit code: 1
  ─

  1 uncaught exception

@fregante fregante marked this pull request as ready for review September 28, 2023 09:43
@fregante
Copy link
Member Author

I also tried to replace the build, but it's not enough at the moment. This is how far I got:

bun build index.ts --outfile distribution/index.bun.js

Issues:

  • I wasn't able to shake the collector
  • We still need Vite for the demo, which can never be replaced by Bun (because the entry point is index.html)

@fregante
Copy link
Member Author

Love this part

Screenshot

@fregante fregante changed the title Try bun:test Replace AVA with bun:test Sep 28, 2023
@fregante fregante merged commit af4ff81 into main Sep 28, 2023
19 checks passed
@fregante fregante deleted the bun-tests branch September 28, 2023 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant