Update tests

hashicorp · Oct 1, 2018 · aed4b1b · aed4b1b
1 parent 13912c1
commit aed4b1b
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 0 deletions.
diff --git a/vault/identity_store_aliases_test.go b/vault/identity_store_aliases_test.go
@@ -66,6 +66,19 @@ func TestIdentityStore_CaseInsensitiveEntityAliasName(t *testing.T) {
 	if err != nil || (resp != nil && resp.IsError()) {
 		t.Fatalf("bad: err:%v\nresp: %#v", err, resp)
 	}
+
+	// Ensure that reading the alias returns lower cased alias name
+	resp, err = i.HandleRequest(ctx, &logical.Request{
+		Path:      "entity-alias/id/" + aliasID,
+		Operation: logical.ReadOperation,
+	})
+	if err != nil || (resp != nil && resp.IsError()) {
+		t.Fatalf("bad: err:%v\nresp: %#v", err, resp)
+	}
+	aliasName = resp.Data["name"].(string)
+	if aliasName != strings.ToLower(testAliasName) {
+		t.Fatalf("bad alias name; expected: %q, actual: %q", testAliasName, aliasName)
+	}
 }
 
 // This test is required because MemDB does not take care of ensuring

diff --git a/vault/identity_store_group_aliases_test.go b/vault/identity_store_group_aliases_test.go
@@ -69,6 +69,19 @@ func TestIdentityStore_CaseInsensitiveGroupAliasName(t *testing.T) {
 	if err != nil || (resp != nil && resp.IsError()) {
 		t.Fatalf("bad: err:%v\nresp: %#v", err, resp)
 	}
+
+	// Ensure that reading the alias returns lower cased alias name
+	resp, err = i.HandleRequest(ctx, &logical.Request{
+		Path:      "group-alias/id/" + aliasID,
+		Operation: logical.ReadOperation,
+	})
+	if err != nil || (resp != nil && resp.IsError()) {
+		t.Fatalf("bad: err:%v\nresp: %#v", err, resp)
+	}
+	aliasName = resp.Data["name"].(string)
+	if aliasName != strings.ToLower(testAliasName) {
+		t.Fatalf("bad alias name; expected: %q, actual: %q", testAliasName, aliasName)
+	}
 }
 
 func TestIdentityStore_GroupAliasDeletionOnGroupDeletion(t *testing.T) {