diff options
| author |  Malcolm Akinje <malcolm.akinje@gmail.com>
| 2018-06-07 11:21:17 -0400 |
|---|---|---|
| committer |  Miek Gieben <miek@miek.nl>
| 2018-06-07 16:21:17 +0100 |
| commit | 18b11fc85156be8b027c61665c80e239324e2ebf (patch) | |
| tree | 1090fef887b2653b8da3bd8781800204f17ed490 /plugin/pkg/replacer/replacer_test.go | |
| parent | 751a08d6a2e3ace62e40ed8170015e7051d0020c (diff) | |
| download | coredns-18b11fc85156be8b027c61665c80e239324e2ebf.tar.gz coredns-18b11fc85156be8b027c61665c80e239324e2ebf.tar.zst coredns-18b11fc85156be8b027c61665c80e239324e2ebf.zip | |
Normalizing Response Duration in Log Plugin (#1860)
* Current stage of the log files. Test need to be done as well as formatting of times.
* Finished testing. All altered classes test pass along with my additions
* Updated the replacer package to print the units as well. May take out.
* Changed the time units to be within the rules. Fixed the test as well.
* Fixed some tests, updated the readme, fixed the replacer class.
* Updates of standardizing only to seconds in response duration. Need to revert README.
* Reverted readme.
* Added a small test in new replacer.
* Changed replacer to inline the strconv for duration.
Diffstat (limited to 'plugin/pkg/replacer/replacer_test.go')
| -rw-r--r-- | plugin/pkg/replacer/replacer_test.go | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/plugin/pkg/replacer/replacer_test.go b/plugin/pkg/replacer/replacer_test.go index 65bb8280f..2fcaafc92 100644 --- a/plugin/pkg/replacer/replacer_test.go +++ b/plugin/pkg/replacer/replacer_test.go @@ -1,6 +1,7 @@ package replacer import ( + "strings" "testing" "github.com/coredns/coredns/plugin/pkg/dnstest" @@ -30,6 +31,9 @@ func TestNewReplacer(t *testing.T) { if v.replacements["{size}"] != "29" { // size of request t.Errorf("Expected size to be 29, got %q", v.replacements["{size}"]) } + if !strings.Contains(v.replacements["{duration}"], "s") { + t.Errorf("Expected units of time to be in seconds") + } default: t.Fatal("Return Value from New Replacer expected pass type assertion into a replacer type\n") |