Skip to content

fixed:import path#1

Open
0xWeakSheep wants to merge 2 commits intomonad-developers:mainfrom
0xWeakSheep:main
Open

fixed:import path#1
0xWeakSheep wants to merge 2 commits intomonad-developers:mainfrom
0xWeakSheep:main

Conversation

@0xWeakSheep
Copy link

  • Fixed the import path for ExchangeStorage in contract/test/Day4PriceUpdate.t.sol so Foundry resolves it correctly.

    • Renamed a duplicated local variable in contract/test/PositionMargin.t.sol to avoid shadowing.
    • Replaced fail("...") with assertTrue(false, "...") in contract/test/PositionMargin.t.sol to match the current forge-std
      API.
    • Removed the redundant base constructor call in contract/test/utils/MonadPerpExchangeHarness.sol (constructor did nothing;
      base ctor runs automatically).

    All changes are about test setup/compile compatibility, not main contract logic.

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