fix: false positive warning in some edge cases (#225)

* fix: false positive warning in some edge cases when forRootAsync with useFactory is used

* chore: remove unnecessary warnings in tests, restore mock

Author: myovchev

src/mikro-orm-core.module.ts

@@ -235,7 +235,7 @@ export class MikroOrmCoreModule implements NestModule, OnApplicationShutdown {
 
       return config?.getDriver().createEntityManager();
     } catch {
-      if (options && 'useFactory' in options && 'inject' in options && (options.inject as unknown[]).length > 0) {
+      if (options && 'useFactory' in options && 'inject' in options && !options.driver && (options.inject as unknown[]).length > 0) {
         // eslint-disable-next-line no-console
         console.warn('Support for driver specific imports in modules defined with `useFactory` and `inject` requires an explicit `driver` option. See https://github.com/mikro-orm/nestjs/pull/204');
       }

tests/mikro-orm.module.test.ts

@@ -131,6 +131,7 @@ describe('MikroORM Module', () => {
             ...testOptions,
             logger: logger.log.bind(logger),
           }),
+          driver: testOptions.driver,
           inject: ['my-logger'],
           providers: [myLoggerProvider],
         })],
@@ -165,6 +166,7 @@ describe('MikroORM Module', () => {
             ...testOptions,
             logger: logger.log.bind(logger),
           }),
+          driver: testOptions.driver,
           inject: ['my-logger'],
           providers: [myLoggerProvider],
           scope: Scope.REQUEST,
@@ -227,6 +229,7 @@ describe('MikroORM Module', () => {
       }).compile();
       expect(warnSpy).toBeCalledWith('Support for driver specific imports in modules defined with `useFactory` and `inject` requires an explicit `driver` option. See https://github.com/mikro-orm/nestjs/pull/204');
       await module.get<MikroORM>(MikroORM).close();
+      warnSpy.mockRestore();
     });
 
     it('forFeature should return repository', async () => {