suitcase 1.6.0 → 1.6.1

data/lib/suitcase/hotel/hotel.rb

@@ -76,7 +76,8 @@ module Suitcase
                   :location_description, :short_description,
                   :hotel_in_destination, :proximity_distance,
                   :property_description, :number_of_floors, :number_of_rooms,
-                  :deep_link, :tripadvisor_rating
+                  :deep_link, :tripadvisor_rating, :general_policies,
+                  :checkin_instructions, :general_policies
 
     # Internal: Initialize a new Hotel.
     #
@@ -211,15 +212,23 @@ module Suitcase
     # Returns a reformatted Hash with the specified accessors.
     def self.parse_information(parsed)
       handle_errors(parsed)
+      
+      binding.pry
 
-      summary = if parsed["hotelId"]
-                  parsed
-                else
-                  parsed["HotelInformationResponse"]["HotelSummary"]
-                end
+      if parsed["hotelId"]
+        summary = parsed
+        parsed_info = {}
+      else
+        res = parsed["HotelInformationResponse"]
+        summary = res["HotelSummary"]
+        parsed_info = {
+          general_policies: res["HotelDetails"]["hotelPolicy"],
+          checkin_instructions: res["HotelDetails"]["checkInInstructions"]
+        }
+      end
       proximity_distance = summary["proximityDistance"].to_s
       proximity_distance << summary["proximityUnit"].to_s
-      parsed_info = {
+      parsed_info.merge!(
         id: summary["hotelId"],
         name: summary["name"],
         address: summary["address1"],
@@ -237,7 +246,7 @@ module Suitcase
         proximity_distance: proximity_distance,
         tripadvisor_rating: summary["tripAdvisorRating"],
         deep_link: summary["deepLink"]
-      }
+      )
       parsed_info[:amenities] = parsed["HotelInformationResponse"]["PropertyAmenities"]["PropertyAmenity"].map do |x|
         Amenity.new(id: x["amenityId"], description: x["amenity"])
       end if parsed["HotelInformationResponse"]
@@ -353,6 +362,9 @@ module Suitcase
 
       parsed["HotelRoomAvailabilityResponse"]["HotelRoomResponse"].map do |raw_data|
         room_data = {}
+        room_data[:non_refundable] = raw_data["nonRefundable"]
+        room_data[:deposit_required] = raw_data["depositRequired"]
+        room_data[:cancellation_policy] = raw_data["cancellationPolicy"]
         room_data[:rate_code] = raw_data["rateCode"]
         room_data[:room_type_code] = raw_data["roomTypeCode"]
         room_data[:room_type_description] = raw_data["roomTypeDescription"]
@@ -372,6 +384,7 @@ module Suitcase
         room_data[:bed_types] = [raw_data["BedTypes"]["BedType"]].flatten.map do |x|
           BedType.new(id: x["@id"], description: x["description"])
         end if raw_data["BedTypes"] && raw_data["BedTypes"]["BedType"]
+
         Room.new(room_data)
       end
     end

data/lib/suitcase/hotel/room.rb

@@ -8,7 +8,8 @@ module Suitcase
                     :average_rate, :max_nightly_rate, :currency_code, :value_adds,
                     :room_type_description, :price_breakdown, :total_price,
                     :average_nightly_rate, :promo, :arrival, :departure, :rooms,
-                    :bed_types
+                    :bed_types, :cancellation_policy, :non_refundable,
+                    :deposit_required
 
       extend Helpers
 

data/lib/suitcase/version.rb

@@ -1,3 +1,3 @@
 module Suitcase
-  VERSION = "1.6.0"
+  VERSION = "1.6.1"
 end

data/test/hotels/hotel_test.rb

@@ -90,5 +90,24 @@ describe Suitcase::Hotel do
     it "returns an Array of available Rooms" do
       @rooms.must_be_kind_of(Array)
     end
+
+    it "each room must contain a cancellation policy" do
+      @rooms.each do |r|
+        r.cancellation_policy.wont_be_nil
+      end
+    end
+
+    it "each room must contain the non_refundable option" do
+      @rooms.each do |r|
+        r.non_refundable.wont_be_nil
+      end
+    end
+
+    it "each room must contain the deposit_required" do
+      @rooms.each do |r|
+        r.deposit_required.wont_be_nil
+      end
+      binding.pry
+    end
   end
 end

data/test/keys.rb

@@ -15,21 +15,21 @@ Suitcase.configure do |config|
 end
 
 module Keys
-  SUITCASE_PAYMENT_OPTION = Suitcase::Hotel::PaymentOption.find(currency_code: "USD")[0]
-  CREDIT_CARD_NUMBER_TESTING = "1234123412341234"
+  SUITCASE_PAYMENT_OPTION = Suitcase::Hotel::PaymentOption.find(currency_code: "USD").find { |po| po.name =~ /Master/ }
+  CREDIT_CARD_NUMBER_TESTING = "5401999999999999"
   CREDIT_CARD_CVV_TESTING = "123"
-  CREDIT_CARD_EXPIRATION_DATE_TESTING = "3/14"
+  CREDIT_CARD_EXPIRATION_DATE_TESTING = "2014/03/01"
 
   VALID_RESERVATION_INFO = {
     email: "testemail@gmail.com",
-    first_name: "Test",
-    last_name: "User",
+    first_name: "Test Booking",
+    last_name: "Test Booking",
     home_phone: "1231231234",
     payment_option: SUITCASE_PAYMENT_OPTION,
     credit_card_number: CREDIT_CARD_NUMBER_TESTING, 
-    credit_card_verification_code: "123",
+    credit_card_verification_code: CREDIT_CARD_CVV_TESTING,
     credit_card_expiration_date: CREDIT_CARD_EXPIRATION_DATE_TESTING,
-    address1: "1 Some Place",
+    address1: "travelnow",
     city: "Boston",
     province: "MA",
     country: "US",

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: suitcase
 version: !ruby/object:Gem::Version
-  version: 1.6.0
+  version: 1.6.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-17 00:00:00.000000000 Z
+date: 2012-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest