refactor: modularize tests and reorganize project structure

- Move tests from src/lib.rs to dedicated src/tests.rs and tests/ directory
- Reorganize fixtures into tests/fixtures/ with proper categorization
- Add integration tests for CLI functionality and profile inheritance
- Improve test organization with common test utilities in tests/common/
- Move provider tests to focused test modules

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

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

Author: domenkozar

src/lib.rs

@@ -164,4 +164,25 @@ mod tests {
         let config = DotEnvConfig::from_uri(&uri).unwrap();
         assert_eq!(config.path.to_str().unwrap(), ".env");
     }
+
+    #[test]
+    fn test_from_path_string() {
+        // Test empty path defaults to .env
+        let config = DotEnvConfig::from_path_string("");
+        assert_eq!(config.path.to_str().unwrap(), ".env");
+
+        // Test relative path
+        let config = DotEnvConfig::from_path_string("custom/.env");
+        assert_eq!(config.path.to_str().unwrap(), "custom/.env");
+
+        // Test absolute path
+        let config = DotEnvConfig::from_path_string("/etc/secrets/.env");
+        assert_eq!(config.path.to_str().unwrap(), "/etc/secrets/.env");
+    }
+
+    #[test]
+    fn test_default_config() {
+        let config = DotEnvConfig::default();
+        assert_eq!(config.path.to_str().unwrap(), ".env");
+    }
 }

src/provider/dotenv.rs

@@ -1,95 +1,92 @@
-#[cfg(test)]
-mod tests {
-    use crate::provider::ProviderRegistry;
-
-    #[test]
-    fn test_create_from_string_with_full_uris() {
-        // Test basic 1password URI
-        let provider = ProviderRegistry::create_from_string("1password://Private").unwrap();
-        assert_eq!(provider.name(), "1password");
-
-        // Test 1password with account
-        let provider = ProviderRegistry::create_from_string("1password://work@Production").unwrap();
-        assert_eq!(provider.name(), "1password");
-
-        // Test 1password with token
-        let provider =
-            ProviderRegistry::create_from_string("1password+token://:ops_abc123@Private").unwrap();
-        assert_eq!(provider.name(), "1password");
-    }
+use crate::provider::ProviderRegistry;
+
+#[test]
+fn test_create_from_string_with_full_uris() {
+    // Test basic 1password URI
+    let provider = ProviderRegistry::create_from_string("1password://Private").unwrap();
+    assert_eq!(provider.name(), "1password");
+
+    // Test 1password with account
+    let provider = ProviderRegistry::create_from_string("1password://work@Production").unwrap();
+    assert_eq!(provider.name(), "1password");
+
+    // Test 1password with token
+    let provider =
+        ProviderRegistry::create_from_string("1password+token://:ops_abc123@Private").unwrap();
+    assert_eq!(provider.name(), "1password");
+}
 
-    #[test]
-    fn test_create_from_string_with_plain_names() {
-        // Test plain provider names
-        let provider = ProviderRegistry::create_from_string("env").unwrap();
-        assert_eq!(provider.name(), "env");
+#[test]
+fn test_create_from_string_with_plain_names() {
+    // Test plain provider names
+    let provider = ProviderRegistry::create_from_string("env").unwrap();
+    assert_eq!(provider.name(), "env");
 
-        let provider = ProviderRegistry::create_from_string("keyring").unwrap();
-        assert_eq!(provider.name(), "keyring");
+    let provider = ProviderRegistry::create_from_string("keyring").unwrap();
+    assert_eq!(provider.name(), "keyring");
 
-        let provider = ProviderRegistry::create_from_string("dotenv").unwrap();
-        assert_eq!(provider.name(), "dotenv");
+    let provider = ProviderRegistry::create_from_string("dotenv").unwrap();
+    assert_eq!(provider.name(), "dotenv");
 
-        let provider = ProviderRegistry::create_from_string("1password").unwrap();
-        assert_eq!(provider.name(), "1password");
+    let provider = ProviderRegistry::create_from_string("1password").unwrap();
+    assert_eq!(provider.name(), "1password");
 
-        let provider = ProviderRegistry::create_from_string("lastpass").unwrap();
-        assert_eq!(provider.name(), "lastpass");
-    }
+    let provider = ProviderRegistry::create_from_string("lastpass").unwrap();
+    assert_eq!(provider.name(), "lastpass");
+}
 
-    #[test]
-    fn test_create_from_string_with_colon() {
-        // Test provider names with colon
-        let provider = ProviderRegistry::create_from_string("env:").unwrap();
-        assert_eq!(provider.name(), "env");
+#[test]
+fn test_create_from_string_with_colon() {
+    // Test provider names with colon
+    let provider = ProviderRegistry::create_from_string("env:").unwrap();
+    assert_eq!(provider.name(), "env");
 
-        let provider = ProviderRegistry::create_from_string("keyring:").unwrap();
-        assert_eq!(provider.name(), "keyring");
-    }
+    let provider = ProviderRegistry::create_from_string("keyring:").unwrap();
+    assert_eq!(provider.name(), "keyring");
+}
 
-    #[test]
-    fn test_invalid_onepassword_scheme() {
-        // Test that 'onepassword' scheme gives proper error
-        let result = ProviderRegistry::create_from_string("onepassword");
-        match result {
-            Err(err) => assert!(err.to_string().contains("Use '1password' instead")),
-            Ok(_) => panic!("Expected error for 'onepassword' scheme"),
-        }
+#[test]
+fn test_invalid_onepassword_scheme() {
+    // Test that 'onepassword' scheme gives proper error
+    let result = ProviderRegistry::create_from_string("onepassword");
+    match result {
+        Err(err) => assert!(err.to_string().contains("Use '1password' instead")),
+        Ok(_) => panic!("Expected error for 'onepassword' scheme"),
+    }
 
-        let result = ProviderRegistry::create_from_string("onepassword:");
-        match result {
-            Err(err) => assert!(err.to_string().contains("Use '1password' instead")),
-            Ok(_) => panic!("Expected error for 'onepassword:' scheme"),
-        }
+    let result = ProviderRegistry::create_from_string("onepassword:");
+    match result {
+        Err(err) => assert!(err.to_string().contains("Use '1password' instead")),
+        Ok(_) => panic!("Expected error for 'onepassword:' scheme"),
+    }
 
-        let result = ProviderRegistry::create_from_string("onepassword://Private");
-        match result {
-            Err(err) => assert!(err.to_string().contains("Use '1password' instead")),
-            Ok(_) => panic!("Expected error for 'onepassword://' scheme"),
-        }
+    let result = ProviderRegistry::create_from_string("onepassword://Private");
+    match result {
+        Err(err) => assert!(err.to_string().contains("Use '1password' instead")),
+        Ok(_) => panic!("Expected error for 'onepassword://' scheme"),
     }
+}
 
-    #[test]
-    fn test_dotenv_with_custom_path() {
-        // Test dotenv provider with custom path
-        let provider =
-            ProviderRegistry::create_from_string("dotenv://localhost/custom/path/.env").unwrap();
-        assert_eq!(provider.name(), "dotenv");
+#[test]
+fn test_dotenv_with_custom_path() {
+    // Test dotenv provider with custom path
+    let provider =
+        ProviderRegistry::create_from_string("dotenv://localhost/custom/path/.env").unwrap();
+    assert_eq!(provider.name(), "dotenv");
 
-        // Test with the simplified format
-        let provider = ProviderRegistry::create_from_string("dotenv:/custom/path/.env").unwrap();
-        assert_eq!(provider.name(), "dotenv");
-    }
+    // Test with the simplified format
+    let provider = ProviderRegistry::create_from_string("dotenv:/custom/path/.env").unwrap();
+    assert_eq!(provider.name(), "dotenv");
+}
 
-    #[test]
-    fn test_unknown_provider() {
-        let result = ProviderRegistry::create_from_string("unknown");
-        assert!(result.is_err());
-        match result {
-            Err(crate::SecretSpecError::ProviderNotFound(scheme)) => {
-                assert_eq!(scheme, "unknown");
-            }
-            _ => panic!("Expected ProviderNotFound error"),
+#[test]
+fn test_unknown_provider() {
+    let result = ProviderRegistry::create_from_string("unknown");
+    assert!(result.is_err());
+    match result {
+        Err(crate::SecretSpecError::ProviderNotFound(scheme)) => {
+            assert_eq!(scheme, "unknown");
         }
+        _ => panic!("Expected ProviderNotFound error"),
     }
 }