fix: resolve test compilation errors and warnings

Fixed issues after initial warning cleanup:
- Added HashMap import to transform.rs test module
- Added LogLevel import to logging.rs test module
- Removed unused imports from test files:
  * integration_tests.rs: CopyMode, is_zero_copy_available, Write
  * error_handling_integration_tests.rs: LogLevel, TempDir
  * filter_integration_test.rs: PathBuf
  * delta_integration_test.rs: Write
- Removed unused imports from source files:
  * logging.rs: PathBuf in test module
  * filter.rs: PathBuf in test module
  * telemetry.rs: ProgressPublisher in test module
- Added #[allow(unused_variables)] to platform-specific functions in file_metadata.rs
- Added unicode-normalization feature to Cargo.toml to fix cfg warnings

All tests now compile without errors or warnings.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: Shane Wall

Cargo.lock

@@ -53,6 +53,9 @@ tempfile = "3.10"
 glob = "0.3"
 regex = "1.10"
 
+# Unicode normalization for path handling (optional)
+unicode-normalization = { version = "0.1", optional = true }
+
 # Date/time
 chrono = { version = "0.4", features = ["serde"] }
 
@@ -174,5 +177,8 @@ delta-manifest = ["dep:rusqlite"]
 # Extended attributes and metadata preservation
 extended-metadata = ["dep:xattr"]
 
+# Unicode normalization for consistent path encoding
+unicode-normalization = ["dep:unicode-normalization"]
+
 # Full feature set
 full = ["zero-copy", "protocols", "smb-native", "s3-native", "ssh-backend", "backend-abstraction", "delta-manifest", "extended-metadata"]

Cargo.toml

@@ -274,6 +274,7 @@ impl FileMetadata {
     }
 
     /// Apply permissions to destination
+    #[allow(unused_variables)]
     fn apply_permissions(&self, dest_path: &Path) -> Result<()> {
         #[cfg(unix)]
         if let Some(mode) = self.permissions {
@@ -293,6 +294,7 @@ impl FileMetadata {
     }
 
     /// Apply ownership to destination (Unix only, requires privileges)
+    #[allow(unused_variables)]
     fn apply_ownership(&self, dest_path: &Path) -> Result<()> {
         #[cfg(unix)]
         {

src/core/file_metadata.rs

@@ -473,7 +473,6 @@ fn add_pattern_to_filter(
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::path::PathBuf;
 
     #[test]
     fn test_glob_matching() {

src/core/filter.rs

@@ -348,6 +348,7 @@ impl Default for TransformBuilder {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use std::collections::HashMap;
 
     #[test]
     fn test_path_transform_basename() {

src/core/transform.rs

@@ -104,7 +104,7 @@ pub fn init_test_logging() {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::path::PathBuf;
+    use crate::config::LogLevel;
     use tempfile::NamedTempFile;
 
     #[test]

src/logging.rs

@@ -276,7 +276,7 @@ impl TelemetryLogger {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::core::progress::{ProgressPublisher, FileId};
+    use crate::core::progress::FileId;
     use std::path::PathBuf;
 
     #[test]

src/telemetry.rs

@@ -4,7 +4,6 @@
 
 use orbit::{copy_file, config::CopyConfig, core::delta::CheckMode};
 use std::fs;
-use std::io::Write;
 use tempfile::tempdir;
 
 #[test]

tests/delta_integration_test.rs

@@ -10,15 +10,14 @@
  */
 
 use orbit::{
-    config::{CopyConfig, ErrorMode, LogLevel},
+    config::{CopyConfig, ErrorMode},
     error::{OrbitError, ErrorCategory},
     instrumentation::OperationStats,
     core::CopyStats,
 };
 use std::sync::atomic::{AtomicU32, Ordering};
 use std::sync::Arc;
 use std::time::{Duration, Instant};
-use tempfile::TempDir;
 
 /// Simulated flaky operation that fails N times before succeeding
 struct FlakyOperation {

tests/error_handling_integration_tests.rs

@@ -4,7 +4,6 @@
 
 use orbit::{copy_directory, CopyConfig, CopyMode};
 use std::fs;
-use std::path::PathBuf;
 use tempfile::TempDir;
 
 #[test]