Remove comments

cabinetoffice · Jul 5, 2024 · f540018 · f540018
1 parent c115ccd
commit f540018
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 107 deletions.
diff --git a/src/config/index.ts b/src/config/index.ts
@@ -32,9 +32,3 @@ export const FEATURE_FLAG_ENABLE_AUTH = getEnvironmentValue(
 export const ID = 'id';
 export const PARAM_ID = `/:${ID}`;
 
-// export function VALIDATION_TEST(_VALIDATION_TEST: any) {
-//     throw new Error('Function not implemented.');
-// }
-// export function VALIDATION_TEST_URL(VALIDATION_TEST_URL: any, authentication: any, get: any) {
-// 	throw new Error('Function not implemented.');
-// }
diff --git a/test/unit/middleware/validation.middleware.spec.ts b/test/unit/middleware/validation.middleware.spec.ts
@@ -96,104 +96,3 @@ describe('Validation Middleware test suites', () => {
         expect(logErrorMock).toHaveBeenCalledWith(MOCK_ERROR.message);
     });
 });
-// jest.mock('express-validator');
-// jest.mock('../../../src/utils/validateFilepath.ts');
-// jest.mock('../../../src/utils/logger');
-
-// import {
-//     describe,
-//     expect,
-//     test,
-//     jest,
-//     afterEach,
-//     beforeEach,
-// } from '@jest/globals';
-// import { Request } from 'express';
-// import { validationResult } from 'express-validator';
-
-// import * as config from '../../../src/config';
-// import { validateFilepath } from '../../../src/utils/validateFilepath';
-// import { mockLogInfo, mockLogErrorRequest } from '../../mock/log.mock';
-// import { checkValidations } from '../../../src/middleware/validation.middleware';
-// import { ErrorMessages } from '../../../src/validation/error.messages';
-// import { MOCK_ERROR, MOCK_POST_INFO } from '../../mock/data';
-// import { mockNext, mockResponse } from '../../mock/express.mock';
-// import { mockID } from '../../mock/session.mock';
-
-// const validationResultMock = validationResult as unknown as jest.Mock;
-// const validateFilepathMock = validateFilepath as jest.Mock<
-// 	typeof validateFilepath
-// >;
-
-// const mockRequest = () => {
-//     const req = {} as Request;
-//     req.path = config.INFO_URL;
-//     req.body = MOCK_POST_INFO;
-//     req.params = { [config.ID]: mockID };
-//     return req;
-// };
-
-// describe('Validation Middleware test suites', () => {
-//     let res: any, req: any;
-//     beforeEach(() => {
-//         res = mockResponse();
-//         req = mockRequest();
-//     });
-
-//     afterEach(() => {
-//         jest.resetAllMocks();
-//     });
-
-//     test('Should call next if errorList is empty', () => {
-//         validationResultMock.mockImplementationOnce(() => {
-//             return { isEmpty: () => true };
-//         });
-//         checkValidations(req, res, mockNext);
-
-//         expect(mockLogInfo).toBeCalledTimes(0);
-//         expect(mockLogErrorRequest).toBeCalledTimes(0);
-//         expect(mockNext).toHaveBeenCalledTimes(1);
-//     });
-
-//     test(`should call res.render with ${config.LANDING_PAGE} view if errorList is not empty and id empty`, () => {
-//         const fieldKey = 'test';
-//         validationResultMock.mockImplementationOnce(() => {
-//             return {
-//                 isEmpty: () => false,
-//                 array: () => [{ path: fieldKey, msg: ErrorMessages.TEST_INFO_ERROR }],
-//             };
-//         });
-//         validateFilepathMock.mockImplementationOnce(() => config.INFO_URL);
-//         req.body[fieldKey] = '';
-//         req.params[config.ID] = '';
-//         checkValidations(req, res, mockNext);
-
-//         expect(mockLogInfo).toHaveBeenCalledTimes(1);
-//         expect(mockLogInfo).toHaveBeenCalledWith(
-//             `Validation error on ${config.LANDING_PAGE} page` // need to specify page
-//         );
-
-//         expect(res.render).toHaveBeenCalledTimes(1);
-//         expect(res.render).toHaveBeenCalledWith(config.LANDING_PAGE, {
-//             ...req.body,
-//             [config.ID]: '',
-//             errors: {
-//                 errorList: [
-//                     { href: `#${fieldKey}`, text: ErrorMessages.TEST_INFO_ERROR },
-//                 ],
-//                 [fieldKey]: { text: ErrorMessages.TEST_INFO_ERROR },
-//             },
-//         });
-//     });
-
-//     test('should catch the error log error message and call next(err)', () => {
-//         validationResultMock.mockImplementationOnce(() => {
-//             throw new Error(MOCK_ERROR.message);
-//         });
-//         checkValidations(req, res, mockNext);
-
-//         expect(mockNext).toHaveBeenCalledTimes(1);
-//         expect(mockLogErrorRequest).toHaveBeenCalledTimes(1);
-//         expect(mockLogErrorRequest).toHaveBeenCalledWith(req, MOCK_ERROR.message);
-//     });
-// });