Lint files

vilasmaciel · vilasmaciel · commit d20e087327d2 · 2022-10-14T11:37:45.000+02:00
diff --git a/.prettierignore b/.prettierignore
@@ -0,0 +1 @@
+src/apps/backoffice/frontend
diff --git a/.prettierrc b/.prettierrc
@@ -3,7 +3,6 @@
   "tabWidth": 2,
   "singleQuote": true,
   "trailingComma": "none",
-  "semicolons": true,
   "quoteProps": "as-needed",
   "arrowParens": "avoid"
 }
diff --git a/package.json b/package.json
@@ -14,7 +14,7 @@
     "dev:mooc:backend": "NODE_ENV=dev ts-node-dev --ignore-watch node_modules  ./src/apps/mooc/backend/start.ts",
     "dev:backoffice:frontend": "cd ./src/apps/backoffice/frontend && npm start",
     "dev:backoffice:backend": "NODE_ENV=dev ts-node-dev --ignore-watch node_modules ./src/apps/backoffice/backend/start.ts",
-    "lint": "prettier --write --ignore-path src/apps/backoffice/frontend src/**/*.ts{,x}",
+    "lint": "prettier --write src/**/*.ts{,x}",
     "test": "npm run test:unit && npm run test:features",
     "test:unit": "NODE_ENV=test jest",
     "start:mooc:backend": "NODE_ENV=production node dist/src/apps/mooc/backend/start",
@@ -102,7 +102,7 @@
   },
   "lint-staged": {
     "{src,tests}/**/*.ts": [
-      "prettier --write --ignore-path src/apps/backoffice/frontend",
+      "prettier --write",
       "git add"
     ]
   }
diff --git a/src/Contexts/Backoffice/Courses/application/BackofficeCoursesResponse.ts b/src/Contexts/Backoffice/Courses/application/BackofficeCoursesResponse.ts
@@ -1,4 +1,4 @@
-import { BackofficeCourse } from "../domain/BackofficeCourse";
+import { BackofficeCourse } from '../domain/BackofficeCourse';
 
 interface BackofficeCourseResponse {
   id: string;
diff --git a/src/Contexts/Backoffice/Courses/application/SearchAll/SearchAllCoursesQuery.ts b/src/Contexts/Backoffice/Courses/application/SearchAll/SearchAllCoursesQuery.ts
@@ -1,3 +1,3 @@
-import { Query } from "../../../../Shared/domain/Query";
+import { Query } from '../../../../Shared/domain/Query';
 
 export class SearchAllCoursesQuery implements Query {}
diff --git a/src/Contexts/Backoffice/Courses/application/SearchByCriteria/SearchCoursesByCriteriaQueryHandler.ts b/src/Contexts/Backoffice/Courses/application/SearchByCriteria/SearchCoursesByCriteriaQueryHandler.ts
@@ -7,7 +7,8 @@ import { CoursesByCriteriaSearcher } from './CoursesByCriteriaSearcher';
 import { SearchCoursesByCriteriaQuery } from './SearchCoursesByCriteriaQuery';
 
 export class SearchCoursesByCriteriaQueryHandler
-  implements QueryHandler<SearchCoursesByCriteriaQuery, BackofficeCoursesResponse> {
+  implements QueryHandler<SearchCoursesByCriteriaQuery, BackofficeCoursesResponse>
+{
   constructor(private searcher: CoursesByCriteriaSearcher) {}
 
   subscribedTo(): Query {
diff --git a/src/Contexts/Backoffice/Courses/domain/BackofficeCourseDuration.ts b/src/Contexts/Backoffice/Courses/domain/BackofficeCourseDuration.ts
@@ -1,3 +1,3 @@
-import { StringValueObject } from "../../../Shared/domain/value-object/StringValueObject";
+import { StringValueObject } from '../../../Shared/domain/value-object/StringValueObject';
 
 export class BackofficeCourseDuration extends StringValueObject {}
diff --git a/src/Contexts/Backoffice/Courses/domain/BackofficeCourseId.ts b/src/Contexts/Backoffice/Courses/domain/BackofficeCourseId.ts
@@ -1,3 +1,3 @@
-import { Uuid } from "../../../Shared/domain/value-object/Uuid";
+import { Uuid } from '../../../Shared/domain/value-object/Uuid';
 
 export class BackofficeCourseId extends Uuid {}
diff --git a/src/Contexts/Backoffice/Courses/domain/BackofficeCourseName.ts b/src/Contexts/Backoffice/Courses/domain/BackofficeCourseName.ts
@@ -1,3 +1,3 @@
-import { StringValueObject } from "../../../Shared/domain/value-object/StringValueObject";
+import { StringValueObject } from '../../../Shared/domain/value-object/StringValueObject';
 
 export class BackofficeCourseName extends StringValueObject {}
diff --git a/src/Contexts/Backoffice/Courses/infrastructure/persistence/ElasticBackofficeCourseRepository.ts b/src/Contexts/Backoffice/Courses/infrastructure/persistence/ElasticBackofficeCourseRepository.ts
@@ -5,7 +5,8 @@ import { BackofficeCourseRepository } from '../../domain/BackofficeCourseReposit
 
 export class ElasticBackofficeCourseRepository
   extends ElasticRepository<BackofficeCourse>
-  implements BackofficeCourseRepository {
+  implements BackofficeCourseRepository
+{
   async searchAll(): Promise<BackofficeCourse[]> {
     return this.searchAllInElastic(BackofficeCourse.fromPrimitives);
   }
diff --git a/src/Contexts/Mooc/Courses/application/SearchAll/CoursesFinder.ts b/src/Contexts/Mooc/Courses/application/SearchAll/CoursesFinder.ts
@@ -1,5 +1,5 @@
-import { CourseRepository } from "../../domain/CourseRepository";
-import { CoursesResponse } from "./CoursesResponse";
+import { CourseRepository } from '../../domain/CourseRepository';
+import { CoursesResponse } from './CoursesResponse';
 
 export class CoursesFinder {
   constructor(private coursesRepository: CourseRepository) {}
diff --git a/src/Contexts/Mooc/Courses/application/SearchAll/CoursesResponse.ts b/src/Contexts/Mooc/Courses/application/SearchAll/CoursesResponse.ts
@@ -1,4 +1,4 @@
-import { Course } from "../../domain/Course";
+import { Course } from '../../domain/Course';
 
 interface CourseResponse {
   id: string;
diff --git a/src/Contexts/Mooc/Courses/domain/Course.ts b/src/Contexts/Mooc/Courses/domain/Course.ts
@@ -18,7 +18,7 @@ export class Course extends AggregateRoot {
 
   static create(id: CourseId, name: CourseName, duration: CourseDuration): Course {
     const course = new Course(id, name, duration);
- 
+
     course.record(
       new CourseCreatedDomainEvent({
         aggregateId: course.id.value,
diff --git a/src/Contexts/Mooc/Courses/domain/CourseDuration.ts b/src/Contexts/Mooc/Courses/domain/CourseDuration.ts
@@ -1,3 +1,3 @@
 import { StringValueObject } from '../../../Shared/domain/value-object/StringValueObject';
 
-export class CourseDuration extends StringValueObject {}
+export class CourseDuration extends StringValueObject {}
diff --git a/src/Contexts/Mooc/Courses/domain/CourseName.ts b/src/Contexts/Mooc/Courses/domain/CourseName.ts
@@ -12,4 +12,4 @@ export class CourseName extends StringValueObject {
       throw new CourseNameLengthExceeded(`The Course Name <${value}> has more than 30 characters`);
     }
   }
-}
+}
diff --git a/src/Contexts/Mooc/Courses/domain/CourseNameLengthExceeded.ts b/src/Contexts/Mooc/Courses/domain/CourseNameLengthExceeded.ts
@@ -1,3 +1,3 @@
-import { InvalidArgumentError } from "../../../Shared/domain/value-object/InvalidArgumentError";
+import { InvalidArgumentError } from '../../../Shared/domain/value-object/InvalidArgumentError';
 
-export class CourseNameLengthExceeded extends InvalidArgumentError {}
+export class CourseNameLengthExceeded extends InvalidArgumentError {}
diff --git a/src/Contexts/Mooc/Courses/domain/CreateCourseCommand.ts b/src/Contexts/Mooc/Courses/domain/CreateCourseCommand.ts
@@ -1,20 +1,20 @@
 import { Command } from '../../../Shared/domain/Command';
 
 type Params = {
-    id: string;
-    name: string;
-    duration: string;
+  id: string;
+  name: string;
+  duration: string;
 };
 
 export class CreateCourseCommand extends Command {
-    id: string;
-    name: string;
-    duration: string;
+  id: string;
+  name: string;
+  duration: string;
 
-    constructor({ id, name, duration }: Params) {
-        super();
-        this.id = id;
-        this.name = name;
-        this.duration = duration;
-    }
+  constructor({ id, name, duration }: Params) {
+    super();
+    this.id = id;
+    this.name = name;
+    this.duration = duration;
+  }
 }
diff --git a/src/Contexts/Mooc/Courses/infrastructure/persistence/MongoCourseRepository.ts b/src/Contexts/Mooc/Courses/infrastructure/persistence/MongoCourseRepository.ts
@@ -8,7 +8,7 @@ interface CourseDocument {
   _id: string;
   name: string;
   duration: string;
-};
+}
 
 export class MongoCourseRepository extends MongoRepository<Course> implements CourseRepository {
   public save(course: Course): Promise<void> {
diff --git a/src/Contexts/Mooc/Courses/infrastructure/persistence/TypeOrmCourseRepository.ts b/src/Contexts/Mooc/Courses/infrastructure/persistence/TypeOrmCourseRepository.ts
@@ -22,7 +22,7 @@ export class TypeOrmCourseRepository extends TypeOrmRepository<Course> implement
   protected entitySchema(): EntitySchema<Course> {
     return CourseEntity;
   }
-  
+
   public async searchAll(): Promise<Course[]> {
     const repository = await this.repository();
 
diff --git a/src/Contexts/Mooc/CoursesCounter/application/Find/FindCoursesCounterQueryHandler.ts b/src/Contexts/Mooc/CoursesCounter/application/Find/FindCoursesCounterQueryHandler.ts
@@ -5,7 +5,8 @@ import { Query } from '../../../../Shared/domain/Query';
 import { CoursesCounterFinder } from './CoursesCounterFinder';
 
 export class FindCoursesCounterQueryHandler
-  implements QueryHandler<FindCoursesCounterQuery, FindCoursesCounterResponse> {
+  implements QueryHandler<FindCoursesCounterQuery, FindCoursesCounterResponse>
+{
   constructor(private finder: CoursesCounterFinder) {}
 
   subscribedTo(): Query {
diff --git a/src/Contexts/Mooc/CoursesCounter/domain/CoursesCounterIncrementedDomainEvent.ts b/src/Contexts/Mooc/CoursesCounter/domain/CoursesCounterIncrementedDomainEvent.ts
@@ -14,7 +14,7 @@ export class CoursesCounterIncrementedDomainEvent extends DomainEvent {
 
   toPrimitives() {
     return {
-      total: this.total,
+      total: this.total
     };
   }
 
diff --git a/src/Contexts/Mooc/Shared/infrastructure/RabbitMQ/RabbitMQEventBusFactory.ts b/src/Contexts/Mooc/Shared/infrastructure/RabbitMQ/RabbitMQEventBusFactory.ts
@@ -4,7 +4,6 @@ import { RabbitMQEventBus } from '../../../../Shared/infrastructure/EventBus/Rab
 import { RabbitMQqueueFormatter } from '../../../../Shared/infrastructure/EventBus/RabbitMQ/RabbitMQqueueFormatter';
 import { RabbitMQConfig } from './RabbitMQConfigFactory';
 
-
 export class RabbitMQEventBusFactory {
   static create(
     failoverPublisher: DomainEventFailoverPublisher,
diff --git a/src/Contexts/Shared/domain/DomainEvent.ts b/src/Contexts/Shared/domain/DomainEvent.ts
@@ -14,7 +14,7 @@ export abstract class DomainEvent {
   readonly occurredOn: Date;
   readonly eventName: string;
 
-  constructor(params: { eventName: string, aggregateId: string, eventId?: string, occurredOn?: Date }) {
+  constructor(params: { eventName: string; aggregateId: string; eventId?: string; occurredOn?: Date }) {
     const { aggregateId, eventName, eventId, occurredOn } = params;
     this.aggregateId = aggregateId;
     this.eventId = eventId || Uuid.random().value;
diff --git a/src/Contexts/Shared/domain/NewableClass.ts b/src/Contexts/Shared/domain/NewableClass.ts
@@ -1,3 +1,3 @@
 export interface NewableClass<T> extends Function {
   new (...args: any[]): T;
-}
+}
diff --git a/src/Contexts/Shared/domain/value-object/IntValueObject.ts b/src/Contexts/Shared/domain/value-object/IntValueObject.ts
@@ -1,7 +1,7 @@
-import { ValueObject } from "./ValueObject";
+import { ValueObject } from './ValueObject';
 
 export abstract class NumberValueObject extends ValueObject<number> {
   isBiggerThan(other: NumberValueObject): boolean {
     return this.value > other.value;
   }
-}
+}
diff --git a/src/Contexts/Shared/domain/value-object/StringValueObject.ts b/src/Contexts/Shared/domain/value-object/StringValueObject.ts
@@ -1,4 +1,3 @@
-import { ValueObject } from "./ValueObject";
+import { ValueObject } from './ValueObject';
 
-export abstract class StringValueObject extends ValueObject<string> {
-}
+export abstract class StringValueObject extends ValueObject<string> {}
diff --git a/src/Contexts/Shared/domain/value-object/ValueObject.ts b/src/Contexts/Shared/domain/value-object/ValueObject.ts
@@ -1,10 +1,10 @@
-import { InvalidArgumentError } from "./InvalidArgumentError";
+import { InvalidArgumentError } from './InvalidArgumentError';
 
 export type Primitives = String | string | number | Boolean | boolean | Date;
 
 export abstract class ValueObject<T extends Primitives> {
   readonly value: T;
-  
+
   constructor(value: T) {
     this.value = value;
     this.ensureValueIsDefined(value);
@@ -19,8 +19,8 @@ export abstract class ValueObject<T extends Primitives> {
   equals(other: ValueObject<T>): boolean {
     return other.constructor.name === this.constructor.name && other.value === this.value;
   }
-    
-  toString(): string  {
-      return this.value.toString();
+
+  toString(): string {
+    return this.value.toString();
   }
-}
+}
diff --git a/src/Contexts/Shared/infrastructure/EventBus/DomainEventSubscribers.ts b/src/Contexts/Shared/infrastructure/EventBus/DomainEventSubscribers.ts
@@ -3,7 +3,7 @@ import { DomainEvent } from '../../domain/DomainEvent';
 import { DomainEventSubscriber } from '../../domain/DomainEventSubscriber';
 
 export class DomainEventSubscribers {
-  constructor(public items: Array<DomainEventSubscriber<DomainEvent>>) { }
+  constructor(public items: Array<DomainEventSubscriber<DomainEvent>>) {}
 
   static from(container: ContainerBuilder): DomainEventSubscribers {
     const subscriberDefinitions = container.findTaggedServiceIds('domainEventSubscriber') as Map<String, Definition>;
diff --git a/src/Contexts/Shared/infrastructure/EventBus/RabbitMq/RabbitMQConfigurer.ts b/src/Contexts/Shared/infrastructure/EventBus/RabbitMq/RabbitMQConfigurer.ts
@@ -5,12 +5,11 @@ import { RabbitMQExchangeNameFormatter } from './RabbitMQExchangeNameFormatter';
 import { RabbitMQqueueFormatter } from './RabbitMQqueueFormatter';
 
 export class RabbitMQConfigurer {
-
   constructor(
     private connection: RabbitMqConnection,
     private queueNameFormatter: RabbitMQqueueFormatter,
     private messageRetryTtl: number
-  ) { }
+  ) {}
 
   async configure(params: { exchange: string; subscribers: Array<DomainEventSubscriber<DomainEvent>> }): Promise<void> {
     const retryExchange = RabbitMQExchangeNameFormatter.retry(params.exchange);
@@ -28,9 +27,9 @@ export class RabbitMQConfigurer {
   private async addQueue(subscriber: DomainEventSubscriber<DomainEvent>, exchange: string) {
     const retryExchange = RabbitMQExchangeNameFormatter.retry(exchange);
     const deadLetterExchange = RabbitMQExchangeNameFormatter.deadLetter(exchange);
-    
+
     const routingKeys = this.getRoutingKeysFor(subscriber);
-    
+
     const queue = this.queueNameFormatter.format(subscriber);
     const deadLetterQueue = this.queueNameFormatter.formatDeadLetter(subscriber);
     const retryQueue = this.queueNameFormatter.formatRetry(subscriber);
@@ -49,7 +48,7 @@ export class RabbitMQConfigurer {
 
   private getRoutingKeysFor(subscriber: DomainEventSubscriber<DomainEvent>) {
     const routingKeys = subscriber.subscribedTo().map(event => event.EVENT_NAME);
-    
+
     const queue = this.queueNameFormatter.format(subscriber);
     routingKeys.push(queue);
 
diff --git a/src/Contexts/Shared/infrastructure/EventBus/RabbitMq/RabbitMQConsumer.ts b/src/Contexts/Shared/infrastructure/EventBus/RabbitMq/RabbitMQConsumer.ts
@@ -15,7 +15,7 @@ export class RabbitMQConsumer {
   constructor(params: {
     subscriber: DomainEventSubscriber<DomainEvent>;
     deserializer: DomainEventDeserializer;
-    connection: RabbitMqConnection
+    connection: RabbitMqConnection;
     queueName: string;
     exchange: string;
     maxRetries: Number;
@@ -43,18 +43,18 @@ export class RabbitMQConsumer {
 
   private async handleError(message: ConsumeMessage) {
     if (this.hasBeenRedeliveredTooMuch(message)) {
-      await this.deadLetter(message)
+      await this.deadLetter(message);
     } else {
       await this.retry(message);
     }
   }
 
   private async retry(message: ConsumeMessage) {
-    await this.connection.retry(message, this.queueName, this.exchange)
+    await this.connection.retry(message, this.queueName, this.exchange);
   }
 
   private async deadLetter(message: ConsumeMessage) {
-    await this.connection.deadLetter(message, this.queueName, this.exchange)
+    await this.connection.deadLetter(message, this.queueName, this.exchange);
   }
 
   private hasBeenRedeliveredTooMuch(message: ConsumeMessage) {
diff --git a/src/Contexts/Shared/infrastructure/EventBus/RabbitMq/RabbitMQConsumerFactory.ts b/src/Contexts/Shared/infrastructure/EventBus/RabbitMq/RabbitMQConsumerFactory.ts
@@ -5,16 +5,20 @@ import { RabbitMqConnection } from './RabbitMqConnection';
 import { RabbitMQConsumer } from './RabbitMQConsumer';
 
 export class RabbitMQConsumerFactory {
-  constructor(private deserializer: DomainEventDeserializer, private connection: RabbitMqConnection, private maxRetries: Number) { }
-
-  build(subscriber: DomainEventSubscriber<DomainEvent>, exchange: string, queueName: string,) {
+  constructor(
+    private deserializer: DomainEventDeserializer,
+    private connection: RabbitMqConnection,
+    private maxRetries: Number
+  ) {}
 
+  build(subscriber: DomainEventSubscriber<DomainEvent>, exchange: string, queueName: string) {
     return new RabbitMQConsumer({
-      subscriber, deserializer: this.deserializer, connection: this.connection,
+      subscriber,
+      deserializer: this.deserializer,
+      connection: this.connection,
       queueName,
       exchange,
       maxRetries: this.maxRetries
-
     });
   }
 }
diff --git a/src/Contexts/Shared/infrastructure/QueryBus/InMemoryQueryBus.ts b/src/Contexts/Shared/infrastructure/QueryBus/InMemoryQueryBus.ts
@@ -9,6 +9,6 @@ export class InMemoryQueryBus implements QueryBus {
   async ask<R extends Response>(query: Query): Promise<R> {
     const handler = this.queryHandlersInformation.get(query);
 
-    return await handler.handle(query) as Promise<R>;
+    return (await handler.handle(query)) as Promise<R>;
   }
 }
diff --git a/src/Contexts/Shared/infrastructure/persistence/mongo/MongoConfig.ts b/src/Contexts/Shared/infrastructure/persistence/mongo/MongoConfig.ts
@@ -1,6 +1,5 @@
 interface MongoConfig {
   url: string;
 }
-  
+
 export default MongoConfig;
-  
diff --git a/src/Contexts/Shared/infrastructure/persistence/typeorm/ValueObjectTransformer.ts b/src/Contexts/Shared/infrastructure/persistence/typeorm/ValueObjectTransformer.ts
@@ -1,9 +1,9 @@
-import { NewableClass } from "../../../domain/NewableClass"
-import { Primitives, ValueObject } from "../../../domain/value-object/ValueObject"
-  
+import { NewableClass } from '../../../domain/NewableClass';
+import { Primitives, ValueObject } from '../../../domain/value-object/ValueObject';
+
 export const ValueObjectTransformer = <T extends Primitives>(ValueObject: NewableClass<ValueObject<any>>) => {
   return {
     to: (value: ValueObject<T>): T => value.value,
     from: (value: T): ValueObject<T> => new ValueObject(value)
-  }
-}
+  };
+};
diff --git a/src/apps/backoffice/backend/command/ConfigureRabbitMQCommand.ts b/src/apps/backoffice/backend/command/ConfigureRabbitMQCommand.ts
@@ -20,4 +20,3 @@ export class ConfigureRabbitMQCommand {
     await connection.close();
   }
 }
-
diff --git a/src/apps/backoffice/backend/controllers/CoursesPostController.ts b/src/apps/backoffice/backend/controllers/CoursesPostController.ts
diff --git a/src/apps/mooc/backend/routes/courses-counter.route.ts b/src/apps/mooc/backend/routes/courses-counter.route.ts

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@`
`3`	`3`	`"tabWidth": 2,`
`4`	`4`	`"singleQuote": true,`
`5`	`5`	`"trailingComma": "none",`
`6`		`- "semicolons": true,`
`7`	`6`	`"quoteProps": "as-needed",`
`8`	`7`	`"arrowParens": "avoid"`
`9`	`8`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import { BackofficeCourse } from "../domain/BackofficeCourse";`
	`1`	`+import { BackofficeCourse } from '../domain/BackofficeCourse';`
`2`	`2`
`3`	`3`	`interface BackofficeCourseResponse {`
`4`	`4`	`id: string;`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`		`-import { Query } from "../../../../Shared/domain/Query";`
	`1`	`+import { Query } from '../../../../Shared/domain/Query';`
`2`	`2`
`3`	`3`	`export class SearchAllCoursesQuery implements Query {}`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`		`-import { StringValueObject } from "../../../Shared/domain/value-object/StringValueObject";`
	`1`	`+import { StringValueObject } from '../../../Shared/domain/value-object/StringValueObject';`
`2`	`2`
`3`	`3`	`export class BackofficeCourseDuration extends StringValueObject {}`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`		`-import { Uuid } from "../../../Shared/domain/value-object/Uuid";`
	`1`	`+import { Uuid } from '../../../Shared/domain/value-object/Uuid';`
`2`	`2`
`3`	`3`	`export class BackofficeCourseId extends Uuid {}`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,8 @@ import { BackofficeCourseRepository } from '../../domain/BackofficeCourseReposit`
`5`	`5`
`6`	`6`	`export class ElasticBackofficeCourseRepository`
`7`	`7`	`extends ElasticRepository<BackofficeCourse>`
`8`		`- implements BackofficeCourseRepository {`
	`8`	`+ implements BackofficeCourseRepository`
	`9`	`+{`
`9`	`10`	`async searchAll(): Promise<BackofficeCourse[]> {`
`10`	`11`	`return this.searchAllInElastic(BackofficeCourse.fromPrimitives);`
`11`	`12`	`}`