json2xml 7.0.0__tar.gz → 7.0.1__tar.gz

{json2xml-7.0.0 → json2xml-7.0.1}/HISTORY.rst

@@ -1,4 +1,10 @@
 
+7.0.1 / 2026-08-28
+==================
+
+  * fix: validate explicitly provided empty ``--string``, ``--url``, and file-path inputs instead of silently falling back to stdin
+
+
 7.0.0 / 2026-08-23
 ==================
 

{json2xml-7.0.0 → json2xml-7.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: json2xml
-Version: 7.0.0
+Version: 7.0.1
 Summary: Simple Python Library to convert JSON to XML
 Author-email: Vinit Kumar <mail@vinitkumar.me>
 License: Apache Software License 2.0

{json2xml-7.0.0 → json2xml-7.0.1}/json2xml/__init__.py

@@ -2,4 +2,4 @@
 
 __author__ = """Vinit Kumar"""
 __email__ = "mail@vinitkumar.me"
-__version__ = "7.0.0"
+__version__ = "7.0.1"

{json2xml-7.0.0 → json2xml-7.0.1}/json2xml/cli.py

@@ -109,13 +109,14 @@ class CLIApplication:
     """Thin command adapter around input resolution, conversion, and output."""
 
     def read_input(self, options: CLIConversionOptions) -> JSONValue:
-        if options.url:
+        # None means omitted; explicit empty values still require source validation.
+        if options.url is not None:
             try:
                 return readfromurl(options.url)
             except URLReadError as error:
                 exit_with_error(f"Error reading from URL: {error}")
 
-        if options.string:
+        if options.string is not None:
             try:
                 return readfromstring(options.string)
             except StringReadError as error:
@@ -125,9 +126,14 @@ class CLIApplication:
                     f"({error})"
                 )
 
-        if options.input_file:
+        if options.input_file is not None:
             if options.input_file == "-":
                 return read_from_stdin()
+            if not options.input_file:
+                exit_with_error(
+                    "Error: Empty JSON file path. "
+                    "Pass a JSON file, use - for stdin, or provide --string/--url."
+                )
             if not Path(options.input_file).is_file():
                 exit_with_error(
                     f"Error: JSON file not found: {options.input_file}. "

{json2xml-7.0.0 → json2xml-7.0.1}/json2xml.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: json2xml
-Version: 7.0.0
+Version: 7.0.1
 Summary: Simple Python Library to convert JSON to XML
 Author-email: Vinit Kumar <mail@vinitkumar.me>
 License: Apache Software License 2.0

{json2xml-7.0.0 → json2xml-7.0.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "json2xml"
-version = "7.0.0"
+version = "7.0.1"
 description = "Simple Python Library to convert JSON to XML"
 readme = "README.rst"
 requires-python = ">=3.10"

{json2xml-7.0.0 → json2xml-7.0.1}/tests/test_cli.py

@@ -218,6 +218,45 @@ class TestCLI:
         assert result.returncode == 1
         assert "Invalid JSON in --string input" in result.stderr
 
+    # @lat: [[tests#CLI input resolution#Explicit empty string rejects piped stdin]]
+    def test_empty_string_does_not_fall_back_to_stdin(self) -> None:
+        """Test an explicit empty string remains the selected input source."""
+        result = subprocess.run(
+            [sys.executable, "-m", "json2xml.cli", "-s", ""],
+            input='{"stdin": "unexpected"}',
+            capture_output=True,
+            text=True,
+        )
+        assert result.returncode == 1
+        assert "Invalid JSON in --string input" in result.stderr
+        assert result.stdout == ""
+
+    # @lat: [[tests#CLI input resolution#Explicit empty URL rejects piped stdin]]
+    def test_empty_url_does_not_fall_back_to_stdin(self) -> None:
+        """Test an explicit empty URL remains the selected input source."""
+        result = subprocess.run(
+            [sys.executable, "-m", "json2xml.cli", "-u", ""],
+            input='{"stdin": "unexpected"}',
+            capture_output=True,
+            text=True,
+        )
+        assert result.returncode == 1
+        assert "Error reading from URL" in result.stderr
+        assert result.stdout == ""
+
+    # @lat: [[tests#CLI input resolution#Explicit empty file path rejects piped stdin]]
+    def test_empty_file_path_does_not_fall_back_to_stdin(self) -> None:
+        """Test an explicit empty file path remains the selected input source."""
+        result = subprocess.run(
+            [sys.executable, "-m", "json2xml.cli", ""],
+            input='{"stdin": "unexpected"}',
+            capture_output=True,
+            text=True,
+        )
+        assert result.returncode == 1
+        assert "Empty JSON file path" in result.stderr
+        assert result.stdout == ""
+
     def test_no_input_error(self) -> None:
         """Test error when no input is provided."""
         # Force isatty to return True by using a pty-like environment
@@ -607,6 +646,21 @@ class TestCLIUnitTests:
         captured = capsys.readouterr()
         assert "Error reading from URL" in captured.err
 
+    # @lat: [[tests#CLI failure messages#Empty file path names the empty source]]
+    def test_read_input_empty_file_path(self, capsys: CaptureFixture[str]) -> None:
+        """Test read_input rejects an explicitly empty file path."""
+        args = MagicMock()
+        args.url = None
+        args.string = None
+        args.input_file = ""
+
+        with pytest.raises(SystemExit) as exc_info:
+            read_input(args)
+
+        assert exc_info.value.code == 1
+        captured = capsys.readouterr()
+        assert "Empty JSON file path" in captured.err
+
     def test_read_input_json_file_error(self, capsys: CaptureFixture[str]) -> None:
         """Test read_input handles JSON file read errors."""
         from json2xml.utils import JSONReadError