Skip to content

Commit

Permalink
fix: add node20 support (#32)
Browse files Browse the repository at this point in the history
* fix: add node 20 support

* chore: update ci

* chore: increase test timeout to 5 minutes
  • Loading branch information
nghaninn authored Nov 11, 2024
1 parent 4e4b2f8 commit 1caa048
Show file tree
Hide file tree
Showing 7 changed files with 22 additions and 8 deletions.
18 changes: 12 additions & 6 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,21 +10,27 @@ jobs:
run-tests:
name: Run Tests
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x, 20.x]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 18.x
node-version: ${{ matrix.node-version }}
- run: npm ci
- run: npm run test:ci

test-build:
name: Test Build
runs-on: ubuntu-latest
strategy:
matrix:
node-version: [18.x, 20.x]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 18.x
node-version: ${{ matrix.node-version }}
- run: npm ci
- run: npm run build
2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,6 @@
},
"snyk": true,
"engines": {
"node": "18.x"
"node": ">=18.x"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ const options = {
}),
};

jest.setTimeout(300_000); // 300 seconds

describe("test", () => {
describe("v2", () => {
it("should return true for documents using token registry", async () => {
Expand Down
2 changes: 2 additions & 0 deletions src/verifiers/issuerIdentity/did/didIdentityProof.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ const options = {
}),
};

jest.setTimeout(300_000); // 300 seconds

describe("skip", () => {
it("should return skip message", async () => {
const message = await openAttestationDidIdentityProof.skip(undefined as any, undefined as any);
Expand Down
2 changes: 2 additions & 0 deletions src/verify.v2.integration.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ const v2DnsDidSignedRevocationStoreButRevoked = v2DnsDidSignedRevocationStoreBut
const verifyHomestead = verify;
const verifySepolia = verificationBuilder(openAttestationVerifiers, { network: "sepolia" });

jest.setTimeout(300_000); // 300 seconds

describe("verify(integration)", () => {
let defaultEnvironment: NodeJS.ProcessEnv;
beforeEach(() => {
Expand Down
2 changes: 2 additions & 0 deletions src/verify.v3.integration.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,8 @@ const verifySepolia = verificationBuilder([...openAttestationVerifiers, openAtte
network: "sepolia",
});

jest.setTimeout(300_000); // 300 seconds

describe("verify v3(integration)", () => {
beforeEach(() => {
jest.resetModules();
Expand Down

0 comments on commit 1caa048

Please sign in to comment.