Skip to content

Commit

Permalink
feat/1731 code coverage in Codecov
Browse files Browse the repository at this point in the history
  • Loading branch information
count-sum committed Sep 11, 2024
1 parent a001342 commit 456b0d7
Show file tree
Hide file tree
Showing 4 changed files with 27 additions and 3 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/coordinator-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,5 +94,10 @@ jobs:
repo-token: ${{ secrets.GITHUB_TOKEN }}
message-path: |
${{ github.workspace }}/build/reports/jacoco/jacocoRootReport/coverageDeltaReports/DeltaSummary.md
- name: Upload coverage to Codecov
uses: codecov/codecov-action@v4
with:
files: ${{ github.workspace }}/build/reports/jacoco/jacocoRootReport/coverageReports/*.md
flags: kotlin
fail_ci_if_error: true
token: ${{ secrets.CODECOV_TOKEN }}
11 changes: 11 additions & 0 deletions .github/workflows/run-smc-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,17 @@ jobs:
- name: Run smart contracts tests and generate coverage report
run: pnpm -F contracts run coverage

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v4
with:
fail_ci_if_error: true
files: ./contracts/coverage/coverage-final.json
flags: hardhat
name: codecov-contracts
verbose: true
env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

solidity-format-check:
runs-on: ubuntu-latest
name: Solidity format check
Expand Down
8 changes: 8 additions & 0 deletions codecov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
comment:
layout: " diff, flags, files"
behavior: default
require_changes: false
require_base: false
require_head: true
hide_project_coverage: false

2 changes: 1 addition & 1 deletion contracts/test/LineaRollup.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1742,7 +1742,7 @@ describe("Linea Rollup contract", () => {
calldataAggregatedProof1To155.l1RollingHash,
);

// aggregatedProof1To81.aggregatedProof, // wrong proof on purpose
// aggregatedProof1To81.aggregatedProof, // wrong proof on purpose
const finalizeCall = lineaRollup
.connect(operator)
.finalizeBlocksWithProof(aggregatedProof1To81.aggregatedProof, TEST_PUBLIC_VERIFIER_INDEX, finalizationData);
Expand Down

0 comments on commit 456b0d7

Please sign in to comment.