Skip to content

Add a new "botbrowsertest" gulp-task, to allow running only the browser tests on the bots #13730

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

Merged
merged 1 commit into from
Jul 15, 2021

Conversation

Snuffleupagus
Copy link
Collaborator

Currently it's possible to run e.g. gulp unittest, gulp fonttest, and gulp integrationtest separately on the bots; see https://github.com/mozilla/botio-files-pdfjs
However, it's not possible to run only the gulp browsertest command on the bots without also running the full test-suite. In some cases, e.g. if the "browsertest" times out, having a way to only re-run those would thus save some time and resources.

If/when this patch lands, I'll follow-up with a patch adding a new on_cmd_browsertest.js file to the https://github.com/mozilla/botio-files-pdfjs repository.

…er tests on the bots

Currently it's possible to run e.g. `gulp unittest`, `gulp fonttest`, and `gulp integrationtest` *separately* on the bots; see https://github.com/mozilla/botio-files-pdfjs
However, it's not possible to run *only* the `gulp browsertest` command on the bots without also running the full test-suite. In some cases, e.g. if the "browsertest" times out, having a way to only re-run those would thus save some time and resources.

If/when this patch lands, I'll follow-up with a patch adding a new `on_cmd_browsertest.js` file to the https://github.com/mozilla/botio-files-pdfjs repository.
@Snuffleupagus
Copy link
Collaborator Author

Unfortunately this can only be tested locally, since there's not yet a botio-command that can trigger this new gulp-task.

@timvandermeij timvandermeij merged commit ad34f16 into mozilla:master Jul 15, 2021
@timvandermeij
Copy link
Contributor

Looks good! This has indeed been a bit of a silly missing feature given that the other test targets were supported.

@timvandermeij
Copy link
Contributor

Once you have a PR for botio, we should also add that command to https://github.com/mozilla/pdf.js/wiki/Bots.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants