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.

Sign up for GitHub

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

Jump to bottom

Add E2E test for indexing from the chain #122

Merged
merged 1 commit into from
Aug 29, 2024
Merged

Add E2E test for indexing from the chain #122

merged 1 commit into from
Aug 29, 2024

Conversation

neekolas
Copy link
Contributor

@neekolas neekolas commented Aug 29, 2024 •
edited
Loading

tl;dr

  • Adds a TransactionSigner interface
  • Adds a GroupMessagePublisher interface
  • Adds the first E2E test that actually publishes to the local blockchain and stores data in the DB
  • Renames the PrivateKeyString to SigningKeyString to differentiate between the message signer and the private key used to sign blockchain transactions
  • Adds sqlc to the tools.go to lock in the version

This was referenced Aug 29, 2024
Merged
Merged
@neekolas Graphite App
Copy link
Contributor Author

neekolas commented Aug 29, 2024 •
edited
Loading

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

Join @neekolas and the rest of your teammates on Graphite Graphite

@neekolas neekolas force-pushed the 08-28-add_e2e_test_for_indexing_from_the_chain branch 2 times, most recently from b891257 to 840d089 Compare August 29, 2024 04:59
@neekolas neekolas requested a review from richardhuaaa August 29, 2024 05:04
@neekolas neekolas marked this pull request as ready for review August 29, 2024 05:04
@neekolas neekolas force-pushed the 08-28-rename_testing_to_testutils branch from 3eca2e7 to 3dca8b7 Compare August 29, 2024 05:06
@neekolas neekolas force-pushed the 08-28-add_e2e_test_for_indexing_from_the_chain branch from 840d089 to 305811d Compare August 29, 2024 05:06
@neekolas neekolas mentioned this pull request Aug 29, 2024
Merged
richardhuaaa
richardhuaaa approved these changes Aug 29, 2024
View reviewed changes
Copy link
Contributor

@richardhuaaa richardhuaaa left a comment

Choose a reason for hiding this comment

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

Huge

pkg/blockchain/interface.go Show resolved Hide resolved
pkg/config/options.go Outdated Show resolved Hide resolved
pkg/indexer/storer/groupMessage.go Show resolved Hide resolved
@neekolas neekolas force-pushed the 08-28-add_e2e_test_for_indexing_from_the_chain branch from 305811d to 6d0a129 Compare August 29, 2024 22:56
@neekolas Graphite App
Copy link
Contributor Author

neekolas commented Aug 29, 2024 •
edited
Loading

Merge activity

  • Aug 29, 4:23 PM PDT: @neekolas started a stack merge that includes this pull request via Graphite.
  • Aug 29, 4:29 PM PDT: Graphite rebased this pull request as part of a merge.
  • Aug 29, 4:31 PM PDT: @neekolas merged this pull request with Graphite.

@neekolas neekolas changed the base branch from 08-28-rename_testing_to_testutils to graphite-base/122 August 29, 2024 23:24
@neekolas neekolas changed the base branch from graphite-base/122 to main August 29, 2024 23:27
@neekolas neekolas force-pushed the 08-28-add_e2e_test_for_indexing_from_the_chain branch from 6d0a129 to aa61f40 Compare August 29, 2024 23:28
@neekolas neekolas merged commit b8ab087 into main Aug 29, 2024
5 checks passed
@neekolas neekolas deleted the 08-28-add_e2e_test_for_indexing_from_the_chain branch August 29, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richardhuaaa richardhuaaa richardhuaaa approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@neekolas @richardhuaaa