Space - Katie

lib/block.rb

@@ -0,0 +1,24 @@
+require 'date'
+
+module Hotel
+  class Block < Reservation
+    attr_reader :start_date, :end_date, :date_range, :disc_rate, :available_rooms, :reserved_rooms
+
+    def initialize(start_date, end_date, collection_of_rooms, disc_rate)
+      super(start_date,end_date)
+      @date_range = get_date_range(start_date, end_date)
+      @collection_of_rooms = collection_of_rooms
+      @disc_rate = disc_rate
+      @available_rooms = collection_of_rooms
+      @reserved_rooms = []
+    end
+
+    def reserve_room(room)
+      raise RuntimeError.new("room is not available in hotel block") unless @available_rooms.include?(room)
+
+      @available_rooms.delete(room)
+      @reserved_rooms.push(room)
+    end
+
+  end
+end

lib/date_range.rb

@@ -0,0 +1,34 @@
+require 'date'
+
+module Hotel
+  class DateRange
+    attr_accessor :start_date, :end_date
+
+    def initialize(start_date, end_date)
+      raise ArgumentError.new("start date has to be at least 1 day before end date") if (end_date - start_date).to_i < 1
+      @start_date = start_date
+      @end_date = end_date
+    end
+
+    def overlap?(range)
+      self.nights.times do |day|
+        range.nights.times do |day_in_range|
+          return true if (range.start_date + day_in_range) == (@start_date + day)
+        end
+      end
+      return false
+    end
+
+    def include?(date)
+      self.nights.times do |day|
+        return true if date == (@start_date + day)
+      end
+      return false
+    end
+
+    def nights
+      date_diff = @end_date - start_date
+      return date_diff.to_i
+    end
+  end
+end

lib/hotel_controller.rb

@@ -0,0 +1,80 @@
+require 'date'
+
+require_relative 'reservation.rb'
+require_relative 'room_reservation.rb'
+require_relative 'date_range.rb'
+require_relative 'room.rb'
+
+module Hotel
+  class HotelController
+    attr_reader :rooms, :reservations
+
+    def initialize
+      rooms_array = []
+      20.times{ |i| rooms_array.push(Hotel::Room.new(i+1)) }
+      @rooms = rooms_array
+      @reservations = []
+      @blocks = []
+    end
+
+    def list_rooms
+      return @rooms
+    end
+
+    def reserve_room(start_date, end_date, room=@rooms.sample)
+      raise RuntimeError.new("room #{room.room_number} isn't available for those dates") unless available_rooms(start_date,end_date).include?(room)
+
+      new_reservation = RoomReservation.new(start_date, end_date, room)
+      @reservations.push(new_reservation)
+      return new_reservation
+    end
+
+    def create_hotel_block(date_range,collection_of_rooms,disc_rate)
+      raise ArgumentError.new("A block can only contain up to five rooms") if collection_of_rooms.length > 5
+
+
+      collection_of_rooms.each do |room|
+        raise RuntimeError.new("#{room.room_number} is not available") if available_rooms(date_range.start_date,date_range.end_date).include?(room) == false
+
+        reserve_room(date_range.start_date,date_range.end_date,room)
+      end
+
+      new_block = Hotel::Block.new(date_range.start_date, date_range.end_date, collection_of_rooms, disc_rate)
+
+      @blocks.push(new_block)
+
+      return new_block
+
+    end
+
+    def reserve_room_from_block(block,room)
+      raise ArgumentError.new("#{room.room_number} is not available") unless block.available_rooms.include?(room)
+
+      block.reserve_room(room)
+    end
+
+    def reservations(date)
+      reservations_to_return = []
+      @reservations.each do |reservation|
+        reservations_to_return.push(reservation) if reservation.date_range.include?(date)
+      end
+
+      return reservations_to_return
+    end
+
+    def available_rooms(start_date, end_date)
+      raise ArgumentError.new("start date has to be at least 1 day before end date") if (end_date - start_date).to_i < 1
+
+      requested_range = Hotel::DateRange.new(start_date,end_date)
+      avail_rooms = rooms.clone
+      overlapping_reservations = @reservations.select { |res| res.date_range.overlap?(requested_range) }
+
+      overlapping_reservations.each do |res|
+        avail_rooms.delete_if { |room| room == res.room }
+      end
+
+      return avail_rooms
+    end
+
+  end
+end

lib/reservation.rb

@@ -0,0 +1,20 @@
+require 'date'
+
+require_relative 'date_range.rb'
+
+module Hotel
+  class Reservation
+    attr_reader :start_date, :end_date, :date_range
+
+    def initialize(start_date, end_date)
+      @start_date = start_date
+      @end_date = end_date
+    end
+
+    def get_date_range(start_date,end_date)
+      @date_range = Hotel::DateRange.new(start_date, end_date)
+      return @date_range
+    end
+
+  end
+end

lib/room.rb

@@ -0,0 +1,13 @@
+module Hotel
+  class Room
+    attr_reader :room_number, :cost
+
+    def initialize(room_number, cost: 200)
+      raise ArgumentError if room_number.class != Integer
+
+      @room_number = room_number
+      @cost = cost
+    end
+
+  end
+end

lib/room_reservation.rb

@@ -0,0 +1,23 @@
+require 'date'
+
+require_relative 'date_range.rb'
+require_relative 'room.rb'
+
+module Hotel
+  class RoomReservation < Reservation
+    attr_reader :start_date, :end_date, :room, :date_range
+
+    def initialize(start_date, end_date, room)
+      super(start_date,end_date)
+      raise ArgumentError.new("room must be an instance of room") if room.class != Hotel::Room
+
+      @room = room # instance of room class
+      @date_range = get_date_range(start_date, end_date)
+    end
+
+    def cost
+      return Hotel::DateRange.new(@start_date, @end_date).nights * @room.cost
+    end
+
+  end
+end

refactors.txt

@@ -0,0 +1,5 @@
+Changes I could have made:
+-I could have somehow changed the hotel controller class and/or block class in order to keep track of room reservations for blocks once they're made
+-I could have had the number of rooms be something that could be changed (a parameter with a default value?)
+-I could have done the structure of the reservation, room reservation, and block reservation classes differently -- currently the reservation class is the parent class, while the room reservation and block reservation classes are children classes. 
+-I could have added functionality that allows for different rates for different rooms (this would make the block discounted rate actually work)

test/block_test.rb

@@ -0,0 +1,52 @@
+require_relative "test_helper"
+
+describe Hotel::Block do
+  before do
+    @start_date = Date.new(2020,03,07)
+    @end_date = @start_date + 5
+
+    @collection_of_rooms = []
+    [6,10,13,16].each do |room_num|
+      @collection_of_rooms.push(Hotel::Room.new(room_num))
+    end
+
+    @room1 = Hotel::Room.new(1)
+    @collection_of_rooms.push(@room1)
+
+    @disc_rate = 0.2
+
+    @hotel_block = Hotel::Block.new(@start_date,@end_date,@collection_of_rooms,@disc_rate)
+    @hotel_block.reserve_room(@room1)
+
+  end
+
+
+  describe "initializes block correctly" do
+
+    it "block can be succesfully created" do
+      expect(@hotel_block).must_be_kind_of Hotel::Block
+      expect(@hotel_block.start_date).must_equal @start_date
+      expect(@hotel_block.end_date).must_equal @end_date
+      expect(@hotel_block.available_rooms).must_equal @collection_of_rooms
+      expect(@hotel_block.available_rooms).must_be_kind_of Array
+      expect(@hotel_block.available_rooms[0]).must_be_kind_of Hotel::Room
+      expect(@hotel_block.disc_rate).must_equal @disc_rate
+
+    end
+
+  end
+
+  describe "reserve_room" do
+    it "successfully reserves from block" do
+      expect(@hotel_block.reserved_rooms[0]).must_equal @room1
+      expect(@hotel_block.available_rooms.include?(@room1)).must_equal false
+    end
+
+    it "raises an exception if I try to reserve a room from the block that isn't available" do 
+      expect{@hotel_block.reserve_room(@room1)}.must_raise RuntimeError
+    end
+
+  end
+
+
+end

test/coverage/.last_run.json

@@ -0,0 +1,5 @@
+{
+  "result": {
+    "covered_percent": 100.0
+  }
+}

test/coverage/.resultset.json

@@ -0,0 +1,7 @@
+{
+  "Minitest": {
+    "coverage": {
+    },
+    "timestamp": 1583526383
+  }
+}

test/coverage/index.html

@@ -0,0 +1,93 @@
+<!DOCTYPE html>
+<html xmlns='http://www.w3.org/1999/xhtml'>
+  <head>
+    <title>Code coverage for Test</title>
+    <meta http-equiv="content-type" content="text/html; charset=utf-8" />
+    <script src='./assets/0.12.2/application.js' type='text/javascript'></script>
+    <link href='./assets/0.12.2/application.css' media='screen, projection, print' rel='stylesheet' type='text/css' />
+    <link rel="shortcut icon" type="image/png" href="./assets/0.12.2/favicon_green.png" />
+    <link rel="icon" type="image/png" href="./assets/0.12.2/favicon.png" />
+  </head>
+
+  <body>
+    <div id="loading">
+      <img src="./assets/0.12.2/loading.gif" alt="loading"/>
+    </div>
+    <div id="wrapper" class="hide">
+      <div class="timestamp">Generated <abbr class="timeago" title="2020-03-06T12:26:23-08:00">2020-03-06T12:26:23-08:00</abbr></div>
+      <ul class="group_tabs"></ul>
+
+      <div id="content">
+        <div class="file_list_container" id="AllFiles">
+  <h2>
+    <span class="group_name">All Files</span>
+    (<span class="covered_percent">
+      <span class="green">
+  100.0%
+</span>
+
+     </span>
+     covered at
+     <span class="covered_strength">
+       <span class="red">
+         0.0
+       </span>
+    </span> hits/line
+    )
+  </h2>
+
+  <a name="AllFiles"></a>
+
+  <div>
+    <b>0</b> files in total.
+  </div>
+
+  <div class="t-line-summary">
+    <b>0</b> relevant lines,
+    <span class="green"><b>0</b> lines covered</span> and
+    <span class="red"><b>0</b> lines missed. </span>
+    (<span class="green">
+  100.0%
+</span>
+)
+  </div>
+
+
+
+  <div class="file_list--responsive">
+    <table class="file_list">
+      <thead>
+        <tr>
+          <th>File</th>
+          <th class="cell--number">% covered</th>
+          <th class="cell--number">Lines</th>
+          <th class="cell--number">Relevant Lines</th>
+          <th class="cell--number">Lines covered</th>
+          <th class="cell--number">Lines missed</th>
+          <th class="cell--number">Avg. Hits / Line</th>
+
+        </tr>
+      </thead>
+      <tbody>
+
+      </tbody>
+    </table>
+  </div>
+</div>
+
+
+
+      </div>
+
+      <div id="footer">
+        Generated by <a href="https://github.com/colszowka/simplecov">simplecov</a> v0.18.5
+        and simplecov-html v0.12.2<br/>
+        using Minitest
+      </div>
+
+      <div class="source_files">
+
+      </div>
+    </div>
+  </body>
+</html>