Skip to content

Commit

Permalink
feat(repo): adding conformance test workflow (#126)
Browse files Browse the repository at this point in the history
Adds the conformance test - node didn't want it in the upstream repo so
it has been offloaded to celestiaorg/node-api-conformance-test.

This will likely fail at first, will fix
  • Loading branch information
distractedm1nd authored Jun 23, 2024
1 parent 51d10a3 commit 7b13823
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 10 deletions.
11 changes: 5 additions & 6 deletions .github/workflows/ci_release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,10 @@ on:
- major

jobs:
# TODO: Uncomment once test is available in released node.
#api-equivalence:
#uses: ./github/workflows/test-equivalence.yml
#with:
#GO_VERSION: "1.22"
api-equivalence:
uses: ./.github/workflows/test-equivalence.yml
with:
GO_VERSION: "1.22"

lint:
uses: ./.github/workflows/lint.yml
Expand All @@ -42,7 +41,7 @@ jobs:

# Make a release if this is a manually trigger job, i.e. workflow_dispatch
release:
needs: [lint, test]
needs: [lint, test, api-equivalence]
runs-on: ubuntu-latest
if: ${{ github.event_name == 'workflow_dispatch' }}
permissions: "write-all"
Expand Down
7 changes: 3 additions & 4 deletions .github/workflows/test-equivalence.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,18 +10,17 @@ on:
required: true

jobs:
test-equivalence:
api-equivalence:
name: Test API Equivalence
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4
with:
repository: celestiaorg/celestia-node
ref: v0.13.4
repository: celestiaorg/node-api-conformance-test
- uses: actions/setup-go@v5
with:
go-version: ${{ inputs.GO_VERSION }}

- name: Run Structs Equivalence Test
run: cd celestia-node && go test -v -tags=conformance ./api -run TestAPIEquivalence
run: go test

0 comments on commit 7b13823

Please sign in to comment.