From 67932239489d955a0af69b41d78282ad5ed29a34 Mon Sep 17 00:00:00 2001 From: mrasu Date: Sat, 9 Feb 2019 23:39:51 +0900 Subject: [PATCH] Fix etcd_cache_test to runnable multiple times. (#2546) Currently, when you run `TestEtcdCredentials` at etcd_credentials_test.go multiple times without clearing data of etcd, you will get following errors. ``` etcd_credentials_test.go:38: Failed to create root role: etcdserver: role name already exists etcd_credentials_test.go:41: Failed to create user: etcdserver: user name already exists ``` --- test/etcd_credentials_test.go | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/test/etcd_credentials_test.go b/test/etcd_credentials_test.go index 70586e4d1..ebcc20b16 100644 --- a/test/etcd_credentials_test.go +++ b/test/etcd_credentials_test.go @@ -37,9 +37,21 @@ func TestEtcdCredentials(t *testing.T) { if _, err := etc.Client.RoleAdd(ctx, "root"); err != nil { t.Errorf("Failed to create root role: %s", err) } + defer func() { + if _, err := etc.Client.RoleDelete(ctx, "root"); err != nil { + t.Errorf("Failed to delete root role: %s", err) + } + }() + if _, err := etc.Client.UserAdd(ctx, username, password); err != nil { t.Errorf("Failed to create user: %s", err) } + defer func() { + if _, err := etc.Client.UserDelete(ctx, username); err != nil { + t.Errorf("Failed to delete user: %s", err) + } + }() + if _, err := etc.Client.UserGrantRole(ctx, username, "root"); err != nil { t.Errorf("Failed to assign role to root user: %v", err) }