Skip to content

chore: fix dispatch-revert#159

Merged
stevieraykatz merged 1 commit into
mainfrom
fix-dispatch-revert
Jun 10, 2026
Merged

chore: fix dispatch-revert#159
stevieraykatz merged 1 commit into
mainfrom
fix-dispatch-revert

Conversation

@stevieraykatz

@stevieraykatz stevieraykatz commented Jun 10, 2026

Copy link
Copy Markdown
Member

Fixes an issue caught in CI here. Basically we were asserting the wrong revert reason; malformed abi will revert for its own explicit error before the inactivity error.

@github-actions

Copy link
Copy Markdown

Interface Coverage

✅ All interface functions have test coverage.

@github-actions

Copy link
Copy Markdown

📊 Forge Coverage (src/lib/)

🟡 ≥95% across all metrics — some metrics below 99%.

File Lines Stmts Branches Funcs
🟢 B20FactoryLib.sol 100.00% 100.00% 100.00% 100.00%
🔴 test/lib/ForceFeeder.sol 0.00% 0.00% 100.00% 0.00%
🔴 test/lib/PrecompileProbe.sol 0.00% 0.00% 0.00% 0.00%
🟢 MockActivationRegistry.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockActivationRegistryStorage.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockB20.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockB20Asset.sol 100.00% 100.00% 100.00% 100.00%
🟡 MockB20Factory.sol 98.96% 99.10% 100.00% 100.00%
🟢 MockB20Stablecoin.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockB20Storage.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockPolicyRegistry.sol 100.00% 100.00% 100.00% 100.00%
🟢 MockPolicyRegistryStorage.sol 100.00% 100.00% 100.00% 100.00%
Total 96.68% 97.22% 98.39% 96.89%

Full report: download artifact. To browse locally: make coverage (runs forge coverage + genhtml + opens the HTML report).

@github-actions

Copy link
Copy Markdown

✅ Fork tests: all 613 passed

base/base is fully in sync with the base-std spec.

@stevieraykatz stevieraykatz merged commit 13781d0 into main Jun 10, 2026
10 checks passed
@stevieraykatz stevieraykatz deleted the fix-dispatch-revert branch June 10, 2026 21:06
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.

2 participants