Skip to content

Commit

Permalink
Fix types in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
johnlanda committed Feb 7, 2024
1 parent f967efd commit d10c600
Showing 1 changed file with 12 additions and 9 deletions.
21 changes: 12 additions & 9 deletions path_config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -298,9 +298,9 @@ func TestConfig_JWKS_Pairs_Update(t *testing.T) {
data := map[string]interface{}{
"jwks_url": "",
"jwks_ca_pem": "",
"jwks_pairs": []map[string]interface{}{
{"jwks_url": s.server.URL + "/certs", "jwks_ca_pem": cert},
{"jwks_url": s2.server.URL + "/certs", "jwks_ca_pem": cert2},
"jwks_pairs": []interface{}{
map[string]interface{}{"jwks_url": s.server.URL + "/certs", "jwks_ca_pem": cert},
map[string]interface{}{"jwks_url": s2.server.URL + "/certs", "jwks_ca_pem": cert2},
},
"oidc_discovery_url": "",
"oidc_discovery_ca_pem": "",
Expand Down Expand Up @@ -366,9 +366,9 @@ func TestConfig_JWKS_Pairs_Update_Invalid(t *testing.T) {
data := map[string]interface{}{
"jwks_url": "",
"jwks_ca_pem": "",
"jwks_pairs": []map[string]interface{}{
{"jwks_url": s.server.URL + "/certs_missing", "jwks_ca_pem": cert},
{"jwks_url": s2.server.URL + "/certs", "jwks_ca_pem": cert2},
"jwks_pairs": []interface{}{
map[string]interface{}{"jwks_url": s.server.URL + "/certs_missing", "jwks_ca_pem": cert},
map[string]interface{}{"jwks_url": s2.server.URL + "/certs", "jwks_ca_pem": cert2},
},
"oidc_discovery_url": "",
"oidc_discovery_ca_pem": "",
Expand Down Expand Up @@ -397,10 +397,13 @@ func TestConfig_JWKS_Pairs_Update_Invalid(t *testing.T) {
t.Fatalf("got unexpected error: %v", resp.Error())
}

// remove the /certs_missing url from the map
delete(data["jwks_pairs"].(map[string]interface{}), s.server.URL+"/certs_missing")
// remove the /certs_missing url from the config
newPairs := []interface{}{
map[string]interface{}{"jwks_url": s.server.URL + "/certs_invalid", "jwks_ca_pem": cert},
map[string]interface{}{"jwks_url": s2.server.URL + "/certs", "jwks_ca_pem": cert2},
}

data["jwks_pairs"].(map[string]interface{})[s.server.URL+"/certs_invalid"] = cert
data["jwks_pairs"] = newPairs

req = &logical.Request{
Operation: logical.UpdateOperation,
Expand Down

0 comments on commit d10c600

Please sign in to comment.