makersacademy · lukestorey95 · Apr 23, 2022 · Apr 23, 2022 · Apr 23, 2022 · Apr 23, 2022
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -1,24 +1,24 @@
 # Your name
 
-Please write your full name here to make it easier to find your pull request.
+Luke Storey
 
-# User stories 
+# User stories
 
 Please list which user stories you've implemented (delete the ones that don't apply).
 
-- [ ] User story 1: "I want to instruct a plane to land at an airport"
-- [ ] User story 2: "I want to instruct a plane to take off from an airport and confirm that it is no longer in the airport"
-- [ ] User story 3: "I want to prevent landing when the airport is full"
-- [ ] User story 4: "I would like a default airport capacity that can be overridden as appropriate"
-- [ ] User story 5: "I want to prevent takeoff when weather is stormy"
-- [ ] User story 6: "I want to prevent landing when weather is stormy"
+- [x] User story 1: "I want to instruct a plane to land at an airport"
+- [x] User story 2: "I want to instruct a plane to take off from an airport and confirm that it is no longer in the airport"
+- [x] User story 3: "I want to prevent landing when the airport is full"
+- [x] User story 4: "I would like a default airport capacity that can be overridden as appropriate"
+- [x] User story 5: "I want to prevent takeoff when weather is stormy"
+- [x] User story 6: "I want to prevent landing when weather is stormy"
 
 # README checklist
 
 Does your README contains instructions for
 
-- [ ] how to install,
-- [ ] how to run,
-- [ ] and how to test your code?
+- [x] how to install,
+- [x] how to run,
+- [x] and how to test your code?
 
-[Here is a pill](https://github.com/makersacademy/course/blob/main/pills/readmes.md) that can help you write a great README!
+[Here is a pill](https://github.com/makersacademy/course/blob/main/pills/readmes.md) that can help you write a great README!
diff --git a/Gemfile b/Gemfile
@@ -6,6 +6,7 @@ group :test do
   gem 'rspec'
   gem 'simplecov', require: false
   gem 'simplecov-console', require: false
+  gem 'simplecov-shields-badge', require: false
 end
 
 group :development, :test do

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -45,6 +45,8 @@ GEM
       simplecov
       terminal-table
     simplecov-html (0.12.3)
+    simplecov-shields-badge (0.1.0)
+      simplecov (~> 0.15)
     simplecov_json_formatter (0.1.3)
     terminal-table (3.0.1)
       unicode-display_width (>= 1.1.1, < 3)
@@ -58,6 +60,7 @@ DEPENDENCIES
   rubocop (= 1.20)
   simplecov
   simplecov-console
+  simplecov-shields-badge
 
 RUBY VERSION
    ruby 3.0.2p107

diff --git a/README.md b/README.md
@@ -1,89 +1,187 @@
-Airport Challenge
-=================
+# Airport Challenge
 
+[![Ruby Style Guide](https://img.shields.io/badge/code_style-rubocop-brightgreen.svg)](https://github.com/rubocop/rubocop)
+[![Coverage](./badge.svg)](https://github.com/lukestorey95/airport_challenge)
+
+This program allows the user to simulate a working airport. Planes are able to land and take off from an airport so long as the weather isn't stormy.
+
+<br>
+
+## Installation
+
+```
+$ git clone https://github.com/lukestorey95/airport_challenge.git
+
+$ cd airport_challenge
+
+$ bundle
 ```
-        ______
-        _\____\___
-=  = ==(____MA____)
-          \_____\___________________,-~~~~~~~`-.._
-          /     o o o o o o o o o o o o o o o o  |\_
-          `~-.__       __..----..__                  )
-                `---~~\___________/------------`````
-                =  ===(_________)
+
+<br>
+
+## Quickstart
+
+```
+irb
+
+> Dir['./lib/*.rb'].each {|file| require file }
+
+> airport = Airport.new
+> plane = Plane.new
+
+> airport.land(plane)
+
+> airport.take_off(plane)
+```
+
+> For more usage please see user stories below
+
+<br>
+
+## Running Tests
+
+```
+$ rspec
+```
+
+<details>
+  <summary>Click to see test results</summary>
+
+![Test results](./test_results.png)
+
+</details>
+
+<br>
+
+## My Process
+
+1. Break down user story into objects and behaviour
+2. Feature test and note down errors/expected errors
+3. Write failing test that replicates errors
+4. Write the minimum code to make test pass
+5. Refactor and ensure tests still pass
+6. Repeat step 2 and ensure behaviour works as intended
+
+<br>
+
+## User Stories
+
+> I want to instruct a **_plane_** to **_land_** at an **_airport_**
 
 ```
+airport = Airport.new
+plane = Plane.new
+airport.land(plane)
+```
+
+---
+
+> I want to instruct a **_plane_** to **_take off_** from an **_airport_** and **_confirm_** that it is no longer in the **_airport_**
 
-Instructions
----------
+```
+airport = Airport.new
+plane = Plane.new
+airport.land(plane)
+airport.take_off(plane)
+airport.planes
+```
 
-* Feel free to use google, your notes, books, etc. but work on your own
-* If you refer to the solution of another coach or student, please put a link to that in your README
-* If you have a partial solution, **still check in a partial solution**
-* You must submit a pull request to this repo with your code by 10am Monday morning
+---
 
-Steps
--------
+> I want to **_prevent landing_** when the **_airport_** is **_full_**
 
-1. Fork this repo, and clone to your local machine
-2. Run the command `gem install bundler` (if you don't have bundler already)
-3. When the installation completes, run `bundle`
-4. Complete the following task:
+```
+airport = Airport.new
+plane = Plane.new
+airport.land(plane)
+airport.land(plane)
+```
 
-Task
------
+---
 
-We have a request from a client to write the software to control the flow of planes at an airport. The planes can land and take off provided that the weather is sunny. Occasionally it may be stormy, in which case no planes can land or take off.  Here are the user stories that we worked out in collaboration with the client:
+> I would like a **_default airport capacity_** that can **_be overridden_** as appropriate
 
 ```
-As an air traffic controller 
-So I can get passengers to a destination 
-I want to instruct a plane to land at an airport
+airport = Airport.new(3)
+plane = Plane.new
+airport.capacity.times { airport.land(plane) }
+airport.land(plane)
+```
 
-As an air traffic controller 
-So I can get passengers on the way to their destination 
-I want to instruct a plane to take off from an airport and confirm that it is no longer in the airport
+---
 
-As an air traffic controller 
-To ensure safety 
-I want to prevent landing when the airport is full 
+> I want to **_prevent takeoff_** when **_weather_** is stormy
 
-As the system designer
-So that the software can be used for many different airports
-I would like a default airport capacity that can be overridden as appropriate
+```
+airport = Airport.new
+plane = Plane.new
+weather = Weather.new
+airport.land(plane)
+airport.take_off(plane)
+#=> should raise error if weather is stormy
+```
 
-As an air traffic controller 
-To ensure safety 
-I want to prevent takeoff when weather is stormy 
+---
 
-As an air traffic controller 
-To ensure safety 
-I want to prevent landing when weather is stormy 
+> I want to **_prevent landing_** when **_weather_** is stormy
+
+```
+airport = Airport.new
+plane = Plane.new
+weather = Weather.new
+airport.land(plane)
+#=> should raise error if weather is stormy
 ```
 
-Your task is to test drive the creation of a set of classes/modules to satisfy all the above user stories. You will need to use a random number generator to set the weather (it is normally sunny but on rare occasions it may be stormy). In your tests, you'll need to use a stub to override random weather to ensure consistent test behaviour.
+<br>
 
-Your code should defend against [edge cases](http://programmers.stackexchange.com/questions/125587/what-are-the-difference-between-an-edge-case-a-corner-case-a-base-case-and-a-b) such as inconsistent states of the system ensuring that planes can only take off from airports they are in; planes that are already flying cannot take off and/or be in an airport; planes that are landed cannot land again and must be in an airport, etc.
+## Edge Cases
 
-For overriding random weather behaviour, please read the documentation to learn how to use test doubles: https://www.relishapp.com/rspec/rspec-mocks/docs . There’s an example of using a test double to test a die that’s relevant to testing random weather in the test.
+> I want to **_prevent_** a **_plane_** from **_taking off_**, if it is not in the correct **_airport_**
 
-Please create separate files for every class, module and test suite.
+```
+airport = Airport.new
+plane = Plane.new
+airport.take_off(plane)
+#=> should raise error
+```
 
-In code review we'll be hoping to see:
+> I want to **_prevent_** a **_landed plane_** from **_landing_** again
 
-* All tests passing
-* High [Test coverage](https://github.com/makersacademy/course/blob/main/pills/test_coverage.md) (>95% is good)
-* The code is elegant: every class has a clear responsibility, methods are short etc. 
+```
+airport = Airport.new
+plane = Plane.new
+airport.land(plane)
+airport.land(plane)
+#=> should raise error
+```
 
-Reviewers will potentially be using this [code review rubric](docs/review.md).  Referring to this rubric in advance will make the challenge somewhat easier.  You should be the judge of how much challenge you want this at this moment.
+> I want to **_prevent_** a **_flying plane_** from **_taking off_** again
 
-**BONUS**
+```
+airport = Airport.new
+plane = Plane.new
+airport.land(plane)
+airport.take_off(plane)
+airport.take_off(plane)
+#=> should raise error
+```
+
+<br>
+
+## Feature Test
 
-* Write an RSpec **feature** test that lands and takes off a number of planes
+> I want to **_land_** and then **_take off_** several **_planes_** at once
 
-Note that is a practice 'tech test' of the kinds that employers use to screen developer applicants.  More detailed submission requirements/guidelines are in [CONTRIBUTING.md](CONTRIBUTING.md)
+```
+airport = Airport.new(3)
+planes = Array.new(3) { Plane.new }
+
+planes.each { |plane| airport.land(plane) }
 
-Finally, don’t overcomplicate things. This task isn’t as hard as it may seem at first.
+# airport should contain the planes and the planes should be landed
 
-* **Submit a pull request early.**
+planes.each { |plane| airport.take_off(plane) }
 
-* Finally, please submit a pull request before Monday at 10am with your solution or partial solution.  However much or little amount of code you wrote please please please submit a pull request before Monday at 10am.
+# airport should not contain the planes and the planes should be flying
+```
diff --git a/README_INSTRUCTIONS.md b/README_INSTRUCTIONS.md
@@ -0,0 +1,89 @@
+Airport Challenge
+=================
+
+```
+        ______
+        _\____\___
+=  = ==(____MA____)
+          \_____\___________________,-~~~~~~~`-.._
+          /     o o o o o o o o o o o o o o o o  |\_
+          `~-.__       __..----..__                  )
+                `---~~\___________/------------`````
+                =  ===(_________)
+
+```
+
+Instructions
+---------
+
+* Feel free to use google, your notes, books, etc. but work on your own
+* If you refer to the solution of another coach or student, please put a link to that in your README
+* If you have a partial solution, **still check in a partial solution**
+* You must submit a pull request to this repo with your code by 10am Monday morning
+
+Steps
+-------
+
+1. Fork this repo, and clone to your local machine
+2. Run the command `gem install bundler` (if you don't have bundler already)
+3. When the installation completes, run `bundle`
+4. Complete the following task:
+
+Task
+-----
+
+We have a request from a client to write the software to control the flow of planes at an airport. The planes can land and take off provided that the weather is sunny. Occasionally it may be stormy, in which case no planes can land or take off.  Here are the user stories that we worked out in collaboration with the client:
+
+```
+As an air traffic controller 
+So I can get passengers to a destination 
+I want to instruct a plane to land at an airport
+
+As an air traffic controller 
+So I can get passengers on the way to their destination 
+I want to instruct a plane to take off from an airport and confirm that it is no longer in the airport
+
+As an air traffic controller 
+To ensure safety 
+I want to prevent landing when the airport is full 
+
+As the system designer
+So that the software can be used for many different airports
+I would like a default airport capacity that can be overridden as appropriate
+
+As an air traffic controller 
+To ensure safety 
+I want to prevent takeoff when weather is stormy 
+
+As an air traffic controller 
+To ensure safety 
+I want to prevent landing when weather is stormy 
+```
+
+Your task is to test drive the creation of a set of classes/modules to satisfy all the above user stories. You will need to use a random number generator to set the weather (it is normally sunny but on rare occasions it may be stormy). In your tests, you'll need to use a stub to override random weather to ensure consistent test behaviour.
+
+Your code should defend against [edge cases](http://programmers.stackexchange.com/questions/125587/what-are-the-difference-between-an-edge-case-a-corner-case-a-base-case-and-a-b) such as inconsistent states of the system ensuring that planes can only take off from airports they are in; planes that are already flying cannot take off and/or be in an airport; planes that are landed cannot land again and must be in an airport, etc.
+
+For overriding random weather behaviour, please read the documentation to learn how to use test doubles: https://www.relishapp.com/rspec/rspec-mocks/docs . There’s an example of using a test double to test a die that’s relevant to testing random weather in the test.
+
+Please create separate files for every class, module and test suite.
+
+In code review we'll be hoping to see:
+
+* All tests passing
+* High [Test coverage](https://github.com/makersacademy/course/blob/main/pills/test_coverage.md) (>95% is good)
+* The code is elegant: every class has a clear responsibility, methods are short etc. 
+
+Reviewers will potentially be using this [code review rubric](docs/review.md).  Referring to this rubric in advance will make the challenge somewhat easier.  You should be the judge of how much challenge you want this at this moment.
+
+**BONUS**
+
+* Write an RSpec **feature** test that lands and takes off a number of planes
+
+Note that is a practice 'tech test' of the kinds that employers use to screen developer applicants.  More detailed submission requirements/guidelines are in [CONTRIBUTING.md](CONTRIBUTING.md)
+
+Finally, don’t overcomplicate things. This task isn’t as hard as it may seem at first.
+
+* **Submit a pull request early.**
+
+* Finally, please submit a pull request before Monday at 10am with your solution or partial solution.  However much or little amount of code you wrote please please please submit a pull request before Monday at 10am.