Add encodeID for mongo and add FindUserInfoByID test

server/backend/database/memory/database_test.go

@@ -56,6 +56,10 @@ func TestDB(t *testing.T) {
 		testcases.RunListUserInfosTest(t, db)
 	})
 
+	t.Run("FindUserInfoByID test", func(t *testing.T) {
+		testcases.RunFindUserInfoByIDTest(t, db)
+	})
+
 	t.Run("FindProjectInfoBySecretKey test", func(t *testing.T) {
 		testcases.RunFindProjectInfoBySecretKeyTest(t, db)
 	})

server/backend/database/mongo/client.go

@@ -468,23 +468,6 @@ func (c *Client) FindUserInfo(ctx context.Context, username string) (*database.U
 	return &userInfo, nil
 }
 
-// FindUserInfoByID returns a user by ID.
-func (c *Client) FindUserInfoByID(ctx context.Context, id string) (*database.UserInfo, error) {
-	result := c.collection(colUsers).FindOne(ctx, bson.M{
-		"_id": id,
-	})
-
-	userInfo := database.UserInfo{}
-	if err := result.Decode(&userInfo); err != nil {
-		if err == mongo.ErrNoDocuments {
-			return nil, fmt.Errorf("%s: %w", id, database.ErrUserNotFound)
-		}
-		return nil, fmt.Errorf("decode user info: %w", err)
-	}
-
-	return &userInfo, nil
-}
-
 // ListUserInfos returns all users.
 func (c *Client) ListUserInfos(
 	ctx context.Context,
@@ -502,6 +485,28 @@ func (c *Client) ListUserInfos(
 	return infos, nil
 }
 
+// FindUserInfoByID returns a user by ID.
+func (c *Client) FindUserInfoByID(ctx context.Context, clientID types.ID) (*database.UserInfo, error) {
+	encodedClientID, err := encodeID(clientID)
+	if err != nil {
+		return nil, err
+	}
+
+	result := c.collection(colUsers).FindOne(ctx, bson.M{
+		"_id": encodedClientID,
+	})
+
+	userInfo := database.UserInfo{}
+	if err := result.Decode(&userInfo); err != nil {
+		if err == mongo.ErrNoDocuments {
+			return nil, fmt.Errorf("%s: %w", clientID, database.ErrUserNotFound)
+		}
+		return nil, fmt.Errorf("decode user info: %w", err)
+	}
+
+	return &userInfo, nil
+}
+
 // ActivateClient activates the client of the given key.
 func (c *Client) ActivateClient(ctx context.Context, projectID types.ID, key string) (*database.ClientInfo, error) {
 	encodedProjectID, err := encodeID(projectID)

server/backend/database/mongo/client_test.go

@@ -73,6 +73,10 @@ func TestClient(t *testing.T) {
 		testcases.RunListUserInfosTest(t, cli)
 	})
 
+	t.Run("FindUserInfoByID test", func(t *testing.T) {
+		testcases.RunFindUserInfoByIDTest(t, cli)
+	})
+
 	t.Run("FindProjectInfoBySecretKey test", func(t *testing.T) {
 		testcases.RunFindProjectInfoBySecretKeyTest(t, cli)
 	})

server/backend/database/testcases/testcases.go

@@ -295,6 +295,24 @@ func RunListUserInfosTest(t *testing.T, db database.Database) {
 	})
 }
 
+// RunFindUserInfoByIDTest runs the FindUserInfoByID test for the given db.
+func RunFindUserInfoByIDTest(t *testing.T, db database.Database) {
+	t.Run("RunFindUserInfoByID test", func(t *testing.T) {
+		ctx := context.Background()
+
+		username := "findUserInfoTestAccount"
+		password := "temporary-password"
+
+		user, _, err := db.EnsureDefaultUserAndProject(ctx, username, password, clientDeactivateThreshold)
+		assert.NoError(t, err)
+
+		info1, err := db.FindUserInfoByID(ctx, user.ID)
+		assert.NoError(t, err)
+
+		assert.Equal(t, user.ID, info1.ID)
+	})
+}
+
 // RunActivateClientDeactivateClientTest runs the ActivateClient and DeactivateClient tests for the given db.
 func RunActivateClientDeactivateClientTest(t *testing.T, db database.Database, projectID types.ID) {
 	t.Run("activate and find client test", func(t *testing.T) {