Fix deal mock celo using mint()

[Mouradif]

dealFork was failing for mock Celo because foundry couldn't find the storage slot for some reason. This PR uses the mint() function on Celo instead

Also: fixed a variable name shadowing in a test

[cursor]

Stale comment

This PR switches CELO mock funding in SetupFork from dealFork(..., true) to explicit mint() calls, and refactors WeekendSituationTest to avoid repeated local casting of the factory.

Findings

test/integration/WeekendSituationTest.t.sol

• 🔴 Critical — L35, L42, L54: fpmmFactory is declared as address in V3IntegrationBase, but this PR treats it as IFPMMFactory. This introduces a compile-time regression (member deployedFPMMAddresses not found in address) and blocks the test suite.

Suggested fix (pick one):

// Option A: keep base state var as address and cast locally
fpmmFactory = lookupProxyOrFail("FPMMFactory");
...
IFPMMFactory factory = IFPMMFactory(fpmmFactory);
address[] memory fpmms = factory.deployedFPMMAddresses();

// Option B: change base type once (broader impact)
IFPMMFactory internal fpmmFactory;

Final verdict: 🚫 Needs changes

  

_{Sent by Cursor Automation: PR Review}

[cursor]

Stale comment

This PR switches fork CELO funding from dealFork(...) to explicit mint() on the etched MockCELO, removes local fpmmFactory shadowing in weekend integration tests, and updates foundry.lock revisions.

test/integration/WeekendSituationTest.t.sol

• 🔴 CRITICAL
• line 35 (with downstream impact at lines 42 and 54): fpmmFactory is inherited from V3IntegrationBase as address, but this change treats it as IFPMMFactory and then calls fpmmFactory.deployedFPMMAddresses(). This is a type-level regression that will fail compilation (Member "deployedFPMMAddresses" not found in address).
• Fix by either:
  1. Keeping fpmmFactory as address and using a non-shadowing local typed alias (e.g. IFPMMFactory factory = IFPMMFactory(fpmmFactory);), or
  2. Promoting the base-state variable type to IFPMMFactory consistently where it is declared and consumed.

Final verdict: 🚫 Needs changes

  

_{Sent by Cursor Automation: PR Review}

[cursor]

This PR replaces the CELO fork funding path in SetupFork to use mint() on the etched mock token and cleans up the FPMM factory access pattern in the weekend integration test. The changes are small, targeted, and aligned with the PR goal.

No findings.

Final verdict: ✅ LGTM

  

_{Sent by Cursor Automation: PR Review}

[philbow61]

For me, the SetupFork script is still failing on celo mainnet?