Skip to content

Commit 0041c2b

Browse files
committed
ci(build): tolerate base-branch test failures in Coverage Base job
Merge-order races can leave the base branch with a pre-existing failing test unrelated to the current PR (e.g. tronprotocol#6803 vs tronprotocol#6806 ordering). The Coverage Base job only needs jacoco XML for coverage diffing, so a stale test failure must not block the whole job. Add continue-on-error: true to the Build (base) and Test with RocksDB engine (base) steps, and relax if-no-files-found from error to warn so that a compile-level failure degrades gracefully instead of producing a misleading second failure.
1 parent cd5e116 commit 0041c2b

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.github/workflows/pr-build.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -260,9 +260,15 @@ jobs:
260260
coverage-base-x86_64-gradle-
261261
262262
- name: Build (base)
263+
# Test failures on the base branch are tolerated: merge-order races can
264+
# leave the base with a pre-existing failing test that is unrelated to
265+
# this PR. The only output we need from this job is the jacoco XML for
266+
# coverage diffing, so we must not let a stale test failure block it.
267+
continue-on-error: true
263268
run: ./gradlew clean build --no-daemon --no-build-cache
264269

265270
- name: Test with RocksDB engine (base)
271+
continue-on-error: true
266272
run: ./gradlew :framework:testWithRocksDb --no-daemon --no-build-cache
267273

268274
- name: Generate module coverage reports (base)
@@ -274,7 +280,7 @@ jobs:
274280
name: jacoco-coverage-base
275281
path: |
276282
**/build/reports/jacoco/test/jacocoTestReport.xml
277-
if-no-files-found: error
283+
if-no-files-found: warn
278284

279285
coverage-gate:
280286
name: Coverage Gate

0 commit comments

Comments
 (0)