Remove MongoBackofficeCourseRepository from Backoffice context

Author: vilasmaciel

src/Contexts/Backoffice/Courses/infrastructure/persistence/MongoBackofficeCourseRepository.ts

@@ -1,10 +1,11 @@
 services:
 
-  Backoffice.courses.BackofficeCourseRepository:
-    class: ../../../../../Contexts/Backoffice/Courses/infrastructure/persistence/MongoBackofficeCourseRepository
-    arguments: ['@Shared.MongoConnectionManager']
+  Backoffice.courses.ElasticConfig:
+    factory:
+      class: ../../../../../Contexts/Backoffice/Courses/infrastructure/persistence/BackofficeElasticConfigFactory
+      method: 'createConfig'
 
-  Backoffice.courses.BackofficeCourseRepositoryElastic:
+  Backoffice.courses.BackofficeCourseRepository:
     class: ../../../../../Contexts/Backoffice/Courses/infrastructure/persistence/ElasticBackofficeCourseRepository
     arguments: ['@Shared.ElasticConnectionManager']
 

src/apps/backoffice/backend/dependency-injection/Courses/application.yaml

@@ -3,17 +3,11 @@ services:
     class: ../../../../../Contexts/Shared/infrastructure/WinstonLogger
     arguments: []
 
-  Shared.MongoConnectionManager:
-    factory:
-      class: ../../../../../Contexts/Shared/infrastructure/persistence/mongo/MongoClientFactory
-      method: 'createClient'
-    arguments: ['mooc', '@Apps.Backoffice.Backend.MongoConfig']
-
   Shared.ElasticConnectionManager:
     factory:
       class: ../../../../../Contexts/Shared/infrastructure/persistence/elasticsearch/ElasticClientFactory
       method: 'createClient'
-    arguments: ['mooc', '@Apps.Backoffice.Backend.ElasticConfig']
+    arguments: ['mooc', '@Backoffice.courses.ElasticConfig']
 
   Shared.QueryHandlersInformation:
     class: ../../../../../Contexts/Shared/infrastructure/QueryBus/QueryHandlersInformation

src/apps/backoffice/backend/dependency-injection/Shared/application.yaml

@@ -3,9 +3,5 @@ imports:
 
 services:
   Backoffice.Backend.EnvironmentArranger:
-    class: ../../../../../tests/Contexts/Shared/infrastructure/mongo/MongoEnvironmentArranger
-    arguments: ['@Shared.MongoConnectionManager']
-
-  Backoffice.Backend.ElasticEnvironmentArranger:
     class: ../../../../../tests/Contexts/Shared/infrastructure/elastic/ElasticEnvironmentArranger
     arguments: ['@Shared.ElasticConnectionManager']

src/apps/backoffice/backend/dependency-injection/application_test.yaml

@@ -1,13 +1,4 @@
 services:
-  Apps.Backoffice.Backend.MongoConfig:
-    factory:
-      class: ../../../../../Contexts/Backoffice/Courses/infrastructure/persistence/BackofficeMongoConfigFactory
-      method: 'createConfig'
-
-  Apps.Backoffice.Backend.ElasticConfig:
-    factory:
-      class: ../../../../../Contexts/Backoffice/Courses/infrastructure/persistence/BackofficeElasticConfigFactory
-      method: 'createConfig'
 
   Apps.Backoffice.Backend.controllers.StatusGetController:
     class: ../../controllers/StatusGetController

src/apps/backoffice/backend/dependency-injection/apps/application.yaml

@@ -4,18 +4,14 @@ import { ElasticBackofficeCourseRepository } from '../../../../../src/Contexts/B
 import { EnvironmentArranger } from '../../../Shared/infrastructure/arranger/EnvironmentArranger';
 import { BackofficeCourseMother } from '../domain/BackofficeCourseMother';
 
-const repository: ElasticBackofficeCourseRepository = container.get(
-  'Backoffice.courses.BackofficeCourseRepositoryElastic'
-);
-const environmentArranger: Promise<EnvironmentArranger> = container.get(
-  'Backoffice.Backend.ElasticEnvironmentArranger'
-);
+const repository: ElasticBackofficeCourseRepository = container.get('Backoffice.courses.BackofficeCourseRepository');
+const environmentArranger: Promise<EnvironmentArranger> = container.get('Backoffice.Backend.EnvironmentArranger');
 
 afterEach(async () => {
   await (await environmentArranger).arrange();
 });
 
-describe('ElasticBackofficeCourseRepository', () => {
+describe('BackofficeCourseRepository', () => {
   describe('#searchAll', () => {
     it('should return the existing courses', async () => {
       const courses = [BackofficeCourseMother.random(), BackofficeCourseMother.random()];

tests/Contexts/Backoffice/Courses/infrastructure/ElasticBackofficeCourseRepository.test.ts renamed to tests/Contexts/Backoffice/Courses/infrastructure/BackofficeCourseRepository.test.ts

@@ -14,7 +14,7 @@ afterAll(async () => {
   await (await environmentArranger).close();
 });
 
-describe('MongoCourseRepository', () => {
+describe('CourseRepository', () => {
   describe('#save', () => {
     it('should save a course', async () => {
       const course = CourseMother.random();
@@ -25,11 +25,12 @@ describe('MongoCourseRepository', () => {
 
   describe('#search', () => {
     it('should return an existing course', async () => {
-      const course = CourseMother.random();
+      const expectedCourse = CourseMother.random();
+      await repository.save(expectedCourse);
 
-      await repository.save(course);
+      const course = await repository.search(expectedCourse.id);
 
-      expect(course).toEqual(await repository.search(course.id));
+      expect(expectedCourse).toEqual(course);
     });
 
     it('should not return a non existing course', async () => {

tests/Contexts/Backoffice/Courses/infrastructure/MongoBackofficeCourseRepository.test.ts

@@ -25,11 +25,12 @@ describe('MongoCoursesCounterRepository', () => {
 
   describe('#search', () => {
     it('should return an existing course', async () => {
-      const counter = CoursesCounterMother.random();
+      const expectedCounter = CoursesCounterMother.random();
+      await repository.save(expectedCounter);
 
-      await repository.save(counter);
+      const counter = await repository.search();
 
-      expect(counter).toEqual(await repository.search());
+      expect(expectedCounter).toEqual(counter);
     });
 
     it('should not return null if there is no courses counter', async () => {

tests/Contexts/Mooc/Courses/infrastructure/persistence/MongoCourseRepository.test.ts renamed to tests/Contexts/Mooc/Courses/infrastructure/persistence/CourseRepository.test.ts

@@ -2,7 +2,7 @@ import assert from 'assert';
 import { AfterAll, Before, Given, Then } from 'cucumber';
 import request from 'supertest';
 import app from '../../../../../../src/apps/backoffice/backend/app';
-import container from '../../../../../../src/apps/backoffice/backend/config/dependency-injection';
+import container from '../../../../../../src/apps/backoffice/backend/dependency-injection';
 import { EnvironmentArranger } from '../../../../../Contexts/Shared/infrastructure/arranger/EnvironmentArranger';
 
 let _request: request.Test;