Test suite
The suite is a deep integration harness, not a focused unit spec for the factory. It uses mainnet forks to exercise full deposit/withdraw/harvest flows. That proves the happy path of addFarmsWhitelist, updateVaultConfig, and createVault works end to end, but it never stresses the factory’s guardrails. Tests for the following are missing:
Rejecting a farm that is not whitelisted
Refusing a profit token that is not whitelisted
We recommend adding tests that stress the factory in isolation.