test now pases as guard clause added to 'clearForLanding' function

Rhodine-orleans-lindsay · Jan 31, 2020 · 10e2839 · 10e2839
1 parent e324a6d
commit 10e2839
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/JavaScriptAirport/spec/airportSpec.js b/JavaScriptAirport/spec/airportSpec.js
@@ -55,5 +55,12 @@ describe('Airport', function(){
          airport.clearForTakeoff(plane);
         }).toThrowError('Takeoff prohibited due to storm conditions');
     });
+
+    it('does not clear planes for landing when stormy', function(){
+      spyOn(airport,'isStormy').and.returnValue(true);
+      expect(function(){
+         airport.clearForLanding(plane);
+       }).toThrowError('Landing prohibited due to storm conditions');
+    });
   });
 });
diff --git a/JavaScriptAirport/src/airport.js b/JavaScriptAirport/src/airport.js
@@ -11,9 +11,12 @@ class Airport{
   };
 
   clearForLanding(plane) {
+    if(this.isStormy()){
+      throw new Error('Landing prohibited due to storm conditions');
+    };
     if(this._runway.length  == this.capacity){
         throw new Error("Airport Full!");
-      }
+      };
     this._runway.push(plane);
   };