Skip to content

Commit

Permalink
Switch to yarn
Browse files Browse the repository at this point in the history
Switches from `npm` to `yarn` for dev dependencies for compatibility with other related projects.
  • Loading branch information
rubenarakelyan committed Sep 20, 2023
1 parent eae4031 commit cf871d9
Show file tree
Hide file tree
Showing 11 changed files with 8,767 additions and 20,429 deletions.
9 changes: 4 additions & 5 deletions .github/workflows/integration_tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,8 @@ name: Integration tests

on: [push, pull_request]


jobs:
test-saucelabs:
test:
name: Build & test
runs-on: ubuntu-latest

Expand All @@ -22,12 +21,12 @@ jobs:
cache: 'npm'

- name: Install dependencies
run: npm ci
run: |
npm install yarn
yarn install
- name: Build
run: npm run build

- name: "Run integration tests"
run: npm run wdio:test
env:
SAUCE_ENABLED: "false" # ensure we don't use Sauce Labs
49 changes: 0 additions & 49 deletions .github/workflows/saucelabs.yaml

This file was deleted.

4 changes: 3 additions & 1 deletion .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,9 @@ jobs:
cache: 'npm'

- name: Install dependencies
run: npm ci
run: |
npm install yarn
yarn install
- name: Build
run: npm run build
Expand Down
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,5 +5,6 @@
/coverage
/.idea
*.log
/screenshots
/screenshots/*
!/screenshots/.keep
.env
22 changes: 5 additions & 17 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,15 @@ Contributions welcome, please raise a pull request.

## Requirements

You will need a recent version of Node and npm installed:
You will need a recent version of Node, npm and yarn installed:

```bash
$ node -v
v18.15.0
18.15.0
$ npm -v
v9.5.0
9.5.0
$ yarn -v
1.22.19
```

If you want to run the selenium tests, you will also need a local copy of the Java Development Kit:
Expand Down Expand Up @@ -74,20 +76,6 @@ To run the integration tests locally with Chrome (specified in [wdio.config.js](
npm run wdio
```

To run the integration tests on [Sauce Labs](https://saucelabs.com/), create a `.env` file with the following:

```bash
SAUCE_ENABLED="true"
SAUCE_USERNAME="XXXXXXXX"
SAUCE_ACCESS_KEY="YYYYYYYY"
```

And run the same command:

```bash
npm run wdio
```

Failed integration tests should output screenshots to the `./screenshots/` folder.

To build the project for distribution:
Expand Down
Loading

0 comments on commit cf871d9

Please sign in to comment.