github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-22 19:31:44 +00:00
Code Issues Projects Releases Wiki Activity

k8s.io/component-base/logs: match full help text in unit test

Browse Source 
The test was meant to fail when there are unexpected additional lines like the
"panic calling String method" messages. But `assert.Regexp` does a search for
the regexp, not a full string match, and thus succeeded even if those lines are
present. A full match needs to be requested explicitly with ^ and $.
This commit is contained in:
Patrick Ohly 2023-01-19 08:26:05 +01:00
parent 4add08dcce
commit d6480dacd6
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
staging/src/k8s.io/component-base/logs/api/v1/options_test.go
View File

@ -114,11 +114,11 @@ func TestFlagSet(t *testing.T) {
// --log-flush-frequency duration Maximum number of seconds between log flushes (default 5s)
// -v, --v Level number for the log level verbosity
// --vmodule pattern=N,... comma-separated list of pattern=N settings for file-filtered logging (only works for text log format)
assert.Regexp(t, `.*--logging-format.*default.*text.*
assert.Regexp(t, `^.*--logging-format.*default.*text.*
.*--log-flush-frequency.*default 5s.*
.*-v.*--v.*
.*--vmodule.*pattern=N.*
`, buffer.String())
$`, buffer.String())
})
t.Run("flag", func(t *testing.T) {
@ -141,7 +141,7 @@ func TestFlagSet(t *testing.T) {
// number for the log level verbosity
// -vmodule value
// comma-separated list of pattern=N settings for file-filtered logging (only works for text log format)
assert.Regexp(t, `.*-log-flush-frequency.*
assert.Regexp(t, `^.*-log-flush-frequency.*
.*default 5s.*
.*-logging-format.*
.*default.*text.*
@ -149,7 +149,7 @@ func TestFlagSet(t *testing.T) {
.*
.*-vmodule.*
.*
`, buffer.String())
$`, buffer.String())
})
}