Skip to content

Commit

Permalink
fix: run spec tests with testutils (#3017)
Browse files Browse the repository at this point in the history
BREAKING CHANGE: drop support for node v16
  • Loading branch information
UziTech committed Nov 11, 2023
1 parent 86e0eb5 commit 014d4e6
Show file tree
Hide file tree
Showing 23 changed files with 1,820 additions and 2,141 deletions.
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

1 comment on commit 014d4e6

@vercel
Copy link

@vercel vercel bot commented on 014d4e6 Nov 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.