diff --git a/internal/sdk/cloudian/sdk_test.go b/internal/sdk/cloudian/sdk_test.go index 7376759..47247ab 100644 --- a/internal/sdk/cloudian/sdk_test.go +++ b/internal/sdk/cloudian/sdk_test.go @@ -115,9 +115,9 @@ func TestListUserCredentials(t *testing.T) { } func TestListUsers(t *testing.T) { - var expected []UserID + var expected []User for i := 0; i < 500; i++ { - expected = append(expected, UserID{GroupID: "QA", UserID: strconv.Itoa(i)}) + expected = append(expected, User{UserID: UserID{GroupID: "QA", UserID: strconv.Itoa(i)}}) } cloudianClient, testServer := mockBy(func(w http.ResponseWriter, r *http.Request) { @@ -169,6 +169,10 @@ func TestClient_GetUser(t *testing.T) { client, testServer := mockBy(func(w http.ResponseWriter, r *http.Request) { userId := r.URL.Query().Get("userId") statusCode, _ := strconv.Atoi(userId) + if statusCode == http.StatusOK { + // Only asserting error for now, so user can be empty + json.NewEncoder(w).Encode(User{}) + } w.WriteHeader(statusCode) }) defer testServer.Close() @@ -176,6 +180,7 @@ func TestClient_GetUser(t *testing.T) { for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { _, err := client.GetUser(context.Background(), tt.user) + // FIXME: Assert user as well, not only error if !errors.Is(err, tt.wantErr) { t.Errorf("GetUser() error = %v, wantErr %v", err, tt.wantErr) }