#1522: fixed the typo and test

Author: callmejim1226

src/backend/src/services/projects.services.ts

@@ -607,7 +607,7 @@ export default class ProjectsService {
    * Create a new material type
    * @param name the name of the new material type
    * @param submitter the user who is creating the material type
-   * @throws if the submitter is a leader or the material type with given name already exists
+   * @throws if the submitter is not a leader or the material type with the given name already exists
    */
   static async createMaterialType(name: string, submitter: User): Promise<Material_Type> {
     if (!isLeadership(submitter.role)) throw new AccessDeniedAdminOnlyException('create material type');

src/backend/tests/projects.test.ts

@@ -260,16 +260,16 @@ describe('Projects', () => {
       );
     });
 
-    test('Create material type fails if the material type with the given name already exists'),
-      async () => {
-        vi.spyOn(prisma.material_Type, 'findUnique').mockResolvedValue(toolMaterial);
+    test('Create material type fails if the material type with the given name already exists', async () => {
+      vi.spyOn(prisma.material_Type, 'findUnique').mockResolvedValue(toolMaterial);
 
-        await expect(ProjectsService.createMaterialType('NERSoftwareTools', batman)).rejects.toThrow(
-          new HttpException(400, 'The following material type alraedy exists: NERSoftwareTools')
-        );
-      };
+      await expect(ProjectsService.createMaterialType('NERSoftwareTools', batman)).rejects.toThrow(
+        new HttpException(400, 'The following material type alraedy exists: NERSoftwareTools')
+      );
+    });
 
     test('Create material type works', async () => {
+      vi.spyOn(prisma.material_Type, 'findUnique').mockResolvedValue(null);
       vi.spyOn(prisma.material_Type, 'create').mockResolvedValue(toolMaterial);
 
       const materialType = await ProjectsService.createMaterialType('NERSoftwareTools', batman);