Skip to content

Commit 8e3a921

Browse files
committed
Revert "fix: test_sec_audit: ignore cargo audit output"
This reverts commit d761b01. The output to stdout was fixed in cargo audit 0.21.2, so if we rebuild the docker container the grep is no longer necessary. In fact, the grep has broken this test in our nightly pipeline because it overwrites the return code of cargo audit itself, meaning the non-PR version of this test (which is supposed to fail if there exist any cargo audit warnings) was never failing. (cherry picked from commit 1d98a21) Signed-off-by: Patrick Roy <[email protected]>
1 parent 762f659 commit 8e3a921

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/integration_tests/security/test_sec_audit.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ def set_of_vulnerabilities(output: CommandReturn):
3535
)
3636

3737
git_ab_test_host_command_if_pr(
38-
"cargo audit --deny warnings -q --json |grep -Po '{.*}'",
38+
"cargo audit --deny warnings -q --json",
3939
comparator=set_did_not_grow_comparator(set_of_vulnerabilities),
4040
)

0 commit comments

Comments
 (0)