cleaned up comments

dkrohmer · Sep 26, 2024 · 8d89f5d · 8d89f5d
1 parent 1ca90e5
commit 8d89f5d
Show file tree

Hide file tree

Showing 272 changed files with 801 additions and 2,704 deletions.
diff --git a/src/__tests__/main/controllers/IncrementController.test.ts b/src/__tests__/main/controllers/IncrementController.test.ts
@@ -60,7 +60,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-all-increments correctly', async () => {
@@ -189,7 +188,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-all-increments with default sortby', async () => {
@@ -251,7 +249,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-increment-by-id nulled', async () => {
@@ -264,7 +261,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-latest-increment correctly', async () => {
@@ -298,7 +294,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-latest-increment nulled', async () => {
@@ -310,7 +305,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle update-increment correctly', async () => {
@@ -346,7 +340,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle update-increment nulled', async () => {
@@ -359,7 +352,6 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle delete-increment correctly', async () => {
@@ -380,6 +372,5 @@ describe('IncrementController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 });
diff --git a/src/__tests__/main/controllers/ModelController.test.ts b/src/__tests__/main/controllers/ModelController.test.ts
@@ -59,7 +59,6 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-all-models correctly', async () => {
@@ -188,7 +187,6 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-model-by-id correctly', async () => {
@@ -222,7 +220,6 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-model-by-id nulled', async () => {
@@ -235,7 +232,6 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle update-model correctly', async () => {
@@ -271,7 +267,6 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle update-model nulled', async () => {
@@ -284,7 +279,6 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle delete-model correctly', async () => {
@@ -305,6 +299,5 @@ describe('ModelController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 });
diff --git a/src/__tests__/main/controllers/ProductController.test.ts b/src/__tests__/main/controllers/ProductController.test.ts
@@ -190,7 +190,6 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-product-by-id correctly', async () => {
@@ -224,7 +223,6 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle get-product-by-id nulled', async () => {
@@ -237,7 +235,6 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle create-product correctly', async () => {
@@ -280,7 +277,6 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle update-product correctly', async () => {
@@ -329,7 +325,6 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle update-product nulled', async () => {
@@ -347,7 +342,6 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 
   it('should handle delete-product correctly', async () => {
@@ -368,6 +362,5 @@ describe('ProductController', () => {
     });
 
     expect(response).toBeUndefined();
-    // expect(console.error).toHaveBeenCalledWith(error); // Uncomment if you want to check console.error call
   });
 });
diff --git a/src/__tests__/main/controllers/VersionController.test.ts b/src/__tests__/main/controllers/VersionController.test.ts
@@ -71,8 +71,6 @@ describe('VersionController', () => {
       new Error('Model not found'),
     );
 
-    // const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
-
     const response = await ipcRenderer.invoke('create-version', {
       modelId: null,
       payload: { graph: { graph: {} } },
@@ -176,8 +174,6 @@ describe('VersionController', () => {
     const error = new Error('Fetch error');
     versionService.getAllVersions.mockRejectedValue(error);
 
-    // const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation();
-
     const response = await ipcRenderer.invoke('get-all-versions', {
       sort: 'asc',
       sortby: 'createdAt',
@@ -275,7 +271,6 @@ describe('VersionController', () => {
 
   it('should handle get-latest-version by model ID with nulled modelId', async () => {
     const error = new Error('No modelId provided');
-    // versionService.getLatestVersionByModelId.mockResolvedValue(null);
 
     const response = await ipcRenderer.invoke('get-latest-version', {
       modelId: null,

diff --git a/src/__tests__/main/helpers/entityBuilder.test.ts b/src/__tests__/main/helpers/entityBuilder.test.ts
@@ -1,12 +1,10 @@
-// __tests__/entityBuilder.test.ts
 import { buildProductEntity } from '../../../main/helpers/entityBuilder';
 import { Product } from '../../../main/models/Product';
 import { Responsible } from '../../../main/models/Responsible';
 import { Increment } from '../../../main/models/Increment';
 import { Model } from '../../../main/models/Model';
 import { Version } from '../../../main/models/Version';
 
-// Mock the model classes to avoid importing TypeORM and database dependencies
 jest.mock('../../../main/models/Product');
 jest.mock('../../../main/models/Responsible');
 jest.mock('../../../main/models/Increment');
@@ -182,13 +180,11 @@ describe('buildProductEntity', () => {
 
   it('should truncate the product name to fit within 250 characters including the baseline suffix', () => {
     const productData = new Product();
-    // Create a name that is 240 characters long
-    productData.name = 'A'.repeat(240); // 240 characters
+    productData.name = 'A'.repeat(240);
     productData.startsAt = new Date('2024-01-01T00:00:00.000Z');
 
     const product = buildProductEntity(productData);
 
-    // The name should be 240 characters + " - Baseline" (11 characters) = 251 characters total
-    expect(product.increments[0].name).toBe(`${'A'.repeat(239)} - Baseline`); // Truncated to fit 250 characters total
+    expect(product.increments[0].name).toBe(`${'A'.repeat(239)} - Baseline`);
   });
 });
diff --git a/src/__tests__/main/helpers/isDataUriPng.test.ts b/src/__tests__/main/helpers/isDataUriPng.test.ts
@@ -1,5 +1,5 @@
 import { validate } from 'class-validator';
-import { IsDataUriPng } from '../../../main/helpers/isDataUriPng'; // Adjust the path accordingly
+import { IsDataUriPng } from '../../../main/helpers/isDataUriPng';
 
 class TestDto {
   @IsDataUriPng()
@@ -9,9 +9,9 @@ class TestDto {
 describe('IsDataUriPng', () => {
   it('should validate a valid PNG Data URI', async () => {
     const dto = new TestDto();
-    dto.thumbnail = 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUA'; // Valid PNG Data URI
+    dto.thumbnail = 'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUA';
 
     const errors = await validate(dto);
-    expect(errors.length).toBe(0); // Expect no validation errors
+    expect(errors.length).toBe(0);
   });
 });
diff --git a/src/__tests__/main/helpers/isJsonString.test.ts b/src/__tests__/main/helpers/isJsonString.test.ts
@@ -9,9 +9,9 @@ class TestDto {
 describe('IsJsonString', () => {
   it('should validate a valid stringified JSON', async () => {
     const dto = new TestDto();
-    dto.payload = JSON.stringify({ key: 'value' }); // Valid JSON
+    dto.payload = JSON.stringify({ key: 'value' });
 
     const errors = await validate(dto);
-    expect(errors.length).toBe(0); // Expect no validation errors
+    expect(errors.length).toBe(0);
   });
 });
diff --git a/src/__tests__/main/menu.test.ts b/src/__tests__/main/menu.test.ts
@@ -145,7 +145,6 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle Full Screen',
     );
 
-    // Simulate the mainWindow being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(true);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(false);
@@ -160,7 +159,6 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle Full Screen',
     );
 
-    // Simulate the mainWindow being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(true);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(false);
@@ -235,7 +233,6 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle &Full Screen',
     );
 
-    // Simulate the mainWindow being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(true);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(false);
@@ -250,15 +247,12 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle &Full Screen',
     );
 
-    // Ensure toggleFullScreenItem is defined before accessing its click property
     expect(toggleFullScreenItem).toBeDefined();
 
-    // Simulate the mainWindow initially not being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(false);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(true);
 
-    // Simulate the mainWindow being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(true);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(false);
@@ -412,7 +406,6 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle &Full Screen',
     );
 
-    // Simulate the mainWindow initially not being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(false);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(true);
@@ -427,15 +420,12 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle &Full Screen',
     );
 
-    // Ensure toggleFullScreenItem is defined before accessing its click property
     expect(toggleFullScreenItem).toBeDefined();
 
-    // Simulate the mainWindow initially not being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(false);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(true);
 
-    // Simulate the mainWindow being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(true);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(false);
@@ -450,15 +440,12 @@ describe('MenuBuilder', () => {
       (item) => item.label === 'Toggle &Full Screen',
     );
 
-    // Ensure toggleFullScreenItem is defined before accessing its click property
     expect(toggleFullScreenItem).toBeDefined();
 
-    // Simulate the mainWindow initially not being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(false);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(true);
 
-    // Simulate the mainWindow being in full screen
     (mainWindow.isFullScreen as jest.Mock).mockReturnValue(true);
     toggleFullScreenItem.click();
     expect(mainWindow.setFullScreen).toHaveBeenCalledWith(false);
@@ -469,7 +456,7 @@ describe('MenuBuilder', () => {
       value: 'darwin',
     });
 
-    process.env.NODE_ENV = 'production'; // Ensure environment is production for consistent template
+    process.env.NODE_ENV = 'production';
 
     const menu = menuBuilder.buildMenu();
 
@@ -483,7 +470,7 @@ describe('MenuBuilder', () => {
       value: 'win32',
     });
 
-    process.env.NODE_ENV = 'development'; // Ensure environment is development for consistent template
+    process.env.NODE_ENV = 'development';
 
     const menu = menuBuilder.buildMenu();