feat(FN-3611): updated and renamed cache middlewear and added test

libs/common/src/middleware/index.ts

@@ -1,3 +1,3 @@
 export * from './feature-flags';
 export * from './create-validation-middleware-for-schema';
-export * from './set-no-store-cache-control';
+export * from './set-cache-control';

libs/common/src/middleware/set-cache-control.test.ts

@@ -0,0 +1,21 @@
+import request from 'supertest';
+import express from 'express';
+import { HttpStatusCode } from 'axios';
+import { setCacheControl } from './set-cache-control';
+
+const app = express();
+app.use(setCacheControl);
+app.get('/', (_req, res) => res.send());
+
+afterEach(() => {
+  jest.resetAllMocks();
+});
+
+describe('set-no-store-cache-control', () => {
+  it('should set Cache-Control headers correctly', async () => {
+    const response = await request(app).get('/');
+
+    expect(response.headers['cache-control']).toBe('no-cache, no-store, must-revalidate, max-age=0');
+    expect(response.status).toBe(HttpStatusCode.Ok);
+  });
+});

libs/common/src/middleware/set-cache-control.ts

@@ -0,0 +1,15 @@
+import { Request, Response, NextFunction } from 'express';
+
+/**
+ * middleware to set Cache-Control to no-store, no-cache, must-revalidate and max-age=0
+ * stops cached version of a page being rendered when going back to a page (using the browser back button)
+ * ensures the get controller for the page is called
+ * @param _req
+ * @param res
+ * @param next
+ */
+export const setCacheControl = (_req: Request, res: Response, next: NextFunction) => {
+  res.setHeader('Cache-Control', 'no-cache, no-store, must-revalidate, max-age=0');
+
+  next();
+};

trade-finance-manager-ui/server/routes/utilisation-reports/index.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/no-misused-promises */
 import express from 'express';
-import { PDC_TEAM_IDS, validateFeeRecordCorrectionFeatureFlagIsEnabled, setNoStoreCacheControl } from '@ukef/dtfs2-common';
+import { PDC_TEAM_IDS, validateFeeRecordCorrectionFeatureFlagIsEnabled, setCacheControl } from '@ukef/dtfs2-common';
 import { getUtilisationReports } from '../../controllers/utilisation-reports';
 import { updateUtilisationReportStatus } from '../../controllers/utilisation-reports/update-utilisation-report-status';
 import {
@@ -162,7 +162,7 @@ utilisationReportsRoutes.post(
 utilisationReportsRoutes.get(
   '/:reportId/create-record-correction-request/:feeRecordId',
   validateFeeRecordCorrectionFeatureFlagIsEnabled,
-  setNoStoreCacheControl,
+  setCacheControl,
   validateUserTeam([PDC_TEAM_IDS.PDC_RECONCILE]),
   validateSqlId('reportId'),
   validateSqlId('feeRecordId'),
@@ -180,7 +180,7 @@ utilisationReportsRoutes.post(
 
 utilisationReportsRoutes.get(
   '/:reportId/create-record-correction-request/:feeRecordId/check-the-information',
-  setNoStoreCacheControl,
+  setCacheControl,
   validateFeeRecordCorrectionFeatureFlagIsEnabled,
   validateUserTeam([PDC_TEAM_IDS.PDC_RECONCILE]),
   validateSqlId('reportId'),