diff options
| author |  Miek Gieben <miek@miek.nl>
| 2018-07-19 16:23:06 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2018-07-19 16:23:06 +0100 |
| commit | f3134da45eb2808431a29fa55c58e3271bbba637 (patch) | |
| tree | f4cf5a6e2453959cbb69b4e46e82efb62b370a1c /test/ds_file_test.go | |
| parent | c69bed726b4a7b8726833e009ed6dd70a74f0dec (diff) | |
| download | coredns-f3134da45eb2808431a29fa55c58e3271bbba637.tar.gz coredns-f3134da45eb2808431a29fa55c58e3271bbba637.tar.zst coredns-f3134da45eb2808431a29fa55c58e3271bbba637.zip | |
Clean up tests logging (#1979)
* Clean up tests logging
This cleans up the travis logs so you can see the failures better.
Older tests in tests/ would call log.SetOutput(ioutil.Discard) in
a haphazard way. This add log.Discard and put an `init` function in each
package's dir (no way to do this globally). The cleanup in tests/ is
clear.
All plugins also got this init function to have some uniformity and kill
any (future) logging there in the tests as well.
There is a one-off in pkg/healthcheck because that does log.
Signed-off-by: Miek Gieben <miek@miek.nl>
* bring back original log_test.go
Signed-off-by: Miek Gieben <miek@miek.nl>
* suppress logging here as well
Signed-off-by: Miek Gieben <miek@miek.nl>
Diffstat (limited to 'test/ds_file_test.go')
| -rw-r--r-- | test/ds_file_test.go | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/test/ds_file_test.go b/test/ds_file_test.go index 356fd2b0b..2b2be5795 100644 --- a/test/ds_file_test.go +++ b/test/ds_file_test.go @@ -1,8 +1,6 @@ package test import ( - "io/ioutil" - "log" "testing" "github.com/coredns/coredns/plugin/proxy" @@ -49,8 +47,6 @@ func TestLookupDS(t *testing.T) { } defer i.Stop() - log.SetOutput(ioutil.Discard) - p := proxy.NewLookup([]string{udp}) state := request.Request{W: &mtest.ResponseWriter{}, Req: new(dns.Msg)} |