_cli: make reformat (#94)

Author: woodruffw

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- The `inspect` subcommand now ignores inputs that don't match `*.attestation`,
+  rather than failing on them
+  ([#93](https://github.com/trailofbits/pypi-attestations/pull/93))
+
 ### Added
 
 - The CLI subcommand `verify attestation` now supports `.slsa.attestation`
@@ -22,7 +28,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
-- The CLI entrypoint is now `pypi-attestations` 
+- The CLI entrypoint is now `pypi-attestations`
   ([#82](https://github.com/trailofbits/pypi-attestations/pull/82))
 - The CLI `verify` subcommand has been changed to `verify attestation`,
   as in `pypi-attestations verify attestation --identity ...`

src/pypi_attestations/_cli.py

@@ -390,7 +390,7 @@ def _inspect(args: argparse.Namespace) -> None:
 
     Warning: The information displayed from the attestations are not verified.
     """
-    attestation_files = [f for f in args.files if f.suffix == '.attestation']
+    attestation_files = [f for f in args.files if f.suffix == ".attestation"]
     _validate_files(attestation_files, should_exist=True)
     for file_path in attestation_files:
         try:

test/test_cli.py

@@ -199,21 +199,14 @@ def test_inspect_command(caplog: pytest.LogCaptureFixture) -> None:
 
     # Failure because not an attestation
     with tempfile.NamedTemporaryFile(suffix=".publish.attestation") as f:
-        fake_package_name = Path(f.name.removesuffix(".publish.attestation"))
-        fake_package_name.touch()
+        f.write(b"not an attestation")
+        f.flush()
 
         with pytest.raises(SystemExit):
-            run_main_with_command(["inspect", fake_package_name.as_posix()])
+            run_main_with_command(["inspect", f.name])
 
         assert "Invalid attestation" in caplog.text
 
-    # Failure because file is missing
-    caplog.clear()
-    with pytest.raises(SystemExit):
-        run_main_with_command(["inspect", "not_a_file.txt"])
-
-    assert "not_a_file.txt is not a file." in caplog.text
-
 
 def test_verify_attestation_command(caplog: pytest.LogCaptureFixture) -> None:
     # Happy path