Skip to content

chore: remove test bloat#282

Merged
jackchuma merged 25 commits into
mainfrom
jack/cleanup
May 14, 2026
Merged

chore: remove test bloat#282
jackchuma merged 25 commits into
mainfrom
jack/cleanup

Conversation

@jackchuma

@jackchuma jackchuma commented May 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes unused test/support surface: Kontrol proofs/scripts/profile, Slither config/db, solhint config, Blueprint library/tests, BenchmarkTest, MIPS64 test, CommonTest, MockERC20, SimpleStorage, MockSystemConfig, MockReceiver, ByteUtils, DisputeGames, and most invariant-only test files.
  • Consolidates nested and double-nested multisig coverage into test/universal/MultisigScript.t.sol and inlines the MockReceiver fixture into test/utils/CBMulticall.t.sol.
  • Moves deploy assertions from test/setup/StandardSystemAssertions.sol to test/deploy/SystemDeployAssertions.sol, renames the expected-state struct, and removes unused override plumbing.
  • Keeps targeted coverage by adding a Jovian L1Block gas benchmark, a versioned nonce round-trip fuzz test, and additional SystemConfig gas-limit assertions.
  • Removes Kontrol, Slither, and Solhint wiring from project config, dependency setup, ignore patterns, and validation exclusions.
@cb-heimdall

cb-heimdall commented May 13, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1
@jackchuma jackchuma changed the title chore: cleanup chore: remove test bloat May 14, 2026
@jackchuma jackchuma marked this pull request as ready for review May 14, 2026 13:35
@jackchuma jackchuma merged commit 49d890b into main May 14, 2026
7 checks passed
@jackchuma jackchuma deleted the jack/cleanup branch May 14, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants