Skip to content

feat: integration tests poll requests by RPC instead of indexer#513

Merged
ChaoticTempest merged 6 commits intodevelopfrom
phuong/feat/near-rpc-requests
Sep 9, 2025
Merged

feat: integration tests poll requests by RPC instead of indexer#513
ChaoticTempest merged 6 commits intodevelopfrom
phuong/feat/near-rpc-requests

Conversation

@ChaoticTempest
Copy link
Copy Markdown
Contributor

This makes it so that requests from near contract come in by RPC instead of indexer. indexer.rs now just polls the near contract for pending requests.

Changes

  • removes near lake indexer and lake node; uses near_workspaces::sandbox directly now.
  • removes localstack since only lake was using it.
  • removes AWS dependencies since only localstack and lake was using it.
  • removes toxiproxy since it was very intertwined with lake and basically was a requirement.

@ChaoticTempest ChaoticTempest force-pushed the phuong/feat/near-rpc-requests branch from 72340f4 to a72ad79 Compare September 9, 2025 05:18
Copy link
Copy Markdown
Contributor

@volovyks volovyks left a comment

Choose a reason for hiding this comment

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

Happy to remove all these dependencies and related code!
closing #447 in favor of this PR

#[derive(Debug, Clone, clap::Parser)]
#[group(id = "indexer_options")]
pub struct Options {
/// AWS S3 bucket name for NEAR Lake Indexer
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@auto-mausx multiple parameters become redundant, please, represent that in our docs.

@ChaoticTempest ChaoticTempest merged commit 05aec44 into develop Sep 9, 2025
4 of 5 checks passed
@ChaoticTempest ChaoticTempest deleted the phuong/feat/near-rpc-requests branch September 9, 2025 21:43
@jakmeier
Copy link
Copy Markdown
Contributor

jakmeier commented Sep 11, 2025

This is easy to miss but actually really awesome to get cleaned up! It really helps with compilation time as well as the time to run tests.

The time for running integrations tests in CI went down from 41-42 minutes to just 31-33 minutes.

These are the timing for the 3 time consuming steps:

before after
Build Chain-Signatures Node 6m 33s 4m 50s
Build Chain-Signatures Integration Tests 4m 9s 2m 52s
Test 30m 21s 22m 29s

Thanks @ChaoticTempest for getting this in!

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.

4 participants