Remove test and add changelog entry for SdkLogger Clone implementation

Co-authored-by: cijothomas <[email protected]>

Author: Copilot

opentelemetry-sdk/CHANGELOG.md

@@ -5,6 +5,7 @@
 - TODO: Placeholder for Span processor related things
   - *Fix* SpanProcessor::on_start is no longer called on non recording spans
 - **Fix**: Restore true parallel exports in the async-native `BatchSpanProcessor` by honoring `OTEL_BSP_MAX_CONCURRENT_EXPORTS` ([#2959](https://github.com/open-telemetry/opentelemetry-rust/pull/3028)). A regression in [#2685](https://github.com/open-telemetry/opentelemetry-rust/pull/2685) inadvertently awaited the `export()` future directly in `opentelemetry-sdk/src/trace/span_processor_with_async_runtime.rs` instead of spawning it on the runtime, forcing all exports to run sequentially.
+- **Feature**: Added `Clone` implementation to `SdkLogger` for consistency with `SdkTracer` ([#3058](https://github.com/open-telemetry/opentelemetry-rust/issues/3058)).
 
 ## 0.30.0
 

opentelemetry-sdk/src/logs/logger.rs

@@ -64,24 +64,3 @@ impl opentelemetry::logs::Logger for SdkLogger {
             .any(|processor| processor.event_enabled(level, target, name))
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use crate::logs::SdkLoggerProvider;
-    use opentelemetry::InstrumentationScope;
-
-    #[test]
-    fn test_sdk_logger_clone() {
-        let scope = InstrumentationScope::builder("test").build();
-        let provider = SdkLoggerProvider::builder().build();
-        let logger = SdkLogger::new(scope, provider);
-
-        // Test that clone works - this is the main goal
-        #[allow(clippy::redundant_clone)]
-        let cloned_logger = logger.clone();
-
-        // Verify that both loggers are valid by checking they have the same scope name
-        assert_eq!(logger.scope.name(), cloned_logger.scope.name());
-    }
-}