tests(test-setup): ensure shutdown hooks get run

Signed-off-by: David Mehren <git@herrmehren.de>

src/app-init.ts

@@ -79,4 +79,5 @@ export async function setupApp(
   const { httpAdapter } = app.get(HttpAdapterHost);
   app.useGlobalFilters(new ErrorExceptionMapping(httpAdapter));
   app.useWebSocketAdapter(new WsAdapter(app));
+  app.enableShutdownHooks();
 }

src/main.ts

@@ -42,13 +42,6 @@ async function bootstrap(): Promise<void> {
   // Setup code must be added there!
   await setupApp(app, appConfig, authConfig, mediaConfig, logger);
 
-  /**
-   * enableShutdownHooks consumes memory by starting listeners. In cases where you are running multiple Nest apps in a
-   * single Node process (e.g., when running parallel tests with Jest), Node may complain about excessive listener processes.
-   * For this reason, enableShutdownHooks is not enabled in the tests.
-   */
-  app.enableShutdownHooks();
-
   // Start the server
   await app.listen(appConfig.port);
   logger.log(`Listening on port ${appConfig.port}`, 'AppBootstrap');

test/test-setup.ts

@@ -89,6 +89,7 @@ export class TestSetup {
     }
     if (connectionOptions.type === 'sqlite') {
       // Bail out early, as SQLite runs from memory anyway
+      await this.app.close();
       return;
     }
     if (appConnection.isConnected) {
@@ -106,6 +107,7 @@ export class TestSetup {
         await connection.close();
       }
     }
+    await this.app.close();
   }
 }