From 1794147b3c4b9585d8e22d3534d41b6d2c1e2ba8 Mon Sep 17 00:00:00 2001 From: "Carol (Nichols || Goulding)" Date: Mon, 6 Feb 2023 15:06:30 -0500 Subject: [PATCH] fix: Only test querier panic logging; router2 and ingester2 now exit on panic --- influxdb_iox/tests/end_to_end_cases/error.rs | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/influxdb_iox/tests/end_to_end_cases/error.rs b/influxdb_iox/tests/end_to_end_cases/error.rs index dd7e6b4213..bfdabb1047 100644 --- a/influxdb_iox/tests/end_to_end_cases/error.rs +++ b/influxdb_iox/tests/end_to_end_cases/error.rs @@ -14,17 +14,6 @@ pub async fn test_panic() { &mut cluster, vec![Step::Custom(Box::new(move |state: &mut StepTestState| { async move { - let router = state.cluster().router(); - assert_panic_logging(router.router_grpc_connection(), router.log_path().await) - .await; - - let ingester = state.cluster().ingester(); - assert_panic_logging( - ingester.ingester_grpc_connection(), - ingester.log_path().await, - ) - .await; - let querier = state.cluster().querier(); assert_panic_logging(querier.querier_grpc_connection(), querier.log_path().await) .await; @@ -43,7 +32,7 @@ async fn assert_panic_logging(connection: Connection, log_path: Box) { if let influxdb_iox_client::error::Error::Internal(err) = err { assert_eq!(&err.message, "This is a test panic"); } else { - panic!("wrong error type"); + panic!("wrong error type. got {err:?}"); } // check logs