Skip to content

chore: skip migrated PP bats tests for op-pessimistic CI leg#260

Open
arnaubennassar wants to merge 1 commit into
mainfrom
feat/skip-migrated-pp-tests
Open

chore: skip migrated PP bats tests for op-pessimistic CI leg#260
arnaubennassar wants to merge 1 commit into
mainfrom
feat/skip-migrated-pp-tests

Conversation

@arnaubennassar

Copy link
Copy Markdown

Summary

  • Gates the bats test files that have been ported to Go in aggkit/test/e2e so they no longer run on the op-pessimistic kurtosis leg, preventing duplicate coverage.
  • The FEP/op-succinct leg still runs them (FEP migration is out of scope).
  • Keeps optimistic-mode.bats ungated (not migrated).
  • Updates TESTSINVENTORY.md with migration status.

Migrated tests (now skipped for test-single-l2-network-op-pessimistic)

Bats file Go equivalent Tracked by
bridge-e2e.bats TestBridgeCore agglayer/aggkit#1524
e2e-pp.bats TestCertificateSettlement agglayer/aggkit#1524
bridge-sovereign-chain-e2e.bats TestSovereignChain agglayer/aggkit#1524
bridge-e2e-nightly.bats TestBridgeNightly agglayer/aggkit#1524
internal-claims.bats TestInternalClaims agglayer/aggkit#1524
claim-reetrancy.bats TestClaimReentrancy agglayer/aggkit#1524
aggsender-committee-updates.bats TestCommitteeUpdates agglayer/aggkit#1524

Test plan

  • Confirm op-pessimistic CI leg shows zero migrated bats tests running
  • Confirm op-succinct CI leg still runs the migrated bats (expected — not gated)
  • Confirm optimistic-mode.bats still runs for all legs

🤖 Generated with Claude Code

The following bats test files have been ported to Go in aggkit/test/e2e (PR
feat/migrate-e2e, issue agglayer/aggkit#1524) and are now gated by the new
test-go-e2e.yml workflow. They should no longer run on the op-pessimistic
kurtosis leg to avoid running the same coverage twice. The FEP/op-succinct
leg still runs them since FEP migration is out of scope.

Migrated tests now skipped for test-name==test-single-l2-network-op-pessimistic:
- bridge-e2e.bats       -> TestBridgeCore
- e2e-pp.bats           -> TestCertificateSettlement
- bridge-sovereign-chain-e2e.bats -> TestSovereignChain
- bridge-e2e-nightly.bats -> TestBridgeNightly
- internal-claims.bats  -> TestInternalClaims
- claim-reetrancy.bats  -> TestClaimReentrancy
- aggsender-committee-updates.bats -> TestCommitteeUpdates

optimistic-mode.bats is NOT gated (not migrated).
TESTSINVENTORY.md updated to note migration status.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant