Refactor CourseCreator test to use command handler

Co-authored-by: aliondev <antonioleon@audiense.com>
Co-authored-by: Rubén Salado <rsaladocid@users.noreply.github.com>

Author: 3 people

tests/Contexts/Mooc/Courses/application/CourseCreator.test.ts

@@ -1,25 +1,25 @@
 import { CourseCreator } from '../../../../../src/Contexts/Mooc/Courses/application/CourseCreator';
 import { CourseMother } from '../domain/CourseMother';
 import { CourseRepositoryMock } from '../__mocks__/CourseRepositoryMock';
-import { CreateCourseRequestMother } from './CreateCourseRequestMother';
+import { CreateCourseCommandMother } from './CreateCourseCommandMother';
 import EventBusMock from '../__mocks__/EventBusMock';
+import { CreateCourseCommandHandler } from '../../../../../src/Contexts/Mooc/Courses/application/CreateCourseCommandHandler';
 
 let repository: CourseRepositoryMock;
-let creator: CourseCreator;
+let handler: CreateCourseCommandHandler;
 
 const eventBus = new EventBusMock();
 
 beforeEach(() => {
   repository = new CourseRepositoryMock();
-  creator = new CourseCreator(repository, eventBus);
+  const creator = new CourseCreator(repository, eventBus);
+  handler = new CreateCourseCommandHandler(creator);
 });
 
 it('should create a valid course', async () => {
-  const request = CreateCourseRequestMother.random();
-
-  const course = CourseMother.fromRequest(request);
-
-  await creator.run(request);
+  const command = CreateCourseCommandMother.random();
+  await handler.handle(command);
 
+  const course = CourseMother.fromCommand(command);
   repository.assertLastSavedCourseIs(course);
 });

tests/Contexts/Mooc/Courses/application/CreateCourseCommandMother.ts

@@ -0,0 +1,14 @@
+import { CourseDurationMother } from '../domain/CourseDurationMother';
+import { CourseIdMother } from '../../Shared/domain/Courses/CourseIdMother';
+import { CourseNameMother } from '../domain/CourseNameMother';
+import { CreateCourseCommand } from '../../../../../src/Contexts/Mooc/Courses/application/CreateCourseCommand';
+
+export class CreateCourseCommandMother {
+  static create(id: string, name: string, duration: string): CreateCourseCommand {
+    return new CreateCourseCommand({ id, name, duration });
+  }
+
+  static random(): CreateCourseCommand {
+    return this.create(CourseIdMother.random().value, CourseNameMother.random().value, CourseDurationMother.random().value);
+  }
+}

tests/Contexts/Mooc/Courses/application/CreateCourseRequestMother.ts

@@ -1,4 +1,4 @@
-import { CreateCourseRequestMother } from '../application/CreateCourseRequestMother';
+import { CreateCourseCommandMother } from '../application/CreateCourseCommandMother';
 import { CourseMother } from './CourseMother';
 import { Course } from '../../../../../src/Contexts/Mooc/Courses/domain/Course';
 import { CourseIdMother } from '../../Shared/domain/Courses/CourseIdMother';
@@ -8,13 +8,13 @@ import { CourseDurationMother } from './CourseDurationMother';
 describe('Course', () => {
 
   it('should return a new course instance', () => {
-    const request = CreateCourseRequestMother.random();
+    const command = CreateCourseCommandMother.random();
 
-    const course = CourseMother.fromRequest(request);
+    const course = CourseMother.fromCommand(command);
 
-    expect(course.id.value).toBe(request.id);
-    expect(course.name.value).toBe(request.name);
-    expect(course.duration.value).toBe(request.duration);
+    expect(course.id.value).toBe(command.id);
+    expect(course.name.value).toBe(command.name);
+    expect(course.duration.value).toBe(command.duration);
   });
 
   it('should record a CourseCreatedDomainEvent after its creation', () => {

tests/Contexts/Mooc/Courses/domain/Course.test.ts

@@ -6,17 +6,18 @@ import { CreateCourseRequest } from '../../../../../src/Contexts/Mooc/Courses/ap
 import { CourseIdMother } from '../../Shared/domain/Courses/CourseIdMother';
 import { CourseNameMother } from './CourseNameMother';
 import { CourseDurationMother } from './CourseDurationMother';
+import { CreateCourseCommand } from '../../../../../src/Contexts/Mooc/Courses/application/CreateCourseCommand';
 
 export class CourseMother {
   static create(id: CourseId, name: CourseName, duration: CourseDuration): Course {
     return new Course(id, name, duration);
   }
 
-  static fromRequest(request: CreateCourseRequest): Course {
+  static fromCommand(command: CreateCourseCommand): Course {
     return this.create(
-      CourseIdMother.create(request.id),
-      CourseNameMother.create(request.name),
-      CourseDurationMother.create(request.duration)
+      CourseIdMother.create(command.id),
+      CourseNameMother.create(command.name),
+      CourseDurationMother.create(command.duration)
     );
   }