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 25, 2024
1 parent 5ab600c commit c2b6d69
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 c2b6d69

Please sign in to comment.