Skip to content

fix: assert combined report file exists in recursive scan test - #384

Open
andrewwhitecdw wants to merge 1 commit into
NVIDIA:mainfrom
andrewwhitecdw:codequality/test-cli-assert-combined-report-file-exists-in
Open

fix: assert combined report file exists in recursive scan test#384
andrewwhitecdw wants to merge 1 commit into
NVIDIA:mainfrom
andrewwhitecdw:codequality/test-cli-assert-combined-report-file-exists-in

Conversation

@andrewwhitecdw

Copy link
Copy Markdown

This PR addresses the following issue in tests/unit/test_cli.py: assert combined report file exists in recursive scan test.

Changes

  • tests/unit/test_cli.py: assert combined report file exists in recursive scan test.

Details

--- a/tests/unit/test_cli.py
+++ b/tests/unit/test_cli.py
@@ -1,2 +1,3 @@
-    assert result.exit_code == 0
-    payload = json.loads(out_file.read_text(encoding="utf-8"))
+    assert result.exit_code == 0
+    assert out_file.exists()
+    payload = json.loads(out_file.read_text(encoding="utf-8"))

Tests

  • tests/unit/test_cli.py

Contributor guidelines

Per this repo's CONTRIBUTING.md:

  • All commits are signed off (Signed-off-by trailer, DCO).

Signed-off-by: andrewwhitecdw <andrewwhitecdw@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant