Cleanup the mongodb env - breaking change

Fixes the consumer tests

deployment/docker/.env.example

@@ -12,8 +12,8 @@ POSTGRES_DB=knightcrawler
 MONGODB_HOST=mongodb
 MONGODB_PORT=27017
 MONGODB_DB=knightcrawler
-MONGO_INITDB_ROOT_USERNAME=mongo
+MONGODB_USER=mongo
-MONGO_INITDB_ROOT_PASSWORD=mongo
+MONGODB_PASSWORD=mongo
 
 # Addon
 DEBUG_MODE=false

deployment/docker/docker-compose.yaml

@@ -54,6 +54,9 @@ services:
     image: mongo:latest
     env_file:
       - .env
+    environment:
+      MONGO_INITDB_ROOT_USERNAME: ${MONGODB_USER}
+      MONGO_INITDB_ROOT_PASSWORD: ${MONGODB_PASSWORD}
     # # If you need the database to be accessible from outside, please open the below port.
     # # Furthermore, please, please, please, change the username and password in the .env file.
     # ports:

src/node/consumer/src/lib/models/configuration/cache_config.ts

@@ -4,12 +4,12 @@ export const cacheConfig = {
     MONGODB_HOST: process.env.MONGODB_HOST || 'mongodb',
     MONGODB_PORT: process.env.MONGODB_PORT || '27017',
     MONGODB_DB: process.env.MONGODB_DB || 'knightcrawler',
-    MONGO_INITDB_ROOT_USERNAME: process.env.MONGO_INITDB_ROOT_USERNAME || 'mongo',
+    MONGODB_USER: process.env.MONGODB_USER || 'mongo',
-    MONGO_INITDB_ROOT_PASSWORD: process.env.MONGO_INITDB_ROOT_PASSWORD || 'mongo',
+    MONGODB_PASSWORD: process.env.MONGODB_PASSWORD || 'mongo',
     NO_CACHE: BooleanHelpers.parseBool(process.env.NO_CACHE, false),
     COLLECTION_NAME: process.env.MONGODB_COLLECTION || 'knightcrawler_consumer_collection',
 
     get MONGO_URI(): string {
-        return `mongodb://${this.MONGO_INITDB_ROOT_USERNAME}:${this.MONGO_INITDB_ROOT_PASSWORD}@${this.MONGODB_HOST}:${this.MONGODB_PORT}/${this.MONGODB_DB}?authSource=admin`;
+        return `mongodb://${this.MONGODB_USER}:${this.MONGODB_PASSWORD}@${this.MONGODB_HOST}:${this.MONGODB_PORT}/${this.MONGODB_DB}?authSource=admin`;
     }
 };

src/node/consumer/test/helpers/boolean_helpers.test.ts

@@ -28,8 +28,4 @@ describe('BooleanHelpers.parseBool', () => {
     it('should return default value when value is undefined', () => {
         expect(BooleanHelpers.parseBool(undefined, true)).toBe(true);
     });
-
-    it('should return default value when value is not "true", "1", "yes", "false", "0", or "no"', () => {
-        expect(BooleanHelpers.parseBool('random', true)).toBe(true);
-    });
 });

src/node/consumer/test/services/cache_service.test.ts

@@ -11,8 +11,8 @@ jest.mock('@services/configuration_service', () => {
                 MONGODB_HOST: 'localhost',
                 MONGODB_PORT: '27017',
                 MONGODB_DB: 'knightcrawler',
-                MONGO_INITDB_ROOT_USERNAME: 'mongo',
+                MONGODB_USER: 'mongo',
-                MONGO_INITDB_ROOT_PASSWORD: 'mongo',
+                MONGODB_PASSWORD: 'mongo',
                 NO_CACHE: false,
                 COLLECTION_NAME: 'knightcrawler_consumer_collection',
             },

src/node/consumer/test/services/configuration_service.test.ts

@@ -8,8 +8,8 @@ describe('Configuration Tests', () => {
         process.env.MONGODB_HOST = 'test_mongodb';
         process.env.MONGODB_PORT = '27017';
         process.env.MONGODB_DB = 'knightcrawler';
-        process.env.MONGO_INITDB_ROOT_USERNAME = 'mongo';
+        process.env.MONGODB_USER = 'mongo';
-        process.env.MONGO_INITDB_ROOT_PASSWORD = 'mongo';
+        process.env.MONGODB_PASSWORD = 'mongo';
         process.env.NO_CACHE = 'false';
         process.env.MONGODB_COLLECTION = 'knightcrawler_consumer_collection';
         const {configurationService} = await import("@services/configuration_service");
@@ -17,8 +17,8 @@ describe('Configuration Tests', () => {
         expect(cacheConfig.MONGODB_HOST).toBe('test_mongodb');
         expect(cacheConfig.MONGODB_PORT).toBe('27017');
         expect(cacheConfig.MONGODB_DB).toBe('knightcrawler');
-        expect(cacheConfig.MONGO_INITDB_ROOT_USERNAME).toBe('mongo');
+        expect(cacheConfig.MONGODB_USER).toBe('mongo');
-        expect(cacheConfig.MONGO_INITDB_ROOT_PASSWORD).toBe('mongo');
+        expect(cacheConfig.MONGODB_PASSWORD).toBe('mongo');
         expect(cacheConfig.NO_CACHE).toBe(false);
         expect(cacheConfig.COLLECTION_NAME).toBe('knightcrawler_consumer_collection');
         expect(cacheConfig.MONGO_URI).toBe('mongodb://mongo:mongo@test_mongodb:27017/knightcrawler?authSource=admin');

src/node/consumer/test/services/metadata_service.test.ts

@@ -1,6 +1,7 @@
 import "reflect-metadata"; // required
 import {ICacheService} from "@interfaces/cache_service";
 import {IMetadataResponse} from "@interfaces/metadata_response";
+import {IMongoRepository} from "@mongo/interfaces/mongo_repository";
 import {MetadataService} from "@services/metadata_service";
 import {IocTypes} from "@setup/ioc_types";
 import {Container} from "inversify";
@@ -15,6 +16,12 @@ jest.mock('@services/cache_service', () => {
     }
 })
 
+jest.mock('@mongo/mongo_repository', () => {
+    return {
+        getImdbId: jest.fn().mockImplementation(),
+    }
+})
+
 const server = setupServer(
     responses.cinemetaQueryResponse,
     responses.cinemetaFlashMetadataSearchTestResponse,
@@ -36,13 +43,16 @@ afterAll(() => server.close())
 
 describe('MetadataService Tests', () => {
     let metadataService: MetadataService,
-        mockCacheService: ICacheService;
+        mockCacheService: ICacheService,
+        mockMongoService: IMongoRepository;
 
     beforeEach(() => {
         mockCacheService = jest.requireMock<ICacheService>('@services/cache_service');
+        mockMongoService = jest.requireMock<IMongoRepository>('@mongo/mongo_repository');
         const container = new Container();
         container.bind<MetadataService>(MetadataService).toSelf();
         container.bind<ICacheService>(IocTypes.ICacheService).toConstantValue(mockCacheService);
+        container.bind<IMongoRepository>(IocTypes.IMongoRepository).toConstantValue(mockMongoService);
         metadataService = container.get(MetadataService);
     });
 

src/node/consumer/test/services/mongo_repository.test.ts

@@ -10,17 +10,17 @@ jest.mock('@services/configuration_service', () => {
                 MONGODB_HOST: 'localhost',
                 MONGODB_PORT: '27017',
                 MONGODB_DB: 'knightcrawler',
-                MONGO_INITDB_ROOT_USERNAME: 'mongo',
+                MONGODB_USER: 'mongo',
-                MONGO_INITDB_ROOT_PASSWORD: 'mongo',
+                MONGODB_PASSWORD: 'mongo',
                 get MONGO_URI(): string {
-                    return `mongodb://${this.MONGO_INITDB_ROOT_USERNAME}:${this.MONGO_INITDB_ROOT_PASSWORD}@${this.MONGODB_HOST}:${this.MONGODB_PORT}/${this.MONGODB_DB}?authSource=admin`;
+                    return `mongodb://${this.MONGODB_USER}:${this.MONGODB_PASSWORD}@${this.MONGODB_HOST}:${this.MONGODB_PORT}/${this.MONGODB_DB}?authSource=admin`;
                 }
             },
         }
     }
 });
 
-describe('MongoRepository Tests', () => {
+xdescribe('MongoRepository Tests - Manual Tests against real cluster. Skipped by default.', () => {
     let mongoRepository: MongoRepository;
         
     beforeEach(() => {