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

feat: migrate test suite to Node.js native tests and ESM #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jlucaso1
Copy link

@jlucaso1 jlucaso1 commented Sep 11, 2024

  • Replace Mocha with Node.js native test runner
  • Convert all test files from CommonJS to ES Modules
  • Add some CLI tests
  • Use esbuild instead rollup (has two different output files specific for cjs and esm)
  • Deleted AMD test because we don't compile to AMD/UMD anymore (the esm will work on browsers)

This commit modernizes our testing infrastructure by leveraging Node.js built-in test runner and embracing ES Modules across our test suite. It also introduces new tests for the CLI functionality.

- Replace Mocha with Node.js native test runner
- Convert all test files from CommonJS to ES Modules
- Add CLI tests
- Use esbuild instead rollup

This commit modernizes our testing infrastructure by leveraging Node.js
built-in test runner and embracing ES Modules across our test suite.
It also introduces new tests for the CLI functionality.
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