test(audit-low): coverage sweep 2 — 15 LOW test findings in one pass

[hardyjosh]

• [A01-2] [LOW] description() always returns empty string — no test asserts the contract #51: testDescription for single-feed + multi-feed Pyth adapter initializes.
• [A05-1] [LOW] No test for InitializeRegistryFailed revert path or Deployment event #56 [A07-1] [LOW] No test for InitializeAdapterFailed in PassthroughProtocolAdapterBeaconSetDeployer #58 [A08-1] [LOW] No test for InitializeOracleFailed in PythOracleAdapterBeaconSetDeployer #59: BadImpl reverter + InitFailure tests on
  Registry/Passthrough/Pyth BeaconSetDeployer construct tests
  (mirrors the existing Morpho/MultiPyth pattern). The Deployment-event
  emission is already covered by testDeploymentEventIsIndexed in each file.
• [A06-1] [LOW] OracleUnifiedDeployer: pauseConfig argument is never varied across tests #57: testOracleUnifiedDeployerPropagatesNonEmptyPauseConfig fuzz.
• [A09-2] [LOW] MultiPythOracleAdapter: ICloneableV2 documentation initialize overload (always-reverts) untested #61: testInitializeSignatureOverloadAlwaysReverts x5 contracts.
• [A09-3] [LOW] MultiPythOracleAdapter: Initializable double-initialize check untested #62: testCannotDoubleInitialize x5 contracts.
• [A10-1] [LOW] PythOracleAdapter: NonPositivePrice and ZeroVaultSharePrice paths only tested in MultiPythOracleAdapter #64 [A10-2] [LOW] PythOracleAdapter: latestRoundData publishTime field encoding not verified #65: NonPositivePrice / ZeroVaultSharePrice / publishTime parity
  on single-feed Pyth adapter via mock to avoid fork RPC env — new
  PythOracleAdapter.latestAnswerMock.t.sol (non-fork) sibling.
• [A11-2] [LOW] MorphoProtocolAdapter: setRegistry success path doesn't verify price() now resolves through new registry #67: testSetRegistryUpdatesPriceResolution on Morpho + Passthrough.
• [A13-1] [LOW] OracleRegistry.t.sol redefines OracleSet event locally instead of importing from source #69: OracleRegistry tests use emit OracleRegistry.OracleSet/AdminSet
  in place of duplicate local event redeclarations.
• [A13-2] [LOW] OracleRegistry.setOracleBulk: empty array success path untested #70: testSetOracleBulkEmptyArrays.
• [A09-5] [LOW] MultiPythOracleAdapter: getFeeds at zero feedCount edge case is unreachable but untested #76: testGetFeedsConsistencyWithGetFeed.
• [A15-3] [LOW] LibCorporateActionsPause: no test for vault that reverts on the read paths #74: testRevertingVaultPropagates + testEoaVaultPropagatesNoCodeBehaviour
  (via InPauseWindowCaller wrapper so vm.expectRevert hooks the right depth
  on an internal-library function).
• [A10-3] [LOW] PythOracleAdapter: ProdFork tests use bare vm.expectRevert() without selector #77: ProdFork bare vm.expectRevert() sites (12 total) all selector-specific;
  L468/L471 use vm.expectRevert(bytes("")) for the abi-decode-failure case.

No source-code changes. Test count delta: 151 -> 169 (+18). nix CI green:
forge fmt, forge fmt --check, rainix-sol-static (0 results), rainix-sol-legal
(compliant), forge test --no-match-contract .

Closes #51, #56, #57, #58, #59, #61, #62, #64, #65, #67, #69, #70, #74, #76, #77.

Co-Authored-By: Claude Opus 4.7 noreply@anthropic.com

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 72f91f23-605d-4589-8180-18afd200adc3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test/audit-low-coverage-sweep-2

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[hardyjosh]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• fix(subgraph): update PythOracleAdapterInitialized event signatures for pauseConfig #234
• fix(audit-low): close 2 hazard-surface findings #233
• test(audit-low): coverage sweep 2 — 15 LOW test findings in one pass #232 👈 (View in Graphite)
• refactor(audit-low): style + refactor sweep — ~22 LOW findings #231
• fix(audit-low): code-correctness sweep — 10 LOW findings in one pass #230
• test(audit): close 10 MEDIUM test-coverage gaps in one sweep #228 : 1 other dependent PR (#229 )
• fix(morpho): require 8-decimal upstream oracle (audit #187) #224 : 2 other dependent PRs (#225 , #227 )
• refactor(interface): rename to AggregatorV2V3Interface + add getRoundData #222 : 1 other dependent PR (#223 )
• chore: remove MultiPythOracleAdapter for Euler launch #238
• docs: README refresh + integrator notes for corporate-action auto-pause (RAI-325) #26 : 1 other dependent PR (#221 )
• feat(deploy): thread CorporateActionPauseConfig through unified deployers (RAI-322) #25
• test: auto-pause revert propagation through protocol adapters (RAI-321) #24
• feat(oracle): wire corporate-action auto-pause into BasePythOracleAdapter (RAI-320) #23
• feat(lib): LibCorporateActionsPause reader helper (RAI-319) #22
• docs(spec): corporate-action-aware auto-pause semantics (RAI-318) #21
• deps: migrate from git submodules to soldeer #237
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[hardyjosh]

Deferred for Euler launch. Test additions for already-working code; not Euler-blocking.