Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Tests improvements #112

Open
robertsLando opened this issue Oct 23, 2024 · 2 comments
Open

Tests improvements #112

robertsLando opened this issue Oct 23, 2024 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@robertsLando
Copy link
Member

robertsLando commented Oct 23, 2024 •
edited
Loading

After some investigation many tests were broke. Here a list of broken tests that require investigation and some possible improvements:

  • test-00-sea: when running produced binary an error is thrown on windows and mac(fixed by fix: follow exact SEA baking procedure for Mac #116)
  • test-50-fs-runtime-layer-3: fs.writeFileSync throws ENOENT: no such file or directory, open /snapshot/test-50-fs-runtime-layer-3/test-z-asset.css on node 20/22
  • test-79-npm: this test is HUGE and has been disabled as it's not reliable. Understand if restore it or not
  • test-46-multi-arch: add macos support
  • test-80-compression-node-opcua: takes forever (from 5 min on linux up to 10 minutes on windows). Make it faster cc @erossignon
  • Enable parallel tests
@robertsLando
Copy link
Member Author

cc @faulpeltz any clue about the test-50-fs-runtime-layer-3 error?

@robertsLando
Copy link
Member Author

Ref #111

@robertsLando robertsLando changed the title Fix tests Tests improvements Oct 23, 2024
@robertsLando robertsLando added enhancement New feature or request good first issue Good for newcomers labels Oct 23, 2024
robertsLando added a commit that referenced this issue Oct 23, 2024
@robertsLando
test: add macos multi arch test
3c51eb7 
Ref #112
@robertsLando robertsLando mentioned this issue Oct 23, 2024
Merged
robertsLando added a commit that referenced this issue Oct 23, 2024
@robertsLando
test: allow to run tests in parallel
2b378f0 
Ref #112
@robertsLando robertsLando mentioned this issue Oct 23, 2024
Merged
robertsLando added a commit that referenced this issue Oct 23, 2024
@robertsLando
test: add macos multi arch test (#113)
cd0095e 
Ref #112
@robertsLando robertsLando mentioned this issue Nov 4, 2024
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@robertsLando