feat: bump express version (#12045)

* feat: bump express version

Signed-off-by: eternityduck <[email protected]>

* fix: update jest versions and tests

Signed-off-by: eternityduck <[email protected]>

* fix: express 5 route matchers

Signed-off-by: eternityduck <[email protected]>

* fix: path regexp version update

Signed-off-by: eternityduck <[email protected]>

* fix: tests

Signed-off-by: eternityduck <[email protected]>

* chore: sync lock file

Signed-off-by: eternityduck <[email protected]>

* fix: missing package

Signed-off-by: eternityduck <[email protected]>

* chore: remove unneeded file

Signed-off-by: eternityduck <[email protected]>

---------

Signed-off-by: eternityduck <[email protected]>
Co-authored-by: Alex <[email protected]>

Author: eternityduck

frontend/server/app.ts

@@ -101,7 +101,7 @@ function createUIServer(options: UIConfigs) {
   const registerHandler = getRegisterHandler(app, basePath);
 
   /** log to stdout */
-  app.use((req, _, next) => {
+  app.use((req, res, next) => {
     console.info(req.method + ' ' + req.originalUrl);
     next();
   });

frontend/server/integration-tests/artifact-proxy.test.ts

@@ -48,7 +48,7 @@ describe('/artifacts/get namespaced proxy', () => {
     const receivedUrls: string[] = [];
     const artifactService = express();
     const response = `artifact service in ${namespace}`;
-    artifactService.all('/*', (req, res) => {
+    artifactService.use((req, res) => {
       receivedUrls.push(req.url);
       res.status(200).send(response);
     });

frontend/server/minio-helper.test.ts

@@ -184,7 +184,7 @@ describe('minio-helper', () => {
       mockedMinioGetObject = minioClient.getObject as any;
     });
 
-    it('unpacks a gzipped tarball', async done => {
+    it('unpacks a gzipped tarball', async () => {
       const objStream = new PassThrough();
       objStream.end(tarGzBuffer);
       mockedMinioGetObject.mockResolvedValueOnce(Promise.resolve(objStream));
@@ -198,11 +198,10 @@ describe('minio-helper', () => {
             .toString()
             .trim(),
         ).toBe('hello world');
-        done();
       });
     });
 
-    it('unpacks a uncompressed tarball', async done => {
+    it('unpacks a uncompressed tarball', async () => {
       const objStream = new PassThrough();
       objStream.end(tarBuffer);
       mockedMinioGetObject.mockResolvedValueOnce(Promise.resolve(objStream));
@@ -216,11 +215,10 @@ describe('minio-helper', () => {
             .toString()
             .trim(),
         ).toBe('hello world');
-        done();
       });
     });
 
-    it('returns the content as a stream', async done => {
+    it('returns the content as a stream', async () => {
       const objStream = new PassThrough();
       objStream.end('hello world');
       mockedMinioGetObject.mockResolvedValueOnce(Promise.resolve(objStream));
@@ -234,7 +232,6 @@ describe('minio-helper', () => {
             .toString()
             .trim(),
         ).toBe('hello world');
-        done();
       });
     });
   });