fix(test): switch --nvd option to JSON for broken long tests (#1425)

* ci: switch nvd option to json for long tests

* ci: update to latest CVE data for long tests every run

* fix(test): `test_update` and `test_update_flags`

Author: BreadGenie

.github/workflows/pythonapp.yml

@@ -176,7 +176,7 @@ jobs:
       - name: try single cli run of tool
         run: |
           python -m pip install -e .
-          NO_EXIT_CVE_NUM=1 python -m cve_bin_tool.cli test/assets
+          NO_EXIT_CVE_NUM=1 python -m cve_bin_tool.cli test/assets -n json -u latest
       - name: Run async tests
         run: >
           pytest --cov --cov-append -n 4 -v

test/test_cli.py

@@ -122,11 +122,20 @@ def test_invalid_parameter(self):
 
     @unittest.skipUnless(LONG_TESTS() > 0, "Skipping long tests")
     def test_update_flags(self):
-        assert main(["cve-bin-tool", "-x", "-u", "never", self.tempdir]) != 0
-        assert main(["cve-bin-tool", "-x", "--update", "daily", self.tempdir]) != 0
-        assert main(["cve-bin-tool", "-x", "-u", "now", self.tempdir]) != 0
+        assert (
+            main(["cve-bin-tool", "-x", "-u", "never", "-n", "json", self.tempdir]) != 0
+        )
+        assert (
+            main(
+                ["cve-bin-tool", "-x", "--update", "daily", "-n", "json", self.tempdir]
+            )
+            != 0
+        )
+        assert (
+            main(["cve-bin-tool", "-x", "-u", "now", "-n", "json", self.tempdir]) != 0
+        )
         with pytest.raises(SystemExit) as e:
-            main(["cve-bin-tool", "-u", "whatever", self.tempdir])
+            main(["cve-bin-tool", "-u", "whatever", "-n", "json", self.tempdir])
         assert e.value.args[0] == -2
 
     @staticmethod
@@ -194,7 +203,7 @@ def test_update(self, caplog):
         test_path = os.path.join(os.path.abspath(os.path.dirname(__file__)), "csv")
 
         with caplog.at_level(logging.INFO):
-            main(["cve-bin-tool", "-u", "never", test_path])
+            main(["cve-bin-tool", "-u", "never", "-n", "json", test_path])
         assert (
             "cve_bin_tool",
             logging.WARNING,
@@ -203,7 +212,9 @@ def test_update(self, caplog):
         caplog.clear()
 
         with caplog.at_level(logging.DEBUG):
-            main(["cve-bin-tool", "-l", "debug", "-u", "daily", test_path])
+            main(
+                ["cve-bin-tool", "-l", "debug", "-u", "daily", "-n", "json", test_path]
+            )
         assert (
             "cve_bin_tool.CVEDB",
             logging.INFO,
@@ -216,7 +227,7 @@ def test_update(self, caplog):
         caplog.clear()
 
         with caplog.at_level(logging.DEBUG):
-            main(["cve-bin-tool", "-l", "debug", "-u", "now", test_path])
+            main(["cve-bin-tool", "-l", "debug", "-u", "now", "-n", "json", test_path])
         db_path = DISK_LOCATION_DEFAULT
         assert (
             "cve_bin_tool.CVEDB",
@@ -230,7 +241,9 @@ def test_update(self, caplog):
         caplog.clear()
 
         with caplog.at_level(logging.DEBUG):
-            main(["cve-bin-tool", "-l", "debug", "-u", "latest", test_path])
+            main(
+                ["cve-bin-tool", "-l", "debug", "-u", "latest", "-n", "json", test_path]
+            )
         assert (
             "cve_bin_tool.CVEDB",
             logging.DEBUG,