status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-02-18 00:27:04 +00:00
Code Issues Projects Releases Wiki Activity

add test cases for h2ping_use_tls default behavior

Browse Source
This commit is contained in:
tarat44 2021-10-09 17:12:52 -04:00
parent e52f5071d0
commit 3fe637156c
2 changed files with 40 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
agent/config/runtime_test.go
View File

@ -2375,6 +2375,42 @@ func TestLoad_IntegrationWithFlags(t *testing.T) {
rt.DataDir = dataDir rt.DataDir = dataDir
}, },
}) })
run(t, testCase{
desc: "h2ping check without h2ping_use_tls set",
args: []string{
`-data-dir=` + dataDir,
},
json: []string{
`{ "check": { "name": "a", "h2ping": "localhost:55555", "interval": "5s" } }`,
},
hcl: []string{
`check = { name = "a" h2ping = "localhost:55555" interval = "5s" }`,
},
expected: func(rt *RuntimeConfig) {
rt.Checks = []*structs.CheckDefinition{
{Name: "a", H2PING: "localhost:55555", H2PingUseTLS: true, OutputMaxSize: checks.DefaultBufSize, Interval: 5 * time.Second},
}
rt.DataDir = dataDir
},
})
run(t, testCase{
desc: "h2ping check with h2ping_use_tls set to false",
args: []string{
`-data-dir=` + dataDir,
},
json: []string{
`{ "check": { "name": "a", "h2ping": "localhost:55555", "h2ping_use_tls": false, "interval": "5s" } }`,
},
hcl: []string{
`check = { name = "a" h2ping = "localhost:55555" h2ping_use_tls = false interval = "5s" }`,
},
expected: func(rt *RuntimeConfig) {
rt.Checks = []*structs.CheckDefinition{
{Name: "a", H2PING: "localhost:55555", H2PingUseTLS: false, OutputMaxSize: checks.DefaultBufSize, Interval: 5 * time.Second},
}
rt.DataDir = dataDir
},
})
run(t, testCase{ run(t, testCase{
desc: "alias check with no node", desc: "alias check with no node",
args: []string{ args: []string{

8
agent/http_decode_test.go
View File

@ -702,7 +702,7 @@ func h2pingUseTLSEqFn(out interface{}, want interface{}) error {
return nil return nil
} }
var h2pingUseTLSFields = []string{`"H2PingUseTLS": %s`, `"h2ping_use_tls": %s`} var h2pingUseTLSFields = []string{`"H2PING": "testing"`, `"H2PingUseTLS": %s`, `"h2ping_use_tls": %s`}
var translateH2PingUseTLS = []translateKeyTestCase{ var translateH2PingUseTLS = []translateKeyTestCase{
{ {
desc: "H2PingUseTLS: both set", desc: "H2PingUseTLS: both set",
@ -715,21 +715,21 @@ var translateH2PingUseTLS = []translateKeyTestCase{
desc: "H2PingUseTLS:: first set", desc: "H2PingUseTLS:: first set",
in: []interface{}{`false`}, in: []interface{}{`false`},
want: false, want: false,
jsonFmtStr: "{" + h2pingUseTLSFields[0] + "}", jsonFmtStr: "{" + strings.Join(h2pingUseTLSFields[0:2], ",") + "}",
equalityFn: h2pingUseTLSEqFn, equalityFn: h2pingUseTLSEqFn,
}, },
{ {
desc: "H2PingUseTLS: second set", desc: "H2PingUseTLS: second set",
in: []interface{}{`false`}, in: []interface{}{`false`},
want: false, want: false,
jsonFmtStr: "{" + h2pingUseTLSFields[1] + "}", jsonFmtStr: "{" + h2pingUseTLSFields[0] + "," + h2pingUseTLSFields[2] + "}",
equalityFn: h2pingUseTLSEqFn, equalityFn: h2pingUseTLSEqFn,
}, },
{ {
desc: "H2PingUseTLS: neither set", desc: "H2PingUseTLS: neither set",
in: []interface{}{}, in: []interface{}{},
want: true, // zero value want: true, // zero value
jsonFmtStr: "{" + `"h2ping":"testing"` + "}", jsonFmtStr: "{" + h2pingUseTLSFields[0] + "}",
equalityFn: h2pingUseTLSEqFn, equalityFn: h2pingUseTLSEqFn,
}, },
} }