-
Notifications
You must be signed in to change notification settings - Fork 8
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 relay test] Update to #178 #184
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RelayerOption parameter
- Fixed helpers for localnet regenesis - Added an application & supplier to the genesis file - Initializing appMap & supplierMap in E2E tests - Add support for the app's codec (for unmarshaling responses) in E2E tests - Adding a placeholder for `e2e/tests/relay.feature` --- Co-authored-by: harry <[email protected]>
* refactor: `MapFn`s receive context arg * feat: add `MapExpand` observable operator * refactor: `RelayerSessionsManager` to be more reactive * chore: add godoc comment * chore: review feedback improvements * trigger CI
Co-authored-by: Daniel Olshansky <[email protected]>
Co-authored-by: Daniel Olshansky <[email protected]>
* relayer/cli: fix: supplier client test chore: add long command description chore: review feedback improvements chore: review feedback improvements fix: sessiontree store path check fix: sessiontree store path check refactor: separate tx and query client contexts 🙄 refactor: relayminer depinject helpers & godoc comments on all constructors chore: review feedback improvements chore: update anvil service port in make targets
…-update * merge/e2e_test/relay_x_relayer_cli: fix: supplier client test chore: add long command description chore: review feedback improvements chore: review feedback improvements fix: sessiontree store path check fix: sessiontree store path check refactor: separate tx and query client contexts 🙄 refactor: relayminer depinject helpers & godoc comments on all constructors chore: review feedback improvements chore: update anvil service port in make targets
Olshansk
approved these changes
Nov 16, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A minor NIT and requires a merge with main but approving pre-emptively!
It's broken on main as is, so we'll fix things afterwards.
* relayer/cli: More tiny comment updates Added a couple more comments Update some comments and TODOs Update the names and references to queryNode/sequencerNode/fullNode etc Update pkg/relayer/cmd/cmd.go [Test] Updating `relay.feature` to run curl command to enable E2E Relay Test (#178) Updated comments for post 177+179 work for okdas Update OpenAPI spec Update .gitignore chore: update comment chore: move shared dependency setup logic to shared pkg chore: cleanup flags and dependencies for appgateserver cmd [Supplier] chore: improve supplier not found error message (#183) [CI] Integrate E2E tests with GitHub CI (#152)
…-update * merge/e2e_test/relay_x_relayer_cli: [LocalNet] Run Relayer and AppGateServer (#179) [Relay] E2E Relay Gaps (#177) More tiny comment updates Added a couple more comments Update some comments and TODOs Update the names and references to queryNode/sequencerNode/fullNode etc Update pkg/relayer/cmd/cmd.go [Test] Updating `relay.feature` to run curl command to enable E2E Relay Test (#178) Updated comments for post 177+179 work for okdas Update OpenAPI spec Update .gitignore chore: update comment chore: move shared dependency setup logic to shared pkg chore: cleanup flags and dependencies for appgateserver cmd [Supplier] chore: improve supplier not found error message (#183) [CI] Integrate E2E tests with GitHub CI (#152)
bryanchriswhite
changed the base branch from
merge/e2e_test/relay_x_relayer_cli
to
main
November 16, 2023 10:01
Co-authored-by: Daniel Olshansky <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Human Summary
This PR is based on a branch that merges #178,
main
, and #177 and builds on top to get an E2E relay test passing (and failing) for the right reasons.-features-path
flag to the E2E go test such that we can select (via glob pattern) which features we want to run on invocation.TheApplicationReceivesASuccessfulRelayResponseSignedBy
.To run the relay test (outside of tilt):
AI Summary
Summary generated by Reviewpad on 14 Nov 23 16:09 UTC
This pull request includes the following changes:
flag
package importfeaturesPathFlag
andkeyRingFlag
variablesfeatures-path
flag inTestMain
functionTheApplicationSendsTheSupplierARequestForServiceWithData
function toTheApplicationSendsTheSupplierARequestForServiceWithData
and added code commentsgetRelayKey
functionTheApplicationReceivesASuccessfulRelayResponseSignedBy
functionrunCurlPostCmd
function inpocketdBin
structErrAppGateEmptyRelayResponse
constantverifyResponse
function inappGateServer
structVerifyRelayRequest
function inrelayerProxy
structIssue
Relates to:
relay.feature
to run curl command to enable E2E Relay Test #178It would be really useful to have the E2E relay test working already such that we can quickly and easily tell if the integration has regressed to a broken state. This should close that loop.
Type of change
Select one or more:
Testing
make go_develop_and_test
Sanity Checklist