Skip to content
This repository was archived by the owner on Aug 29, 2023. It is now read-only.

Adds jobs for testing webkit and webkit-webworker#487

Merged
galargh merged 1 commit into
masterfrom
feat/allow-testing-on-webkit
Mar 15, 2023
Merged

Adds jobs for testing webkit and webkit-webworker#487
galargh merged 1 commit into
masterfrom
feat/allow-testing-on-webkit

Conversation

@achingbrain

@achingbrain achingbrain commented Mar 8, 2023

Copy link
Copy Markdown
Contributor

To use in a project, declare npm scripts similar to test:firefox and test:firefox-webworker:

// package.json
  "scripts": {
    // existing lines:
    "test:firefox": "aegir test -t browser -- --browser firefox",
    "test:firefox-webworker": "aegir test -t webworker -- --browser firefox",
    // add the lines below:
    "test:webkit": "aegir test -t browser -- --browser webkit",
    "test:webkit-webworker": "aegir test -t webworker -- --browser webkit"
  }

Runs webkit tests on MacOS X and Linux, not sure if both are necessary tbh.

To use in a project, declare npm scripts similar to `test:firefox` and `test:firefox-webworker`:

```js
// package.json
// ... other stuff
  "scripts": {
    // ... other stuff
    "test:firefox": "aegir test -t browser -f ./dist/test/browser.js -- --browser firefox",
    "test:firefox-webworker": "aegir test -t webworker -f ./dist/test/browser.js -- --browser firefox",
    "test:webkit": "aegir test -t browser -f ./dist/test/browser.js -- --browser webkit",
    "test:webkit-webworker": "aegir test -t webworker -f ./dist/test/browser.js -- --browser webkit",
  }
// ... other stuff
```
@achingbrain achingbrain requested a review from a team as a code owner March 8, 2023 17:54
@achingbrain achingbrain requested a review from laurentsenta March 8, 2023 17:54
@MarcoPolo

Copy link
Copy Markdown

How can we check that this will actually pass without failing everything?

@achingbrain

Copy link
Copy Markdown
Contributor Author

You can make the changes here manually in a PR to any repo that uses js-test-and-release.yml to run it - any changes made to master in other repos will be overwritten when this PR is merged.

@MarcoPolo

Copy link
Copy Markdown

I see, this doesn't actually introduce a new job unless we have the test:webkit (or alternative) line in package.json.

@galargh galargh merged commit 46481dc into master Mar 15, 2023
@galargh galargh deleted the feat/allow-testing-on-webkit branch March 15, 2023 09:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants