From a235f202767cd68ec320edb8b24550bf9e10c225 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dani=20Santamar=C3=ADa?= <santaka87@gmail.com>
Date: Mon, 19 Apr 2021 10:51:10 +0200
Subject: [PATCH] refactor

---
 .../src/Application/GetStepDuration.ts        |  3 +-
 .../src/Domain/StepId.ts                      |  3 +-
 .../tests/Application/GetStepDuration.test.ts | 30 ++++++++++++-------
 3 files changed, 21 insertions(+), 15 deletions(-)

diff --git a/examples/typescript/ts-step_shotgun_surgery-01_base/src/Application/GetStepDuration.ts b/examples/typescript/ts-step_shotgun_surgery-01_base/src/Application/GetStepDuration.ts
index 3ca0908f..72e1222f 100644
--- a/examples/typescript/ts-step_shotgun_surgery-01_base/src/Application/GetStepDuration.ts
+++ b/examples/typescript/ts-step_shotgun_surgery-01_base/src/Application/GetStepDuration.ts
@@ -3,8 +3,7 @@ import StepRepository from '../Domain/StepRepository'
 import StepDurationCalculatorFactory from "../Domain/StepDurationCalculatorFactory";
 
 class GetStepDuration {
-    constructor(private repository: StepRepository) {
-    }
+    constructor(private repository: StepRepository) {}
 
     execute(stepId: string): number {
         const step = this.repository.find(new StepId(stepId))
diff --git a/examples/typescript/ts-step_shotgun_surgery-01_base/src/Domain/StepId.ts b/examples/typescript/ts-step_shotgun_surgery-01_base/src/Domain/StepId.ts
index 2046471e..9b2ee43d 100644
--- a/examples/typescript/ts-step_shotgun_surgery-01_base/src/Domain/StepId.ts
+++ b/examples/typescript/ts-step_shotgun_surgery-01_base/src/Domain/StepId.ts
@@ -1,6 +1,5 @@
 class StepId {
-    constructor(private stepId: string) {
-    }
+    constructor(private stepId: string) {}
 
     value(): string {
         return this.stepId
diff --git a/examples/typescript/ts-step_shotgun_surgery-01_base/tests/Application/GetStepDuration.test.ts b/examples/typescript/ts-step_shotgun_surgery-01_base/tests/Application/GetStepDuration.test.ts
index ca56467b..40751ab5 100644
--- a/examples/typescript/ts-step_shotgun_surgery-01_base/tests/Application/GetStepDuration.test.ts
+++ b/examples/typescript/ts-step_shotgun_surgery-01_base/tests/Application/GetStepDuration.test.ts
@@ -2,25 +2,33 @@ import GetStepDuration from '../../src/Application/GetStepDuration';
 import StepId from "../../src/Domain/StepId";
 import VideoStep from "../../src/Domain/VideoStep";
 import QuizStep from "../../src/Domain/QuizStep";
+import Step from "../../src/Domain/Step";
+import StepRepository from "../../src/Domain/StepRepository";
 
-test('should get the video step duration', () => {
+test('should get video step duration', () => {
     const stepId = new StepId('stepId')
     const step = new VideoStep(stepId, 13)
-    const stepRepository = {
-        find: jest.fn(() => step)
-    }
+    const stepRepository = stepRepositoryWith(step)
     const getStepDuration = new GetStepDuration(stepRepository)
 
-    expect(getStepDuration.execute(stepId.value())).toBe(14.3)
+    const duration = getStepDuration.execute(stepId.value())
+
+    expect(duration).toBe(14.3)
 });
 
-test('should get the quiz step duration', () => {
+test('should get quiz step duration', () => {
     const stepId = new StepId('stepId')
     const step = new QuizStep(stepId, 5)
-    const stepRepository = {
-        find: jest.fn(() => step)
-    }
+    const stepRepository = stepRepositoryWith(step)
     const getStepDuration = new GetStepDuration(stepRepository)
 
-    expect(getStepDuration.execute(stepId.value())).toBe(37.5)
-});
\ No newline at end of file
+    const duration = getStepDuration.execute(stepId.value())
+
+    expect(duration).toBe(37.5)
+});
+
+function stepRepositoryWith(step: Step): StepRepository {
+    return {
+        find: jest.fn(() => step)
+    }
+}
\ No newline at end of file