From b2e76001c9fb912ff38245a90c071df50d82f99e Mon Sep 17 00:00:00 2001 From: fengkang01 Date: Wed, 8 Jun 2022 17:55:31 +0800 Subject: [PATCH] Fix: mongoDB datastore can't list special email user(#4104) change the function name from verifyUserValue to verifyValue add test case to test kubeapi.go:87 add delete test case Signed-off-by: fengkang --- .../infrastructure/datastore/kubeapi/kubeapi_test.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/apiserver/infrastructure/datastore/kubeapi/kubeapi_test.go b/pkg/apiserver/infrastructure/datastore/kubeapi/kubeapi_test.go index c64bc7c00..328db6ef2 100644 --- a/pkg/apiserver/infrastructure/datastore/kubeapi/kubeapi_test.go +++ b/pkg/apiserver/infrastructure/datastore/kubeapi/kubeapi_test.go @@ -39,8 +39,9 @@ var _ = Describe("Test kubeapi datastore driver", func() { err := kubeStore.Add(context.TODO(), &model.Application{Name: "kubevela-app", Description: "default"}) Expect(err).ToNot(HaveOccurred()) - err = kubeStore.Add(context.TODO(), &model.Application{Name: "cat@delete", Description: "this is for test special symbol"}) + err = kubeStore.Add(context.TODO(), &model.Application{Name: "can@delete", Description: "this is for test special symbol"}) Expect(err).ToNot(HaveOccurred()) + }) It("Test batch add function", func() { @@ -248,5 +249,9 @@ var _ = Describe("Test kubeapi datastore driver", func() { err = kubeStore.Delete(context.TODO(), &app) equal := cmp.Equal(err, datastore.ErrRecordNotExist, cmpopts.EquateErrors()) Expect(equal).Should(BeTrue()) + + app.Name = "can@delete" + err = kubeStore.Delete(context.TODO(), &app) + Expect(err).ShouldNot(HaveOccurred()) }) })