test: add test for directory.rs

Author: wislertt

.dev/27-integration-tests-revamp-plan.md

@@ -142,15 +142,17 @@ tests/integration_tests/version/
 - ✅ Enhanced `ZervFixture.with_vcs_data()` to accept `Option` types for better flexibility
 - ✅ Implemented `formats.rs`: Comprehensive format conversion tests (30 tests)
 - ✅ Implemented `schemas.rs`: Comprehensive schema tests (31 tests)
-- **Result**: 82 tests passing (100% success rate) - 7 source tests + 30 format tests + 31 schema tests + 14 other tests
-- **Performance**: Tests run in <1.3 seconds without Docker
+- ✅ Implemented `templates.rs`: Comprehensive template tests covering all helpers and edge cases (62 tests)
+- ✅ Implemented `directory.rs`: Directory flag tests with Git integration and error handling (4 tests)
+- **Result**: 172 tests passing (100% success rate) - 7 source tests + 30 format tests + 31 schema tests + 62 template tests + 4 directory tests + 38 other tests
+- **Performance**: Tests run in <0.5 seconds without Docker
 
 **Remaining MainConfig Tests:**
 
 - ✅ `formats.rs`: Test `--input-format` (semver/pep440/auto) and `--output-format` (semver/pep440/zerv) combinations, format validation errors, error message consistency (✅ PASSED - 30 tests)
 - ✅ `schemas.rs`: Test `--schema` (zerv-standard/zerv-calver) and `--schema-ron` (custom RON schema) options (✅ PASSED - 31 tests)
-- ❌ `templates.rs`: Test `--output-template` with Handlebars template rendering
-- ❌ `directory.rs`: Test `-C` flag for changing working directory before execution
+- ✅ `templates.rs`: Test `--output-template` with Handlebars template rendering, all helpers (sanitize, hash, prefix, timestamp, math), complex scenarios, edge cases (✅ PASSED - 62 tests)
+- ✅ `directory.rs`: Test `-C` flag for changing working directory before execution (✅ PASSED - 4 tests: 2 Git integration + 2 error handling)
 - ❌ `combinations.rs`: Test MainConfig option combinations (format + schema, template + format, etc.)
 
 #### Phase 3: Implement Override Tests (`overrides/`)

tests/integration_tests/version/main/directory.rs

@@ -0,0 +1,114 @@
+use zerv::test_utils::{
+    GitRepoFixture,
+    TestDir,
+    should_run_docker_tests,
+};
+
+use crate::util::TestCommand;
+
+mod directory_git_integration {
+    use super::*;
+
+    #[test]
+    fn test_directory_flag_with_subdirectory() {
+        if !should_run_docker_tests() {
+            return;
+        }
+
+        let git_repo =
+            GitRepoFixture::tagged("v1.0.0").expect("Failed to create tagged Git repository");
+
+        let parent_dir = git_repo
+            .path()
+            .parent()
+            .expect("Git repo should have parent directory");
+
+        let output = TestCommand::new()
+            .current_dir(parent_dir)
+            .args_from_str(format!(
+                "version -C {} --source git --output-format semver",
+                git_repo.path().file_name().unwrap().to_string_lossy()
+            ))
+            .assert_success();
+
+        assert_eq!(
+            output.stdout().trim(),
+            "1.0.0",
+            "Should detect version from Git repo in subdirectory using -C flag"
+        );
+    }
+
+    #[test]
+    fn test_directory_flag_relative_vs_absolute() {
+        if !should_run_docker_tests() {
+            return;
+        }
+
+        let git_repo =
+            GitRepoFixture::tagged("v1.0.0").expect("Failed to create tagged Git repository");
+
+        let relative_output = TestCommand::new()
+            .current_dir(git_repo.path().parent().unwrap())
+            .args_from_str(format!(
+                "version -C {} --source git --output-format semver",
+                git_repo.path().file_name().unwrap().to_string_lossy()
+            ))
+            .assert_success();
+
+        let absolute_output = TestCommand::new()
+            .args_from_str(format!(
+                "version -C {} --source git --output-format semver",
+                git_repo.path().display()
+            ))
+            .assert_success();
+
+        assert_eq!(
+            relative_output.stdout().trim(),
+            "1.0.0",
+            "Relative path should work"
+        );
+        assert_eq!(
+            absolute_output.stdout().trim(),
+            "1.0.0",
+            "Absolute path should work"
+        );
+        assert_eq!(
+            relative_output.stdout(),
+            absolute_output.stdout(),
+            "Relative and absolute paths should produce identical output"
+        );
+    }
+}
+
+mod directory_error_handling {
+    use super::*;
+
+    #[test]
+    fn test_directory_flag_nonexistent_path() {
+        let output = TestCommand::new()
+            .args_from_str("version -C /nonexistent/path/to/directory")
+            .assert_failure();
+
+        let stderr = output.stderr();
+        assert!(
+            stderr.contains("Error") && stderr.contains("VCS not found"),
+            "Should show VCS not found error when directory doesn't exist. Got: {stderr}"
+        );
+    }
+
+    #[test]
+    fn test_directory_flag_exists_but_not_git() {
+        let test_dir = TestDir::new().expect("Failed to create test directory");
+
+        let output = TestCommand::new()
+            .args_from_str(format!("version -C {}", test_dir.path().display()))
+            .assert_failure();
+
+        let stderr = output.stderr();
+        assert_eq!(
+            stderr.trim(),
+            "Error: VCS not found: Not in a git repository (--source git)",
+            "Should show proper error when directory exists but is not a git repo"
+        );
+    }
+}

tests/integration_tests/version/main/mod.rs

@@ -1,3 +1,4 @@
+pub mod directory;
 pub mod formats;
 pub mod schemas;
 pub mod sources;