Merge remote-tracking branch 'origin/part-two' into part_three

Author: vilasmaciel

package-lock.json

@@ -25,8 +25,9 @@
     "@types/express": "^4.17.2",
     "@types/glob": "^7.1.1",
     "@types/helmet": "0.0.44",
-    "@types/node": "~11.13.0",
+    "@types/node": "~13.1.1",
     "@types/uuid": "^3.4.6",
+    "@types/uuid-validate": "0.0.1",
     "body-parser": "^1.19.0",
     "bson": "^4.0.2",
     "compression": "^1.7.4",
@@ -36,12 +37,13 @@
     "express": "^4.17.1",
     "glob": "^7.1.6",
     "helmet": "^3.21.2",
-    "http-status": "^1.3.2",
+    "http-status": "^1.4.2",
     "mandrill-api": "^1.0.45",
     "node-dependency-injection": "^2.4.2",
     "ts-node": "^8.3.0",
     "typescript": "^3.7.2",
     "uuid": "^3.3.3",
+    "uuid-validate": "0.0.3",
     "winston": "^3.2.1"
   },
   "devDependencies": {

package.json

@@ -1,11 +1,12 @@
-import CourseRepository from '../domain/CourseRepository';
-import Course from '../domain/Course';
-import { CourseId } from '../domain/CourseId';
+import { CourseRepository } from '../domain/CourseRepository';
+import { Course } from '../domain/Course';
+import { CreateCourseRequest } from './CreateCourseRequest';
+import { CourseId } from '../../Shared/domain/Courses/CourseId';
 import { CourseName } from '../domain/CourseName';
 import { CourseDuration } from '../domain/CourseDuration';
 import { EventBus } from '../../../Shared/domain/EventBus';
 
-export default class CourseCreator {
+export class CourseCreator {
   private repository: CourseRepository;
   private eventBus: EventBus;
 
@@ -14,8 +15,12 @@ export default class CourseCreator {
     this.eventBus = eventBus;
   }
 
-  async run(id: CourseId, name: CourseName, duration: CourseDuration): Promise<void> {
-    const course = new Course(id, name, duration);
+  async run(request: CreateCourseRequest): Promise<void> {
+    const course = new Course(
+      new CourseId(request.id),
+      new CourseName(request.name),
+      new CourseDuration(request.duration)
+    );
 
     await this.repository.save(course);
 

src/Contexts/Mooc/Courses/application/CourseCreator.ts

@@ -0,0 +1,5 @@
+export type CreateCourseRequest = {
+  id: string;
+  name: string;
+  duration: string;
+};

src/Contexts/Mooc/Courses/application/CreateCourseRequest.ts

@@ -1,10 +1,10 @@
 import { AggregateRoot } from './AggregateRoot';
 import { CourseCreatedDomainEvent } from './CourseCreatedDomainEvent';
-import { CourseId } from './CourseId';
 import { CourseName } from './CourseName';
 import { CourseDuration } from './CourseDuration';
+import { CourseId } from '../../Shared/domain/Courses/CourseId';
 
-export default class Course extends AggregateRoot {
+export class Course extends AggregateRoot {
   readonly id: CourseId;
   readonly name: CourseName;
   readonly duration: CourseDuration;
@@ -21,9 +21,9 @@ export default class Course extends AggregateRoot {
 
     course.record(
       new CourseCreatedDomainEvent({
-        id: course.id.value(),
-        duration: course.duration.value(),
-        name: course.name.value()
+        id: course.id.value,
+        duration: course.duration.value,
+        name: course.name.value
       })
     );
 

src/Contexts/Mooc/Courses/domain/Course.ts

@@ -1,5 +1,5 @@
-export default class CourseAlreadyExists extends Error {
-  constructor(courseId: string) {
-    super(`Course ${courseId} already exists`);
+export class CourseAlreadyExists extends Error {
+  constructor(id: string) {
+    super(`Course ${id} already exists`);
   }
 }

src/Contexts/Mooc/Courses/domain/CourseAlreadyExists.ts

@@ -1,14 +1,14 @@
 import { DomainEvent } from '../../../Shared/domain/DomainEvent';
 
 type CreateCourseDomainEventBody = {
-  readonly duration: number;
+  readonly duration: string;
   readonly name: string;
 };
 
 export class CourseCreatedDomainEvent extends DomainEvent {
   static readonly EVENT_NAME = 'course.created';
 
-  readonly duration: number;
+  readonly duration: string;
   readonly name: string;
 
   constructor({
@@ -20,7 +20,7 @@ export class CourseCreatedDomainEvent extends DomainEvent {
   }: {
     id: string;
     eventId?: string;
-    duration: number;
+    duration: string;
     name: string;
     occurredOn?: Date;
   }) {

src/Contexts/Mooc/Courses/domain/CourseCreatedDomainEvent.ts

@@ -1,3 +1,3 @@
-import { ValueObject } from '../../../Shared/domain/ValueObject';
+import { StringValueObject } from '../../../Shared/domain/value-object/StringValueObject';
 
-export class CourseDuration extends ValueObject<number> {}
+export class CourseDuration extends StringValueObject {}

src/Contexts/Mooc/Courses/domain/CourseDuration.ts

@@ -1,3 +1,15 @@
-import { ValueObject } from '../../../Shared/domain/ValueObject';
+import { StringValueObject } from '../../../Shared/domain/value-object/StringValueObject';
+import { InvalidArgumentError } from '../../../Shared/domain/value-object/InvalidArgumentError';
 
-export class CourseName extends ValueObject<string> {}
+export class CourseName extends StringValueObject {
+  constructor(value: string) {
+    super(value);
+    this.ensureLengthIsLessThan30Characters(value);
+  }
+
+  private ensureLengthIsLessThan30Characters(value: string): void {
+    if (value.length > 30) {
+      throw new InvalidArgumentError(`The Course Name <${value}> has more than 30 characters`);
+    }
+  }
+}