Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

E2E test migration #226

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

E2E test migration #226

wants to merge 16 commits into from

Conversation

mfw78
Copy link
Collaborator

@mfw78 mfw78 commented Oct 7, 2024

Description

This PR is to close out the milestone of migrating tests to foundry. As all E2E tests have been migrated into the e2e-migration branch for staging purposes, this PR then merges (through use of rebase / fast-forward) these tests to the main branch.

Test Plan

  1. Verify CI/CD green and that all related tests have been migrated.

Related Issues

#130

## Description

Base test contract for forked tests in e2e tests

## Test Plan

CI

## Related Issues
## Description

Migrate nonStandardErc20 e2e test to foundry. Depends on
`FoundryTest.sol` introduced in #215

## Test Plan

CI

## Related Issues

Closes #138
## Description

Migrate the internalBalances e2e test to foundry. Depends on #215.

## Test Plan

CI

## Related Issues

Closes #137
## Description

Migrate the 0xTrade e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #131 #132
@mfw78 mfw78 requested a review from fedgiac October 7, 2024 11:26
@mfw78 mfw78 self-assigned this Oct 7, 2024
mfw78 and others added 12 commits October 7, 2024 11:44
## Description

This PR replaces llamarpc with merkle.io for the RPC as there are higher
RPS.

## Test Plan

1. Ensure CI/CD remains green for fork tests.

## Related Issues

N/A
## Description

Migrate wineOilMarket e2e test to foundry. Depends on #215 for the
`ForkedTest.t.sol`.

## Test Plan

CI

## Related Issues

Closes #143
## Description

Migrate the upgradeAuthenticator e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #142
## Description

In order to make use of cache, and to convey to testers / developers
what the assumptions of fork tests are, the fork block number is
established as a constant within the `Helper.sol`.

## Test Plan

1. Verify that CI/CD passes.

## Related Issues

N/A
## Description

Migrate the burnFees e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #134
## Description

This PR sets the `FORK_URL` environment variable to a mainnet archive
node with sufficient RPS / capacity to facilitate the fork tests.

## Test Plan

1. Verify CI/CD green.

## Related Issues

N/A
## Description

Migrate buyEth e2e test to foundry. Depends on
#215

## Test Plan

CI

## Related Issues
Closes #135
<!-- List related issues here -->
<!-- Closes #issue (optional) -->

---------

Co-authored-by: mfw78 <[email protected]>
## Description

Migrate the offchainAllowances e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #139
## Description

Migrate the smartOrder e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Issue doesn't exist
## Description

Migrate the uniswapTrade e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #141
## Description

Migrate the contractOrdersWithGnosisSafe e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #136
## Description

Migrate the deployment e2e test to foundry. Depends on
#215.

## Test Plan

CI

## Related Issues

Closes #149
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