Skip to content

Commit

Permalink
shouldi: tests: npm audit: Made vuln count check great than 1
Browse files Browse the repository at this point in the history
The repo we are analyzing has a lock file, so unless ~2700 vulns disappered
(highly unlikely) there is a bug somewhere with npm audit, but not us!

Signed-off-by: John Andersen <johnandersenpdx@gmail.com>
  • Loading branch information
pdxjohnny committed Jan 26, 2022
1 parent 714d325 commit fcda708
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion examples/shouldi/tests/test_cli_use.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ async def test_use_javascript(self):
),
)
self.assertGreater(
list(results.values())[0]["static_analysis"][0].high, 2940
list(results.values())[0]["static_analysis"][0].high, 1
)

async def test_use_rust(self):
Expand Down
2 changes: 1 addition & 1 deletion examples/shouldi/tests/test_npm_audit.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,4 @@ async def test_run(self):
/ "javascript-algorithms-ba2d8dc4a8e27659c1420fe52390cb7981df4a94"
)
)
self.assertGreater(results["report"]["high"], 2941)
self.assertGreater(results["report"]["high"], 1)

0 comments on commit fcda708

Please sign in to comment.