refactor(auth): update error handling for missing Google refresh tokens

- Replaced instances of UserError.MissingGoogleField with UserError.MissingGoogleRefreshToken across multiple files to improve clarity in error reporting.
- Updated tests in google.auth.service.test.ts, google.guard.test.ts, and google.required.middleware.test.ts to reflect the new error handling logic.
- Enhanced UserError definitions in user.errors.ts to provide more specific feedback regarding missing Google refresh tokens.

Author: tyler-dane

packages/backend/src/auth/services/google.auth.service.test.ts

@@ -34,7 +34,7 @@ describe("getGcalClient", () => {
     mockFindCompassUserBy.mockResolvedValue(userWithoutGoogle);
 
     await expect(getGcalClient(userId)).rejects.toMatchObject({
-      description: UserError.MissingGoogleField.description,
+      description: UserError.MissingGoogleRefreshToken.description,
     });
 
     expect(mockFindCompassUserBy).toHaveBeenCalledWith("_id", userId);
@@ -59,7 +59,7 @@ describe("getGcalClient", () => {
     mockFindCompassUserBy.mockResolvedValue(userWithEmptyGoogle);
 
     await expect(getGcalClient(userId)).rejects.toMatchObject({
-      description: UserError.MissingGoogleField.description,
+      description: UserError.MissingGoogleRefreshToken.description,
     });
   });
 

packages/backend/src/auth/services/google.auth.service.ts

@@ -45,7 +45,7 @@ export const getGAuthClientForUser = async (
 
     if (!_user?.google?.gRefreshToken) {
       throw error(
-        UserError.MissingGoogleField,
+        UserError.MissingGoogleRefreshToken,
         "User has not connected Google Calendar",
       );
     }
@@ -114,7 +114,7 @@ export const getGcalClient = async (userId: string): Promise<gCalendar> => {
 
   if (!user.google?.gRefreshToken) {
     throw error(
-      UserError.MissingGoogleField,
+      UserError.MissingGoogleRefreshToken,
       "User has not connected Google Calendar",
     );
   }

packages/backend/src/common/errors/user/user.errors.ts

@@ -3,7 +3,7 @@ import { ErrorMetadata } from "@backend/common/types/error.types";
 
 interface UserErrors {
   InvalidValue: ErrorMetadata;
-  MissingGoogleField: ErrorMetadata;
+  MissingGoogleRefreshToken: ErrorMetadata;
   MissingUserIdField: ErrorMetadata;
   UserNotFound: ErrorMetadata;
 }
@@ -14,7 +14,7 @@ export const UserError: UserErrors = {
     status: Status.BAD_REQUEST,
     isOperational: true,
   },
-  MissingGoogleField: {
+  MissingGoogleRefreshToken: {
     description: "User is missing a Google refresh token",
     status: Status.BAD_REQUEST,
     isOperational: true,

packages/backend/src/common/guards/google.guard.test.ts

@@ -142,7 +142,7 @@ describe("google.guard", () => {
       mockFindCompassUserBy.mockResolvedValue(userWithoutGoogle);
 
       await expect(requireGoogleConnection(userId)).rejects.toMatchObject({
-        description: UserError.MissingGoogleField.description,
+        description: UserError.MissingGoogleRefreshToken.description,
       });
     });
   });

packages/backend/src/common/guards/google.guard.ts

@@ -16,7 +16,7 @@ export const requireGoogleConnection = async (
   }
   if (!(await isGoogleConnected(userId))) {
     throw error(
-      UserError.MissingGoogleField,
+      UserError.MissingGoogleRefreshToken,
       "User has not connected Google Calendar",
     );
   }

packages/backend/src/common/middleware/google.required.middleware.test.ts

@@ -76,7 +76,7 @@ describe("google.required.middleware", () => {
       };
       const baseError = new BaseError(
         "User has not connected Google Calendar",
-        UserError.MissingGoogleField.description,
+        UserError.MissingGoogleRefreshToken.description,
         Status.BAD_REQUEST,
         true,
       );