Skip to content

Commit

Permalink
Remove redundant test for vcert log prefix
Browse files Browse the repository at this point in the history
The exact logs output is now verified.

Signed-off-by: Richard Wall <richard.wall@venafi.com>
  • Loading branch information
wallrj committed Oct 29, 2024
1 parent 6f8c52e commit 73526f3
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions pkg/logs/logs_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,12 +37,6 @@ func TestLogs(t *testing.T) {
logs.AddFlags(fs)
fs.Parse(strings.Split(flags, " "))
logs.Initialize()

// Remember to not use `assert.Equal` from within the sub-process
// since (for some reason) `assert.Equal` doesn't cause the test
// sub-process to exit with 1. Instead, it exits with 0, which means
// we can't notice the error from the parent process.
require.Equal(t, "", log.Prefix(), "logs.Initialize should remove the `vCert: ` prefix from the global log logger")
}

log.Print("log Print")
Expand Down

0 comments on commit 73526f3

Please sign in to comment.