Fixing bug related to skier crash state

src/Entities/Skier.js

@@ -61,7 +61,16 @@ export class Skier extends Entity {
         this.y -= Constants.SKIER_STARTING_SPEED;
     }
 
+	checkCrashState(direction) {
+		if ( this.direction === Constants.SKIER_DIRECTIONS.CRASH ) {
+			this.setDirection(direction);
+		}
+	}
+
     turnLeft() {
+
+		this.checkCrashState(Constants.SKIER_DIRECTIONS.LEFT);
+
         if(this.direction === Constants.SKIER_DIRECTIONS.LEFT) {
             this.moveSkierLeft();
         }
@@ -71,11 +80,14 @@ export class Skier extends Entity {
     }
 
     turnRight() {
+
+		this.checkCrashState(Constants.SKIER_DIRECTIONS.RIGHT);
+
         if(this.direction === Constants.SKIER_DIRECTIONS.RIGHT) {
             this.moveSkierRight();
         }
         else {
-            this.setDirection(this.direction + 1);
+			this.setDirection(this.direction + 1);
         }
     }
 
@@ -115,4 +127,4 @@ export class Skier extends Entity {
             this.setDirection(Constants.SKIER_DIRECTIONS.CRASH);
         }
     };
-}
+}

src/Entities/Skier.test.js

@@ -0,0 +1,42 @@
+import "babel-polyfill";
+import * as Constants from "../Constants";
+import { Skier } from "./Skier";
+
+
+describe('when skier crashed', () => {
+
+	let skier, fake;
+
+	beforeEach(() => {
+  		skier = new Skier(0, 0);
+		fake = jest.fn();
+		skier.direction = Constants.SKIER_DIRECTIONS.CRASH;
+	});
+
+	describe('on left key stroke', () => {
+
+		test('turn left should set direction as left', () => {
+			skier.turnLeft();
+    		expect(skier.direction).toBe(Constants.SKIER_DIRECTIONS.LEFT);
+		});
+
+		test('turn left should move skier to left side', () => {
+			skier.moveSkierLeft = fake;
+			skier.turnLeft();
+    		expect(fake.mock.calls.length).toBe(1);
+		});
+	})
+
+	describe('on righ  key stroke', () => {
+		test('turn righ should set direction as right', () => {
+			skier.turnRight();
+    		expect(skier.direction).toBe(Constants.SKIER_DIRECTIONS.RIGHT);
+		});
+
+		test('turn right should move skier to right side', () => {
+			skier.moveSkierRight = fake;
+			skier.turnRight();
+    		expect(fake.mock.calls.length).toBe(1);
+		});
+	});
+});