github/flagger
1
0
Fork 0
mirror of https://github.com/fluxcd/flagger.git synced 2026-04-15 06:57:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #587 from weaveworks/redundant-assertion

Browse Source 
pkg/metrics/providers: remove redundant assertion in prometheus test
This commit is contained in:
Takeshi Yoneda
2020-05-14 18:07:50 +09:00
committed by GitHub
parent a6b8d19629 772099f073
commit 5bd6906c32
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/metrics/providers/prometheus_test.go
-6
View File
@@ -90,9 +90,6 @@ func TestPrometheusProvider_RunQueryWithBasicAuth(t *testing.T) {
promql := r.URL.Query()["query"][0]
assert.Equal(t, expected, promql)
if assert.Contains(t, r.Header, "Authorization") {
}
header, ok := r.Header["Authorization"]
if assert.True(t, ok, "Authorization header not found") {
assert.True(t, strings.Contains(header[0], "Basic"), "Basic authorization header not found")
@@ -174,9 +171,6 @@ func TestPrometheusProvider_IsOnline(t *testing.T) {
promql := r.URL.Query()["query"][0]
assert.Equal(t, expected, promql)
if assert.Contains(t, r.Header, "Authorization") {
}
header, ok := r.Header["Authorization"]
if assert.True(t, ok, "Authorization header not found") {
assert.True(t, strings.Contains(header[0], "Basic"), "Basic authorization header not found")