Int test fix - alternate approach

opentelemetry-otlp/tests/integration_test/Cargo.toml

@@ -16,6 +16,7 @@ anyhow = "1.0.94"
 ctor = "0.2.9"
 tracing-subscriber = { workspace = true, features = ["env-filter","registry", "std", "fmt"] }
 tracing = {workspace = true}
+serial_test = "3.2.0"
 
 [target.'cfg(unix)'.dependencies]
 opentelemetry-appender-tracing = { path = "../../../opentelemetry-appender-tracing", default-features = false}

opentelemetry-otlp/tests/integration_test/src/test_utils.rs

@@ -18,10 +18,12 @@
 //!
 #![cfg(unix)]
 
+use anyhow::Ok;
 use anyhow::Result;
 use opentelemetry::{otel_debug, otel_info};
 use std::fs;
 use std::fs::File;
+use std::fs::OpenOptions;
 use std::os::unix::fs::PermissionsExt;
 use std::sync::{Arc, Mutex, Once, OnceLock};
 use testcontainers::core::wait::HttpWaitStrategy;
@@ -125,6 +127,17 @@ fn upsert_empty_file(path: &str) -> File {
     file
 }
 
+/// Cleans up file specificed as argument by truncating its content.
+///
+/// This function is meant to cleanup the generated json file before a test starts,
+/// preventing entries from previous tests from interfering with the current test's results.
+pub fn cleanup_logs_file(file_path: &str) {
+    let _ = OpenOptions::new()
+        .write(true)
+        .truncate(true)
+        .open(file_path); // ignore result, as file may not exist
+}
+
 ///
 /// Shuts down our collector container. This should be run as part of each test
 /// suite shutting down!

opentelemetry-otlp/tests/integration_test/tests/logs.rs

@@ -39,7 +39,7 @@ mod logtests {
     use super::*;
     use integration_test_runner::logs_asserter::{read_logs_from_json, LogsAsserter};
     use opentelemetry_appender_tracing::layer::OpenTelemetryTracingBridge;
-    use std::{fs::File, time::Duration};
+    use std::fs::File;
     use tracing::info;
     use tracing_subscriber::layer::SubscriberExt;
 
@@ -70,9 +70,11 @@ mod logtests {
     #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
     #[cfg(not(feature = "hyper-client"))]
     #[cfg(not(feature = "reqwest-client"))]
+    #[serial_test::serial]
     pub async fn test_logs() -> Result<()> {
-        // Make sure the container is running
+        test_utils::cleanup_logs_file("./actual/logs.json"); // Ensure logs.json is empty before the test
 
+        // Make sure the container is running
         use integration_test_runner::test_utils;
         use opentelemetry_appender_tracing::layer;
         use tracing::info;
@@ -88,21 +90,20 @@ mod logtests {
             let _guard = tracing::subscriber::set_default(subscriber);
             info!(target: "my-target", "hello from {}. My price is {}.", "banana", 2.99);
         }
-        // TODO: remove below wait before calling logger_provider.shutdown()
-        // tokio::time::sleep(Duration::from_secs(10)).await;
-        let _ = logger_provider.shutdown();
-
-        tokio::time::sleep(Duration::from_secs(10)).await;
 
+        let _ = logger_provider.shutdown();
         assert_logs_results(test_utils::LOGS_FILE, "expected/logs.json")?;
 
         Ok(())
     }
 
-    #[ignore = "TODO: [Fix Me] Failing on CI. Needs to be investigated and resolved."]
+    //#[ignore = "TODO: [Fix Me] Failing on CI. Needs to be investigated and resolved."]
     #[test]
+    #[serial_test::serial]
     #[cfg(any(feature = "tonic-client", feature = "reqwest-blocking-client"))]
     pub fn logs_batch_non_tokio_main() -> Result<()> {
+        test_utils::cleanup_logs_file("./actual/logs.json"); // Ensure logs.json is empty before the test
+
         // Initialize the logger provider inside a tokio runtime
         // as this allows tonic client to capture the runtime,
         // but actual export occurs from the dedicated std::thread
@@ -114,15 +115,14 @@ mod logtests {
             init_logs()
         })?;
 
-        info!("LoggerProvider created");
         let layer = OpenTelemetryTracingBridge::new(&logger_provider);
         let subscriber = tracing_subscriber::registry().with(layer);
         {
             let _guard = tracing::subscriber::set_default(subscriber);
             info!(target: "my-target", "hello from {}. My price is {}.", "banana", 2.99);
         }
         let _ = logger_provider.shutdown();
-        // tokio::time::sleep(Duration::from_secs(10)).await;
+
         assert_logs_results(test_utils::LOGS_FILE, "expected/logs.json")?;
 
         Ok(())