refactor(tests): update normalize_path test to cover strip_extension parameter

Author: AlePouroullis

tests/custom/integration/test_sync_cli.py

@@ -147,7 +147,7 @@ def test_pull_with_quiet_mode(
 def test_pull_with_invalid_path(
     cli_runner: CliRunner,
 ):
-    # GIVEN an invalid base directory
+    # GIVEN an invalid path
     path = "nonexistent/path"
 
     # WHEN running pull command

tests/custom/sync/test_client.py

@@ -37,17 +37,26 @@ def test_normalize_path(sync_client: SyncClient):
     """Test path normalization functionality."""
     # GIVEN various file paths with different formats
     test_cases = [
-        ("path/to/file.prompt", "path/to/file"),
-        ("path\\to\\file.agent", "path/to/file"),
-        ("trailing/slashes/file.agent/", "trailing/slashes/file"),
-        ("multiple//slashes//file.prompt", "multiple/slashes/file"),
+        # Input path, expected with strip_extension=False, expected with strip_extension=True
+        ("path/to/file.prompt", "path/to/file.prompt", "path/to/file"),
+        ("path\\to\\file.agent", "path/to/file.agent", "path/to/file"),
+        ("trailing/slashes/file.agent/", "trailing/slashes/file.agent", "trailing/slashes/file"),
+        ("multiple//slashes//file.prompt", "multiple/slashes/file.prompt", "multiple/slashes/file"),
     ]
 
-    for input_path, expected in test_cases:
-        # WHEN they are normalized
-        normalized = sync_client._normalize_path(input_path)
-        # THEN they should be converted to the expected format
-        assert normalized == expected
+    # Test with strip_extension=False (default)
+    for input_path, expected_with_ext, _ in test_cases:
+        # WHEN they are normalized without stripping extension
+        normalized = sync_client._normalize_path(input_path, strip_extension=False)
+        # THEN they should be converted to the expected format with extension
+        assert normalized == expected_with_ext
+
+    # Test with strip_extension=True
+    for input_path, _, expected_without_ext in test_cases:
+        # WHEN they are normalized with extension stripping
+        normalized = sync_client._normalize_path(input_path, strip_extension=True)
+        # THEN they should be converted to the expected format without extension
+        assert normalized == expected_without_ext
 
     # Test absolute path raises error
     with pytest.raises(HumanloopRuntimeError, match="Absolute paths are not supported"):