Skip to content

Commit

Permalink
Move unit tests to vitest. (#1224)
Browse files Browse the repository at this point in the history
* Add vitest.

* Changes made to tests in order to convert them easier.

Add script that converts tests from jasmine to vitest.

Some changes created by linting fix.

New tests for vitest.

Fix test.

Fix lint error.

New batch updated.

Undo the changes to the test and add it to the browser suite.

Run browser in headless mode.

Improve test speed.

New batch.

New batch.

Last batch converted.

Fix some flaky tests.

* Add playwright dependencies.

* Add coverage options for vitest.

Change properties order.

* Just run everything in the browser.

Remove workspace config.

Rename tests suite.

* Move scripts tests to vitest.

* Remove karma and jasmine.

Remove karma config.

* Remove karma tests.

* Move vitest unit tests to the old karma unit tests.

* Update tests to match latest changes from main.

* Remove saucelabs references.

* Remove duplicate batch of tests.

* Fix tests messed by the convert script.

* Remove mess.

* Remove unnecessary dependencies.
  • Loading branch information
dompuiu authored Jan 10, 2025
1 parent 02bd418 commit 4e1c9a6
Show file tree
Hide file tree
Showing 378 changed files with 15,390 additions and 14,599 deletions.
60 changes: 1 addition & 59 deletions .github/workflows/dev.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,6 @@ name: Alloy Dev
on:
pull_request:
workflow_dispatch:
env:
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
SAUCE_JOB: "Alloy Dev Workflow"
SAUCE_CAPABILITIES_OVERRIDES_PATH: "sauceLabsCapabilities.json"
EDGE_BASE_PATH: ee-pre-prd
ALLOY_ENV: int
SAUCE_TUNNEL_ID: github-action-tunnel

jobs:
linting:
Expand Down Expand Up @@ -49,59 +41,9 @@ jobs:
- name: Install dependencies
if: steps.npm-cache.outputs.cache-hit != 'true'
run: npm ci
- name: Build
run: npm run test:functional:build:int
- uses: saucelabs/sauce-connect-action@v2
with:
username: ${{ secrets.SAUCE_USERNAME }}
accessKey: ${{ secrets.SAUCE_ACCESS_KEY }}
- name: Run Unit Test
run: npx karma start karma.saucelabs.conf.cjs --single-run
run: npm run test
env:
BUILD_NUMBER: $GITHUB_RUN_NUMBER
BUILD_ID: $GITHUB_RUN_ID
JOB_NUMBER: $GITHUB_JOB

alloy-int-e2e:
name: "Dev Functional Test - Saucectl"
runs-on: ubuntu-latest
needs: unit-test
timeout-minutes: 60
steps:
- name: Set up Node.js version
uses: actions/setup-node@v4
with:
node-version: 22

- uses: actions/checkout@v4

- name: Clear cache
uses: actions/cache@v4
id: npm-cache
with:
path: "**/node_modules"
key: ${{ runner.os }}-modules-${{ hashFiles('**/package-lock.json') }}-${{ secrets.NPM_CACHE_VERSION }} # increment NPM_CACHE_VERSION secret to force cache reset

- name: Store Alloy version into env
uses: nyaa8/package-version@v1

- name: Install dependencies
if: steps.npm-cache.outputs.cache-hit != 'true'
run: npm ci

- name: Build
run: npm run test:functional:build:int
env:
ALLOY_PROD_VERSION: ${{ env.PACKAGE_VERSION }}

- uses: saucelabs/[email protected]
env:
GITHUB_TOKEN: ${{ github.token }}
with:
sauce-username: ${{ secrets.SAUCE_USERNAME }}
sauce-access-key: ${{ secrets.SAUCE_ACCESS_KEY }}
env: |
ALLOY_PROD_VERSION=${{ env.PACKAGE_VERSION }}
NPM_PACKAGE_VERSION=${{ env.PACKAGE_VERSION }}
ALLOY_ENV=int
config-file: ./.sauce/dev.yml
3 changes: 0 additions & 3 deletions .github/workflows/publish-to-npm.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
name: Publish To NPM
on: workflow_dispatch
env:
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}

jobs:
publish-to-npm:
Expand Down
1 change: 0 additions & 1 deletion .husky/pre-commit
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
npm run checkthattestfilesexist
STAGED_ONLY=true npm run add-license
npx lint-staged
npm run test:unit
12 changes: 0 additions & 12 deletions .sauceignore

This file was deleted.

87 changes: 0 additions & 87 deletions karma.conf.cjs

This file was deleted.

74 changes: 0 additions & 74 deletions karma.saucelabs.conf.cjs

This file was deleted.

Loading

0 comments on commit 4e1c9a6

Please sign in to comment.