Reject empty string paths in file utility

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

hyperbrowser/client/file_utils.py

@@ -20,6 +20,8 @@ def ensure_existing_file_path(
         ) from exc
     if not isinstance(normalized_path, str):
         raise HyperbrowserError("file_path must resolve to a string path")
+    if not normalized_path:
+        raise HyperbrowserError("file_path must not be empty")
     if not os.path.exists(normalized_path):
         raise HyperbrowserError(missing_file_message)
     if not os.path.isfile(normalized_path):

tests/test_file_utils.py

@@ -70,3 +70,12 @@ def test_ensure_existing_file_path_rejects_non_string_fspath_results():
             missing_file_message="missing",
             not_file_message="not-file",
         )
+
+
+def test_ensure_existing_file_path_rejects_empty_string_paths():
+    with pytest.raises(HyperbrowserError, match="file_path must not be empty"):
+        ensure_existing_file_path(
+            "",
+            missing_file_message="missing",
+            not_file_message="not-file",
+        )