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

L1EP missing gauntlet commands for e2e tests #298

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

sdrug
Copy link
Contributor

@sdrug sdrug commented Jun 5, 2023

Ticket - https://smartcontract-it.atlassian.net/browse/BCI-1218?atlOrigin=eyJpIjoiNWNiNjNmYjEwMmNhNDFhZThjYmRlYmQ5ODEwY2Y3OWEiLCJwIjoiaiJ9

Adding the following commands:

  • Deployment of the L1 AccessController - yarn gauntlet l1_access_controller:deploy --network=<L1_NETWORK>
  • Deployment of the MockAggregator to behave as an L1GasPriceFeed - yarn gauntlet l1_mock_aggregator:deploy --network=<L1_NETWORK>
  • Setting the desired latestAnswer on the mocked aggregator - yarn gauntlet l1_mock_aggregator:set_answer --network=<L1_NETWORK> --answer=<ANSWER> <AGGREGATOR_ADDRESS>

@sdrug sdrug temporarily deployed to integration June 5, 2023 19:03 — with GitHub Actions Inactive
@sdrug sdrug temporarily deployed to integration June 9, 2023 04:57 — with GitHub Actions Inactive
s_answer = answer;
}

function latestAnswer() public view returns (int256) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

@augustbleeds augustbleeds Sep 1, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sdrug are you waiting on review? I think this still needs to be changed. The line above has been changed

@augustbleeds
Copy link
Collaborator

Looks good, just need to change to latestRoundData() and add the commands to the er protocol README.md 👍

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