Add a feature to generate testing transactions #518
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
…ocessor testing. This enables us to skip creating a pr whenever there is new transaction to test with.
yuunlimm
force-pushed
the
yuunlimm/processor-testing-import
branch
from
8d092e8
to
62d4df1
Compare
yuunlimm
force-pushed
the
yuunlimm/processor-testing-import
branch
from
62d4df1
to
797790f
Compare
larry-aptos
approved these changes
Sep 21, 2024
|
Can you link the linear ticket here? i think we have a task tracking this effort. thanks! |
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.
Description
linear task: https://linear.app/aptoslabs/issue/EI-521/export-generated-jsons-to-processors-repo-without-cargo-dependency
THe way we import new transactions when we are testing, we have to create a PR every time we generate new transaction JSONs to be used in the processor tests. This allows us to skip that step.
Previous Steps
New Steps
aptos-indexer-processor/rust/testing-transactions/json_transactions/).Test Plan
used the new txn generated from this change and tests still pass
