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

Adds jobs for testing webkit and webkit-webworker #487

Merged
merged 1 commit into from
Mar 15, 2023

Conversation

achingbrain
Copy link
Contributor

@achingbrain achingbrain commented Mar 8, 2023

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

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

@achingbrain
Copy link
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

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