bookingsync-api 0.1.3 → 0.1.4

data/spec/bookingsync/api/client/bookings_fees_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::BookingsFees do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".bookings_fees", :vcr do
     it "returns bookings fees" do
       expect(client.bookings_fees).not_to be_empty
@@ -11,9 +13,13 @@ describe BookingSync::API::Client::BookingsFees do
   end
 
   describe ".bookings_fee", :vcr do
+    let(:prefetched_bookings_fee_id) {
+      find_resource("#{@casette_base_path}_bookings_fees/returns_bookings_fees.yml", "bookings_fees")[:id]
+    }
+
     it "returns a single bookings_fee" do
-      bookings_fee = client.bookings_fee(8112)
-      expect(bookings_fee.id).to eq 8112
+      bookings_fee = client.bookings_fee(prefetched_bookings_fee_id)
+      expect(bookings_fee.id).to eq prefetched_bookings_fee_id
     end
   end
 end

data/spec/bookingsync/api/client/bookings_payments_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::BookingsPayments do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".bookings_payments", :vcr do
     it "returns bookings_payments" do
       expect(client.bookings_payments).not_to be_empty
@@ -11,9 +13,13 @@ describe BookingSync::API::Client::BookingsPayments do
   end
 
   describe ".bookings_payment", :vcr do
+    let(:prefetched_bookings_payment_id) {
+      find_resource("#{@casette_base_path}_bookings_payments/returns_bookings_payments.yml", "bookings_payments")[:id]
+    }
+
     it "returns a single bookings_payment" do
-      bookings_payment = client.bookings_payment(71982)
-      expect(bookings_payment.id).to eq 71982
+      bookings_payment = client.bookings_payment(prefetched_bookings_payment_id)
+      expect(bookings_payment.id).to eq prefetched_bookings_payment_id
     end
   end
 end

data/spec/bookingsync/api/client/bookings_spec.rb

@@ -3,26 +3,28 @@ require "spec_helper"
 describe BookingSync::API::Client::Bookings do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".bookings", :vcr do
     it "returns bookings" do
-      expect(client.bookings).not_to be_empty
-      assert_requested :get, bs_url("bookings")
+      expect(client.bookings(include_canceled: true)).not_to be_empty
+      assert_requested :get, bs_url("bookings?include_canceled=true")
     end
 
     describe "pagination" do
       context "with per_page setting" do
         it "returns limited number of bookings" do
           bookings = client.bookings(per_page: 2)
-          expect(bookings.size).to eql(2)
+          expect(bookings.size).to eq(2)
         end
       end
 
       context "with a block" do
         it "yields block with batch of bookings" do
-          sizes = [2, 2, 1]
+          sizes = [2, 2]
           index = 0
           client.bookings(per_page: 2) do |bookings|
-            expect(bookings.size).to eql(sizes[index])
+            expect(bookings.size).to eq(sizes[index])
             index += 1
           end
         end
@@ -31,56 +33,66 @@ describe BookingSync::API::Client::Bookings do
       context "with auto_paginate: true" do
         it "returns all bookings joined from many requests" do
           bookings = client.bookings(per_page: 2, auto_paginate: true)
-          expect(bookings.size).to eql(5)
+          expect(bookings.size).to eq(4)
         end
       end
     end
   end
 
   describe ".booking", :vcr do
+    let(:bookings) {
+      find_resources("#{@casette_base_path}_bookings/returns_bookings.yml", "bookings")
+    }
+
     it "returns a single booking" do
-      booking = client.booking(20)
+      prefetched_booking_id = bookings.detect { |b| b["status"] == "Booked" }["id"]
+      booking = client.booking(prefetched_booking_id)
       expect(booking.status).to eq "Booked"
     end
 
     it "returns a single canceled booking" do
-      booking = client.booking(114760, include_canceled: true)
+      prefetched_canceled_booking_id = bookings.detect { |b| b["status"] == "Canceled" }["id"]
+      booking = client.booking(prefetched_canceled_booking_id, include_canceled: true)
       expect(booking.status).to eq "Canceled"
     end
   end
 
   describe ".create_booking", :vcr do
     let(:attributes) {
-      { start_at: "2014-01-03", end_at: "2014-01-04",
+      { start_at: "2017-01-03", end_at: "2017-01-04",
         booked: true }
     }
-    let(:rental) { BookingSync::API::Resource.new(client, id: 20) }
+    let(:rental) { BookingSync::API::Resource.new(client, id: 5116) }
 
     it "creates a booking" do
       client.create_booking(rental, attributes)
-      assert_requested :post, bs_url("rentals/20/bookings"),
+      assert_requested :post, bs_url("rentals/#{rental}/bookings"),
         body: { bookings: [attributes] }.to_json
     end
 
     it "returns newly created booking" do
       VCR.use_cassette("BookingSync_API_Client_Bookings/_create_booking/creates_a_booking") do
         booking = client.create_booking(rental, attributes)
-        expect(booking.account_id).to eql(1)
-        expect(booking.rental_id).to eql(20)
+        expect(booking.links.account).to eq(3837)
+        expect(booking.links.rental).to eq(rental.id)
       end
     end
   end
 
   describe ".edit_booking", :vcr do
+    let(:created_booking_id) {
+      find_resource("#{@casette_base_path}_create_booking/creates_a_booking.yml", "bookings")[:id]
+    }
+
     it "updates given booking by ID" do
-      client.edit_booking(50, end_at: "2019-03-25 21:45:00 UTC")
-      assert_requested :put, bs_url("bookings/50"),
+      client.edit_booking(created_booking_id, end_at: "2019-03-25 21:45:00 UTC")
+      assert_requested :put, bs_url("bookings/#{created_booking_id}"),
         body: { bookings: [{ end_at: "2019-03-25 21:45:00 UTC" }] }.to_json
     end
 
     it "returns updated booking" do
       VCR.use_cassette("BookingSync_API_Client_Bookings/_edit_booking/updates_given_booking_by_ID") do
-        booking = client.edit_booking(50, end_at: "2019-03-25 21:45:00 UTC")
+        booking = client.edit_booking(created_booking_id, end_at: "2019-03-25 21:45:00 UTC")
         expect(booking).to be_kind_of(BookingSync::API::Resource)
         expect(booking.end_at).to eq(Time.parse("2019-03-25 21:45:00 UTC"))
       end
@@ -88,18 +100,22 @@ describe BookingSync::API::Client::Bookings do
 
     it "updates given booking by Resource object" do
       VCR.use_cassette("BookingSync_API_Client_Bookings/_edit_booking/updates_given_booking_by_ID") do
-        resource = BookingSync::API::Resource.new(nil, id: 50)
+        resource = BookingSync::API::Resource.new(nil, id: created_booking_id)
         client.edit_booking(resource, end_at: "2019-03-25 21:45:00 UTC")
-        assert_requested :put, bs_url("bookings/50"),
+        assert_requested :put, bs_url("bookings/#{resource}"),
           body: { bookings: [{ end_at: "2019-03-25 21:45:00 UTC" }] }.to_json
       end
     end
   end
 
   describe ".cancel_booking", :vcr do
+    let(:created_booking_id) {
+      find_resource("#{@casette_base_path}_create_booking/creates_a_booking.yml", "bookings")[:id]
+    }
+
     it "cancels given booking" do
-      client.cancel_booking(50)
-      assert_requested :delete, bs_url("bookings/50")
+      client.cancel_booking(created_booking_id)
+      assert_requested :delete, bs_url("bookings/#{created_booking_id}")
     end
   end
 end

data/spec/bookingsync/api/client/bookings_taxes_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::BookingsTaxes do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".bookings_taxes", :vcr do
     it "returns bookings taxes" do
       expect(client.bookings_taxes).not_to be_empty
@@ -11,9 +13,13 @@ describe BookingSync::API::Client::BookingsTaxes do
   end
 
   describe ".bookings_tax", :vcr do
+    let(:prefetched_bookings_tax_id) {
+      find_resource("#{@casette_base_path}_bookings_taxes/returns_bookings_taxes.yml", "bookings_taxes")[:id]
+    }
+
     it "returns a single bookings_tax" do
-      bookings_tax = client.bookings_tax(8157)
-      expect(bookings_tax.id).to eq 8157
+      bookings_tax = client.bookings_tax(prefetched_bookings_tax_id)
+      expect(bookings_tax.id).to eq prefetched_bookings_tax_id
     end
   end
 end

data/spec/bookingsync/api/client/change_overs_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::ChangeOvers do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".change_overs", :vcr do
     it "returns change_overs" do
       expect(client.change_overs).not_to be_empty
@@ -11,9 +13,13 @@ describe BookingSync::API::Client::ChangeOvers do
   end
 
   describe ".change_over", :vcr do
+    let(:prefetched_change_over_id) {
+      find_resource("#{@casette_base_path}_change_overs/returns_change_overs.yml", "change_overs")[:id]
+    }
+
     it "returns a single change_over" do
-      change_over = client.change_over(2)
-      expect(change_over.id).to eq 2
+      change_over = client.change_over(prefetched_change_over_id)
+      expect(change_over.id).to eq prefetched_change_over_id
     end
   end
 end

data/spec/bookingsync/api/client/clients_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::Clients do
   let(:api) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".clients", :vcr do
     it "returns clients" do
       expect(api.clients).not_to be_empty
@@ -11,19 +13,23 @@ describe BookingSync::API::Client::Clients do
   end
 
   describe ".client", :vcr do
+    let(:prefetched_client_id) {
+      find_resource("#{@casette_base_path}_clients/returns_clients.yml", "clients")[:id]
+    }
+
     it "returns a single client" do
-      client = api.client(77703)
-      expect(client.id).to eq 77703
+      client = api.client(prefetched_client_id)
+      expect(client.id).to eq prefetched_client_id
     end
   end
 
   describe ".create_client", :vcr do
     let(:attributes) do
       {
-        fullname: "John Smith",
-        emails: [{ label: "default", email: "smith@example.com" }],
-        country_code: "IT",
-        city: "Rome"
+        fullname: "Halldor Helgason",
+        emails: [{ label: "default", email: "halldor@example.com" }],
+        country_code: "IS",
+        city: "Reykjavik"
       }
     end
 
@@ -36,24 +42,28 @@ describe BookingSync::API::Client::Clients do
     it "returns newly created client" do
       VCR.use_cassette("BookingSync_API_Client_Clients/_create_client/creates_a_new_client") do
         client = api.create_client(attributes)
-        expect(client.emails).to eq [{ label: "default", email: "smith@example.com" }]
-        expect(client.fullname).to eq ("John Smith")
+        expect(client.emails).to eq [{ label: "default", email: "halldor@example.com" }]
+        expect(client.fullname).to eq ("Halldor Helgason")
       end
     end
   end
 
   describe ".edit_client", :vcr do
+    let(:created_client_id) {
+      find_resource("#{@casette_base_path}_create_client/creates_a_new_client.yml", "clients")[:id]
+    }
+
     it "updates given client by ID" do
-      api.edit_client(77703, fullname: "Gary Smith")
-      assert_requested :put, bs_url("clients/77703"),
-        body: { clients: [{ fullname: "Gary Smith" }] }.to_json
+      api.edit_client(created_client_id, fullname: "Knut Eljassen")
+      assert_requested :put, bs_url("clients/#{created_client_id}"),
+        body: { clients: [{ fullname: "Knut Eljassen" }] }.to_json
     end
 
     it "returns updated client" do
       VCR.use_cassette("BookingSync_API_Client_Clients/_edit_client/updates_given_client_by_ID") do
-        client = api.edit_client(77703, fullname: "Gary Smith")
+        client = api.edit_client(created_client_id, fullname: "Knut Eljassen")
         expect(client).to be_kind_of(BookingSync::API::Resource)
-        expect(client.fullname).to eq("Gary Smith")
+        expect(client.fullname).to eq("Knut Eljassen")
       end
     end
   end

data/spec/bookingsync/api/client/destinations_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::Destinations do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".destinations", :vcr do
     it "returns destinations" do
       expect(client.destinations).not_to be_empty
@@ -11,9 +13,13 @@ describe BookingSync::API::Client::Destinations do
   end
 
   describe ".destination", :vcr do
+    let(:prefetched_destination_id) {
+      find_resource("#{@casette_base_path}_destinations/returns_destinations.yml", "destinations")[:id]
+    }
+
     it "returns a single destination" do
-      destination = client.destination(7301)
-      expect(destination.id).to eq 7301
+      destination = client.destination(prefetched_destination_id)
+      expect(destination.id).to eq prefetched_destination_id
     end
   end
 end

data/spec/bookingsync/api/client/fees_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::Fees do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".fees", :vcr do
     it "returns fees" do
       expect(client.fees).not_to be_empty
@@ -11,9 +13,13 @@ describe BookingSync::API::Client::Fees do
   end
 
   describe ".fee", :vcr do
+    let(:prefetched_fee_id) {
+      find_resource("#{@casette_base_path}_fees/returns_fees.yml", "fees")[:id]
+    }
+
     it "returns a single fee" do
-      fee = client.fee(474)
-      expect(fee.id).to eq 474
+      fee = client.fee(prefetched_fee_id)
+      expect(fee.id).to eq prefetched_fee_id
     end
   end
 

data/spec/bookingsync/api/client/inquiries_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::Inquiries do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".inquiries", :vcr do
     it "returns inquiries" do
       expect(client.inquiries).not_to be_empty
@@ -11,16 +13,19 @@ describe BookingSync::API::Client::Inquiries do
   end
 
   describe ".inquiry", :vcr do
+    let(:prefetched_inquiry_id) {
+      find_resource("#{@casette_base_path}_inquiries/returns_inquiries.yml", "inquiries")[:id]
+    }
+
     it "returns a single inquiry" do
-      inquiry = client.inquiry(6087)
-      expect(inquiry.id).to eq 6087
+      inquiry = client.inquiry(prefetched_inquiry_id)
+      expect(inquiry.id).to eq prefetched_inquiry_id
     end
   end
 
   describe ".create_inquiry", :vcr do
     let(:attributes) do
       {
-        rental_id: 7,
         start_at:  Time.now,
         end_at:    Time.now + 86400, # one day
         firstname: "John",
@@ -28,19 +33,20 @@ describe BookingSync::API::Client::Inquiries do
         email:     "john@example.com"
       }
     end
-    let(:rental) { BookingSync::API::Resource.new(client, id: 7) }
+    let(:rental) { BookingSync::API::Resource.new(client, id: 5116) }
+
 
     it "creates a new inquiry" do
       client.create_inquiry(rental, attributes)
-      assert_requested :post, bs_url("rentals/7/inquiries"),
+      assert_requested :post, bs_url("rentals/#{rental}/inquiries"),
         body: { inquiries: [attributes] }.to_json
     end
 
     it "returns newly created inquiry" do
       VCR.use_cassette("BookingSync_API_Client_Inquiries/_create_inquiry/creates_a_new_inquiry") do
         inquiry = client.create_inquiry(rental, attributes)
-        expect(inquiry.rental_id).to eql(7)
-        expect(inquiry.firstname).to eql("John")
+        expect(inquiry.links.rental).to eq(rental.id)
+        expect(inquiry.firstname).to eq("John")
       end
     end
   end

data/spec/bookingsync/api/client/living_rooms_spec.rb

@@ -3,6 +3,8 @@ require "spec_helper"
 describe BookingSync::API::Client::LivingRooms do
   let(:client) { BookingSync::API::Client.new(test_access_token) }
 
+  before { |ex| @casette_base_path = casette_path(casette_dir, ex.metadata) }
+
   describe ".living_rooms", :vcr do
     it "returns living_rooms" do
       expect(client.living_rooms).not_to be_empty
@@ -11,19 +13,23 @@ describe BookingSync::API::Client::LivingRooms do
   end
 
   describe ".living_room", :vcr do
+    let(:prefetched_living_room_id) {
+      find_resource("#{@casette_base_path}_living_rooms/returns_living_rooms.yml", "living_rooms")[:id]
+    }
+
     it "returns a single living_room" do
-      living_room = client.living_room(6)
-      expect(living_room.id).to eq 6
+      living_room = client.living_room(prefetched_living_room_id)
+      expect(living_room.id).to eq prefetched_living_room_id
     end
   end
 
   describe ".create_living_room", :vcr do
     let(:attributes) { { sofa_beds_count: 2 } }
-    let(:rental) { BookingSync::API::Resource.new(client, id: 3) }
+    let(:rental) { BookingSync::API::Resource.new(client, id: 5116) }
 
     it "creates a new living_room" do
       client.create_living_room(rental, attributes)
-      assert_requested :post, bs_url("rentals/3/living_rooms"),
+      assert_requested :post, bs_url("rentals/#{rental}/living_rooms"),
         body: { living_rooms: [attributes] }.to_json
     end
 
@@ -37,16 +43,19 @@ describe BookingSync::API::Client::LivingRooms do
 
   describe ".edit_living_room", :vcr do
     let(:attributes) { { sofa_beds_count: 3 } }
+    let(:created_living_room_id) {
+      find_resource("#{@casette_base_path}_create_living_room/creates_a_new_living_room.yml", "living_rooms")[:id]
+    }
 
     it "updates given living_room by ID" do
-      client.edit_living_room(6, attributes)
-      assert_requested :put, bs_url("living_rooms/6"),
+      client.edit_living_room(created_living_room_id, attributes)
+      assert_requested :put, bs_url("living_rooms/#{created_living_room_id}"),
         body: { living_rooms: [attributes] }.to_json
     end
 
     it "returns updated living_room" do
       VCR.use_cassette("BookingSync_API_Client_LivingRooms/_edit_living_room/updates_given_living_room_by_ID") do
-        living_room = client.edit_living_room(6, attributes)
+        living_room = client.edit_living_room(created_living_room_id, attributes)
         expect(living_room).to be_kind_of(BookingSync::API::Resource)
         expect(living_room.sofa_beds_count).to eq(attributes[:sofa_beds_count])
       end
@@ -54,9 +63,13 @@ describe BookingSync::API::Client::LivingRooms do
   end
 
   describe ".cancel_living_room", :vcr do
+    let(:created_living_room_id) {
+      find_resource("#{@casette_base_path}_create_living_room/creates_a_new_living_room.yml", "living_rooms")[:id]
+    }
+
     it "cancels given living_room" do
-      client.cancel_living_room(6)
-      assert_requested :delete, bs_url("living_rooms/6")
+      client.cancel_living_room(created_living_room_id)
+      assert_requested :delete, bs_url("living_rooms/#{created_living_room_id}")
     end
   end
 end