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

feat: grpc client for tap aggregator #583

Merged
merged 24 commits into from
Jan 24, 2025
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
d601187
feat: remove jsonrpc client in favor of grpc client
pedrohba1 Jan 7, 2025
78aa4ba
refactor: adjust endpoint in test
pedrohba1 Jan 16, 2025
20052b9
chore: cargo fmt
pedrohba1 Jan 16, 2025
b6ea2a7
refactor: remove commented code and unnecessary stop of handler in test
pedrohba1 Jan 21, 2025
c86e1e0
test: handle error from failed connection
pedrohba1 Jan 22, 2025
d8154d3
feat: log out endpoint in case of failed connection attempt
pedrohba1 Jan 22, 2025
6e426f6
ci: add TapAggregator server mock to minimize changes to tests
pedrohba1 Jan 22, 2025
910f25f
ci: fix line indent
pedrohba1 Jan 22, 2025
f680606
ci: force ri rerun
pedrohba1 Jan 22, 2025
ea023cf
refactor: cargo fmt
pedrohba1 Jan 22, 2025
8c2bdb5
ci: move mock server start to test-and-coverage
pedrohba1 Jan 22, 2025
34d2947
test: test aggregator server helper function
pedrohba1 Jan 24, 2025
36ecace
test: instantiate server instead of using mock
pedrohba1 Jan 24, 2025
90e2e3f
ci: remove unnecessary step
pedrohba1 Jan 24, 2025
990340a
refactor: correct typo
pedrohba1 Jan 24, 2025
eb13c2b
refactor: correct typo
pedrohba1 Jan 24, 2025
54fcc58
refactor: replace space
pedrohba1 Jan 24, 2025
bcec5b5
refactor: remove unnecessary message
pedrohba1 Jan 24, 2025
d99ca12
test: fix hanging test
gusinacio Jan 24, 2025
30be4a0
refactor: change expect with context
gusinacio Jan 24, 2025
ce3206f
test: use get_grpc_url() to spawn a grpc
gusinacio Jan 24, 2025
26ac63b
test: use server_url directly
gusinacio Jan 24, 2025
87dab30
test: update all tests to use get_grpc_url
gusinacio Jan 24, 2025
5decbd1
chore: add comment about compression
gusinacio Jan 24, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
ci: force ri rerun
pedrohba1 authored and gusinacio committed Jan 24, 2025
commit f680606468ba3391e5fd33cb4e7929166dafbfac
Original file line number Diff line number Diff line change
@@ -54,9 +54,10 @@ pub async fn mock_tap_aggregator_server() -> Result<oneshot::Sender<()>, Box<dyn
#[tokio::main]
async fn main() -> Result<(), Box<dyn std::error::Error>> {
// Start the mock server

let shutdown_tx = mock_tap_aggregator_server().await?;

println!("Mock server running. Press Ctrl+C to stop.");
println!("Mock server running. Press Ctrl+C to stop.");

// Wait until the user decides to terminate the process
tokio::signal::ctrl_c().await.expect("Failed to listen for Ctrl+C");