diff options
author | Dave Henderson <dhenderson@gmail.com> | 2022-01-25 15:34:35 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-01-25 15:34:35 -0500 |
commit | 741b0502ee7cb0ecbd6dd0517c4f186729f52718 (patch) | |
tree | a8dbe6a5bbb0aa10dc9ab9c2f5111e017e559358 /internal/metrics/metrics_test.go | |
parent | 44e5e9e43f3583f04613bbbb1996e9b5a13a60ac (diff) | |
parent | 7ca5921a87c819f9848ccd7ec786aab0f896be72 (diff) |
Merge pull request #4545 from hairyhenderson/metrics-restrict-http-methods
metrics: Enforce smaller set of method labels
Diffstat (limited to 'internal/metrics/metrics_test.go')
-rw-r--r-- | internal/metrics/metrics_test.go | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/internal/metrics/metrics_test.go b/internal/metrics/metrics_test.go new file mode 100644 index 0000000..c3f5965 --- /dev/null +++ b/internal/metrics/metrics_test.go @@ -0,0 +1,28 @@ +package metrics + +import ( + "strings" + "testing" +) + +func TestSanitizeMethod(t *testing.T) { + tests := []struct { + method string + expected string + }{ + {method: "get", expected: "GET"}, + {method: "POST", expected: "POST"}, + {method: "OPTIONS", expected: "OPTIONS"}, + {method: "connect", expected: "CONNECT"}, + {method: "trace", expected: "TRACE"}, + {method: "UNKNOWN", expected: "OTHER"}, + {method: strings.Repeat("ohno", 9999), expected: "OTHER"}, + } + + for _, d := range tests { + actual := SanitizeMethod(d.method) + if actual != d.expected { + t.Errorf("Not same: expected %#v, but got %#v", d.expected, actual) + } + } +} |