tests: fix finality tests, run CI tests on pull requests

wormhole-foundation · Aug 27, 2024 · 08582bd · 08582bd
1 parent 9abeccd
commit 08582bd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -1,5 +1,10 @@
 name: Node.js CI
-on: [push]
+on:
+ workflow_dispatch:
+ pull_request:
+ push:
+ branches:
+ - main
 
 concurrency:
  group: ${{ github.workflow }}-${{ github.ref }}

diff --git a/core/base/__tests__/finality.ts b/core/base/__tests__/finality.ts
@@ -3,11 +3,11 @@ import {
  ConsistencyLevels,
  finalityThreshold,
  consistencyLevelToBlock,
-} from '../src/constants/finality.js';
+} from "../src/constants/finality.js";
 
 describe("Finality tests", function () {
  test("Receive expected number of rounds", () => {
- expect(finalityThreshold("Ethereum")).toEqual(64);
+ expect(finalityThreshold("Ethereum")).toEqual(96);
  expect(finalityThreshold("Algorand")).toEqual(0);
  expect(finalityThreshold("Solana")).toEqual(32);
  });
@@ -41,7 +41,7 @@ describe("Finality tests", function () {
  test("Estimates rounds from finalized consistency level", () => {
  // 100 + (# final rounds)
  expect(consistencyLevelToBlock("Ethereum", ConsistencyLevels.Finalized, fromBlock)).toEqual(
- fromBlock + 64n,
+ fromBlock + 96n,
  );
  expect(consistencyLevelToBlock("Solana", ConsistencyLevels.Finalized, fromBlock)).toEqual(
  fromBlock + 32n,