Skip to content

fix: run spec tests with testutils #3017

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

Merged
merged 19 commits into from
Nov 11, 2023
1 change: 1 addition & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
lib
*.min.js
public
78 changes: 0 additions & 78 deletions .github/workflows/codeql-analysis.yml

This file was deleted.

37 changes: 12 additions & 25 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,11 @@ permissions:
contents: read

jobs:
Test:
UnitTests:
strategy:
matrix:
# lowest verison here should also be in `engines` field
node_version: [16, 18, 'lts/*', '*']
node_version: [18, "lts/*", "*"]
runs-on: ubuntu-latest
steps:
- name: Checkout Code
Expand All @@ -25,50 +25,37 @@ jobs:
check-latest: true
- name: Install Dependencies
run: npm ci
- name: Build 🗜️
run: npm run build
- name: Run Unit Tests 👩🏽‍💻
run: npm run test:unit
- name: Run Spec Tests 👩🏽‍💻
run: npm run test:specs
- name: Run UMD Tests 👩🏽‍💻
run: npm run test:umd
- name: Run Types Tests 👩🏽‍💻
run: npm run test:types

Lint:
runs-on: ubuntu-latest
steps:
- name: Checkout Code
uses: actions/checkout@v4
- name: Install Node
uses: actions/setup-node@v4
with:
node-version: 'lts/*'
- name: Install Dependencies
run: npm ci
- name: Lint ✨
run: npm run test:lint

Build:
name: Build and Test Types
OtherTests:
runs-on: ubuntu-latest
steps:
- name: Checkout Code
uses: actions/checkout@v4
- name: Install Node
uses: actions/setup-node@v4
with:
node-version: 'lts/*'
node-version: "lts/*"
- name: Install Dependencies
run: npm ci
- name: Build 🗜️
run: npm run build
- name: Run UMD Tests 👩🏽‍💻
run: npm run test:umd
- name: Run Types Tests 👩🏽‍💻
run: npm run test:types
- name: Lint ✨
run: npm run test:lint

Release:
permissions:
contents: write
needs: [Test, Lint, Build]
needs: [UnitTests, OtherTests]
if: |
github.ref == 'refs/heads/master' &&
github.event.repository.fork == false
Expand All @@ -79,7 +66,7 @@ jobs:
- name: Install Node
uses: actions/setup-node@v4
with:
node-version: 'lts/*'
node-version: "lts/*"
- name: Install Dependencies
run: npm ci
- name: Build 🗜️
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
.DS_Store
.vercel
.vscode
node_modules/
test/compiled_tests
public
Expand Down
9 changes: 0 additions & 9 deletions .vscode/settings.json

This file was deleted.

Loading