fix(testing): Fixed vitest being bundled and hoisted

Author: spuxx-dev

apps/nest/src/main.ts

@@ -13,7 +13,7 @@ async function bootstrap() {
     logger,
   });
 
-  AuthModule.bootstrap(app, authConfig);
+  await AuthModule.bootstrap(app, authConfig);
 
   await app.listen(3000);
   Logger.log(`Application is running on: http://localhost:3000`, 'Bootstrap');

packages/nest-utils/src/auth/auth.module.ts

@@ -1,6 +1,5 @@
 import { DynamicModule, INestApplication, Logger, Module } from '@nestjs/common';
 import { defaultAuthOptions, type AuthOptions } from '../main';
-import { auth } from 'express-openid-connect';
 import { AuthController } from './controllers/auth.controller';
 import { AuthService } from './providers/auth.service';
 import { deepMerge } from '@spuxx/js-utils';
@@ -51,6 +50,7 @@ export class AuthModule {
       Logger.warn('Authentication is disabled. All routes will be accessible.', AuthModule.name);
       return;
     }
+    const { auth } = await import('express-openid-connect');
     app.use(auth(oidc));
     Logger.log(`Authentication is enabled and will be handled by issuer at '${oidc.issuerBaseURL}'.`, AuthModule.name);
   }

packages/nest-utils/src/testing/container/private/mock-oidc.ts

@@ -30,3 +30,12 @@ export class MockOidcModule implements NestModule {
     consumer.apply(MockOidcMiddleware).forRoutes('*');
   }
 }
+
+export function mockExpressOidcPackage() {
+  vitest.doMock('express-openid-connect', () => {
+    return {
+      auth: vitest.fn(() => (_req: Request, _res: Response, next: NextFunction) => next()),
+      requiresAuth: vitest.fn(() => (_req: Request, _res: Response, next: NextFunction) => next()),
+    };
+  });
+}

packages/nest-utils/src/testing/container/test-container.ts

@@ -1,21 +1,13 @@
 import { INestApplication } from '@nestjs/common';
 import { Test, TestingModule } from '@nestjs/testing';
 import { type OmitFunctionMembers } from '@spuxx/js-utils';
-import { NextFunction, Request, Response } from 'express';
 import { AuthOptions, SessionResource } from '../../auth';
 import { AuthModule } from '../../auth/auth.module';
 import { Supertest } from '../supertest';
 import { createEndToEndNestApplication } from './private/end-to-end';
-import { MockOidcModule } from './private/mock-oidc';
+import { mockExpressOidcPackage, MockOidcModule } from './private/mock-oidc';
 import { TestContainerOptions } from './types';
 
-vitest.mock('express-openid-connect', () => {
-  return {
-    auth: vitest.fn(() => (_req: Request, _res: Response, next: NextFunction) => next()),
-    requiresAuth: vitest.fn(() => (_req: Request, _res: Response, next: NextFunction) => next()),
-  };
-});
-
 /**
  * `TestContainer` provides an abstraction of `Nest.createTestContainer()`, offering
  * a custom API for easier handling and use. For more information on testing in NestJS,
@@ -84,6 +76,8 @@ export class TestContainer {
       ...options,
     };
 
+    mockExpressOidcPackage();
+
     // Auto-add conditional components
     if (enableEndToEnd) {
       imports.push(AuthModule.forRoot(authOptions as AuthOptions), MockOidcModule);
@@ -111,7 +105,7 @@ export class TestContainer {
     if (enableEndToEnd) {
       app = await createEndToEndNestApplication(module);
       session = { ...options.session } ?? {};
-      AuthModule.bootstrap(app, authOptions as AuthOptions);
+      await AuthModule.bootstrap(app, authOptions as AuthOptions);
       supertest = new Supertest(app, session);
     }
     // Return the test container