Skip to content

Conversation

@youngkidwarrior
Copy link
Collaborator

Why:
Provide minimal mocks to enable self-contained tests.

Test plan:

  • bunx hardhat compile
  • bunx hardhat test (mocks-related tests pass)

Copy link
Collaborator Author

youngkidwarrior commented Sep 10, 2025

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

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

@youngkidwarrior youngkidwarrior marked this pull request as ready for review September 10, 2025 00:56
Why:
Enable self‑contained tests for rewards flows without external
repos. Provide minimal mocks to exercise ERC‑4626 asset logic
and factory lookups.

Details:
- MockERC20 with approve support
- MockERC4626Vault implementing asset(), deposit/mint
- Minimal SendEarnFactory for share token discovery

Test plan:
- bunx hardhat compile
- bunx hardhat test (mocks‑related tests pass)
@youngkidwarrior youngkidwarrior changed the base branch from feat_rewards_add_fundholder_helper_to_deposit_assets_and_mint_shares_local_fork_ to graphite-base/28 September 12, 2025 11:08
@youngkidwarrior youngkidwarrior force-pushed the test_mocks_add_mockerc20_mockerc4626vault_and_mocksendearnfactory branch from a1f1115 to fb7f937 Compare September 12, 2025 11:08
@youngkidwarrior youngkidwarrior changed the base branch from graphite-base/28 to build_add_compiled_artifacts_and_cache_deployments_for_rewardsmanager September 12, 2025 11:09
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