Skip to content

Commit

Permalink
ignore G204 in latency_testing
Browse files Browse the repository at this point in the history
This is ignored temporarily as gosec resolver doesnt properly support
Call Expressions
https://github.com/securego/gosec/blob/master/resolve.go#L70
  • Loading branch information
yuvalk committed Oct 4, 2021
1 parent 01f8f9a commit 6e001d2
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions functests/5_latency_testing/latency_testing.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,8 +93,7 @@ var _ = table.DescribeTable("Test latency measurement tools tests", func(testGro
if _, err := os.Stat("../../build/_output/bin/latency-e2e.test"); os.IsNotExist(err) {
Skip("The executable test file does not exist , skipping the test.")
}
output, err := exec.Command("../../build/_output/bin/latency-e2e.test", "-ginkgo.focus", test.toolToTest).Output()

output, err := exec.Command("../../build/_output/bin/latency-e2e.test", "-ginkgo.focus", test.toolToTest).Output() // #nosec G204
if err != nil {
//we don't fail the test here because the test might be a negative check
testlog.Info(err.Error())
Expand Down

0 comments on commit 6e001d2

Please sign in to comment.