chore: remove too complex testing

Author: darklight3it

lambda-runtime/src/layers/api_client.rs

@@ -132,8 +132,6 @@ mod tests {
     use httpmock::prelude::*;
     use std::future;
     use tower::Service;
-    use tracing_capture::{CaptureLayer, SharedStorage};
-    use tracing_subscriber::{layer::SubscriberExt, util::SubscriberInitExt};
 
     fn make_invocation() -> LambdaInvocation {
         let (parts, _) = http::Response::new(()).into_parts();
@@ -201,53 +199,6 @@ mod tests {
         assert!(result.is_ok());
     }
 
-    #[tokio::test]
-    async fn test_non_2xx_logs_status_and_body() {
-        let storage = SharedStorage::default();
-        // Use set_default (thread-local) so we don't race with other tests that call
-        // set_global_default. #[tokio::test] uses a single-threaded executor, so the
-        // guard remains valid for the entire test body.
-        let _guard = tracing_subscriber::registry()
-            .with(CaptureLayer::new(&storage))
-            .set_default();
-
-        let server = MockServer::start();
-
-        const BODY: &str = r#"{"errorMessage":"Invoke timeout","errorType":"InvokeTimeout"}"#;
-
-        server.mock(|when, then| {
-            when.any_request();
-            then.status(410).header("content-type", "application/json").body(BODY);
-        });
-
-        let client = make_client(&server);
-        let req = http::Request::builder().uri("/some/path").body(Body::empty()).unwrap();
-        let mut svc = RuntimeApiClientService::new(mock_inner(req), client);
-        svc.call(make_invocation()).await.unwrap();
-
-        let storage = storage.lock();
-        let event = storage
-            .all_events()
-            .find(|e| {
-                e.metadata().level() == &tracing::Level::ERROR
-                    && e.value("status")
-                        .and_then(|v| v.as_debug_str())
-                        .map_or(false, |s| s.contains("410"))
-            })
-            .expect("expected an error event with status 410 to be logged");
-
-        assert!(
-            event["status"].as_debug_str().unwrap().contains("410"),
-            "expected status to contain '410', got: {:?}",
-            event["status"].as_debug_str()
-        );
-        assert!(
-            event["body"].as_debug_str().unwrap().eq(BODY),
-            "expected body to contain 'InvokeTimeout', got: {:?}",
-            event["body"].as_debug_str()
-        );
-    }
-
     #[tokio::test]
     async fn test_inner_service_error_propagates() {
         let server = MockServer::start();