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

Move qunit tests from PhantomJS to Headless Chromium #259

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

Conversation

Krinkle
Copy link
Contributor

@Krinkle Krinkle commented May 1, 2022

PhantomJS is no longer maintained and I was unable to get the tests to pass in a development environment running the latest Debian Linux.

Update grunt-contrib-qunit to the latest version, which uses Puppeteer to drive the latest Chromium version instead of PhantomJS.

grunt-contrib-qunit 6 requires QUnit 2.2.0 or later, so while at it also update the test files from QUnit 2.0.1 to 2.19.0.

See also:

PhantomJS is no longer maintained and I was unable to get the tests
to pass in a development environment running the latest Debian Linux.

Update grunt-contrib-qunit to the latest version, which uses
Puppeteer to drive the latest Chromium version instead of PhantomJS.

grunt-contrib-qunit 6 requires QUnit 2.2.0 or later, so while at it
also update the test files from QUnit 2.0.1 to 2.19.0.

See also:
* <https://github.com/gruntjs/grunt-contrib-qunit>.
* <https://pptr.dev/#?product=Puppeteer&version=v9.0.0&show=api-puppeteerlaunchoptions>
@Krinkle
Copy link
Contributor Author

Krinkle commented May 1, 2022

Test plan:

  • npm install
  • npx grunt qunit

If running inside a container, make sure CI=true is set as required by Headless Chromium in order to work without sudo. GitHub and Travis do this by default.

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