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: add wasmer-argus #4506

Merged
merged 37 commits into from
Mar 21, 2024
Merged

feat: add wasmer-argus #4506

merged 37 commits into from
Mar 21, 2024

Conversation

xdoardo
Copy link
Contributor

@xdoardo xdoardo commented Mar 20, 2024

Description

wasmer-argus is a test runner. It fetches packages from a user-defined WebContainer registry using its graphql endpoints: by default, this endpoint is http://registry.wasmer.io/graphql. Internally, it uses the wasmer-api crate, therefore the target registry must be compatible with the queries defined in that crate.

Since registries, to fetch every package in them, might need an authorization token, wasmer-argus has support for user-defined tokens, which can be gathered automatically from the environment using the WASMER_TOKEN variable or from the CLI arguments with a dedicated flag.

I tried to make the testing process as parallelised as possible. wasmer-argus gathers informations about the available parallelism in the host machine using std::thread::available_parallelism; the user can, however, specify "the amount of parallelism" desired -- which controls the number of concurrent tests -- using another dedicated CLI flag.

Further documentation can be found in the README.md.

Further steps

  • better documentation (--help is already quite explainatory)
  • a bit more testing
  • gather feedback

Copy link
Contributor

@theduke theduke left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@syrusakbary syrusakbary left a comment

Choose a reason for hiding this comment

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

This LGTM, but you should wait on input from @theduke

@theduke theduke merged commit be89624 into wasmerio:master Mar 21, 2024
59 checks passed
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.

3 participants