#1576 Threw an exception when there is an existing phone number in the user database and added 1 test

Author: AShyamkumar19

src/backend/src/services/users.services.ts

@@ -289,6 +289,15 @@ export default class UsersService {
     zipcode: string,
     phoneNumber: string
   ): Promise<string> {
+
+    const existingUser = await prisma.user_Secure_Settings.findFirst({
+      where: { phoneNumber }
+    });
+
+    if (existingUser) {
+      throw new HttpException(400, 'Phone number already in use');
+    };
+
     const newUserSecureSettings = await prisma.user_Secure_Settings.upsert({
       where: { userId: user.userId },
       update: {

src/backend/tests/users.test.ts

@@ -10,7 +10,7 @@ import {
 } from './test-data/users.test-data';
 import { Role } from '@prisma/client';
 import UsersService from '../src/services/users.services';
-import { AccessDeniedException, NotFoundException } from '../src/utils/errors.utils';
+import { AccessDeniedException, HttpException, NotFoundException } from '../src/utils/errors.utils';
 import userTransformer from '../src/transformers/user.transformer';
 
 describe('Users', () => {
@@ -119,5 +119,20 @@ describe('Users', () => {
 
       expect(res).toBe(batmanSecureSettings.userSecureSettingsId);
     });
+
+    test('setting same phone number does not work', async () => {
+      vi.spyOn(prisma.user_Secure_Settings, 'findFirst').mockResolvedValue(batmanSecureSettings);
+      await expect(() =>
+        UsersService.setUserSecureSettings(
+          batman,
+          batmanSecureSettings.nuid,
+          batmanSecureSettings.street,
+          batmanSecureSettings.city,
+          batmanSecureSettings.state,
+          batmanSecureSettings.zipcode,
+          batmanSecureSettings.phoneNumber
+        )
+      ).rejects.toThrow(new HttpException(400, 'Phone number already in use'));
+    });
   });
 });