quicktravel_client 3.9.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d3388b310dd9bf0f503277b7fea3f329e692600cf9db6cf53ae8c8882246f0f
-  data.tar.gz: cb262cc43f77f388ec5250840338a1e2136ed142b5adbe4ffd798b07e62953e4
+  metadata.gz: 5007469a12b06055c4cd7b9ae331a99ffc041952348ccad994ab11de51069e79
+  data.tar.gz: e57da3eb9ef8ff2f49283a9de910d5ffbb4c424f965f96fb8175d6ea452795e1
 SHA512:
-  metadata.gz: 64080794df1529bc263e7e641223d8e8c741289c426d97748c6afb62026eb71141d502462e844c33c4028526825823b895862c6ef0b80a99343304f31c2ad8a0
-  data.tar.gz: a9ba238eed20b7276eb7a9d4253843d0374a197c07a149c2b3b98c9da1477993ef0eab3f8d7678a10bb44f8d87ef0b88f40688523fa58441756317b11aeafb43
+  metadata.gz: 24d5ba5f79aedf800a3fbd40c793a13dab2bfb48f91059460293aa757e5aea062f850b290e2b3b72e83c2269de5a675e9d5dfee761ef7555ba47aa3bc24f8928
+  data.tar.gz: ecb835442572758fe0b02d953bf978f2af920396396e7d9a6937917d5636fd8663e3ce0672fbcf39fed76652acdbb7dd4582f0c0bd19b3827123cb861df77081

data/.ruby-version

@@ -1 +1 @@
-2.5.0
+2.7.1

data/.travis.yml

@@ -1,15 +1,14 @@
 language: ruby
 rvm:
-  - 2.2
-  - 2.3
-  - 2.4
   - 2.5
+  - 2.6
+  - 2.7
 before_install:
-  - gem install bundler --version "1.17.3"
+  - gem install bundler
 script: bundle && bundle exec rake spec
 gemfile:
-  - gemfiles/rails4.gemfile
   - gemfiles/rails5.gemfile
+  - gemfiles/rails6.gemfile
 notifications:
   email:
     - support@travellink.com.au

data/CHANGELOG.md

@@ -3,6 +3,9 @@ All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 This changelog adheres to [Keep a CHANGELOG](http://keepachangelog.com/).
 
+## [4.0.0]
+- [TT-7385] Update Money dependency, test against Ruby 2.7 / Rails 6
+
 ## [3.9.0]
 ### Added
 - [DC-3115] Add customer comments method in booking

data/gemfiles/rails6.gemfile

@@ -0,0 +1,8 @@
+source 'https://rubygems.org'
+gemspec :path => '../'
+
+group :development, :test do
+  gem 'activesupport', '~> 6.0'
+  gem 'activerecord', '~> 6.0'
+  gem 'actionpack', '~> 6.0'
+end

data/lib/quick_travel.rb

@@ -2,6 +2,7 @@
 #
 module QuickTravel
   require 'active_support' # for .try, etc.
+  require 'money_extensions'
 
   require 'quick_travel/cache'
   require 'quick_travel/config'

data/lib/quick_travel/booking.rb

@@ -13,7 +13,7 @@ module QuickTravel
     end
 
     def self.find_by_reference(reference)
-      find_all!("#{api_base}/reference/#{URI.escape(reference)}.json").first
+      find_all!("#{api_base}/reference/#{URI.encode_www_form_component(reference)}.json").first
     end
 
     def documents(regenerate = false)

data/lib/quick_travel/cache.rb

@@ -11,13 +11,20 @@ module QuickTravel
       cache_options ||= {}
       key = "#{@@namespace}_#{key}" unless cache_options[:disable_namespacing]
       cached_value = cache_store.read(key)
-      return cached_value unless cached_value.nil?
+      return cached_value unless cache_empty?(cached_value)
       return nil unless block_given?
       cache_options ||= {}
       cache_options[:expires_in] = 1.day unless cache_options.key?(:expires_in)
       yield.tap { |value| cache_store.write(key, value, cache_options) }
     end
 
+    def self.cache_empty?(cached_value)
+      if cached_value.respond_to?(:body)
+        return cached_value.body.nil? || cached_value.body.empty?
+      end
+      cached_value.nil?
+    end
+
     def self.delete(key, namespace = true)
       key = "#{@@namespace}_#{key}" if namespace
       cache_store.delete(key)

data/lib/quick_travel/passenger_type.rb

@@ -13,7 +13,7 @@ module QuickTravel
     end
 
     def self.pluralize(count, singular, plural = nil)
-      "#{count || 0} " + ((count == 1 || count =~ /^1(\.0+)?$/) ? singular : (plural || singular.pluralize))
+      "#{count || 0} " + ((count == 1) ? singular : (plural || singular.pluralize))
     end
   end
 end

data/lib/quick_travel/version.rb

@@ -1,3 +1,3 @@
 module QuickTravel
-  VERSION = '3.9.0'
+  VERSION = '4.0.0'
 end

data/quicktravel_client.gemspec

@@ -16,14 +16,14 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'httparty', '~> 0.17'
-  spec.add_dependency 'json'
-  spec.add_dependency 'activesupport', '>= 2.0.0'
+  spec.add_dependency 'httparty'
+  spec.add_dependency 'activesupport', '>= 5.0.0'
   spec.add_dependency 'facets'
-  spec.add_dependency 'money', '>= 3.0', '< 6.0' # 6.0 starts to deprecate/split
+  spec.add_dependency 'money', '>= 6.0'
+  spec.add_dependency 'money_extensions', '>= 1.0'
   spec.add_dependency 'will_paginate'
 
-  spec.add_development_dependency 'bundler', '~> 1.7'
+  spec.add_development_dependency 'bundler', '~> 2'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rspec-its'

data/spec/discounts_spec.rb

@@ -26,9 +26,9 @@ describe 'Booking price_changes' do
 
     specify { expect(price_change.target.type).to eq 'Booking' }
     specify { expect(price_change.target.id).to eq booking.id }
-    specify { expect(price_change.original_price).to eq 640.00 }
-    specify { expect(price_change.changed_price).to eq 380.00 }
-    specify { expect(price_change.price_change).to eq(-260.00) }
+    specify { expect(price_change.original_price).to eq 640.00.to_money }
+    specify { expect(price_change.changed_price).to eq 380.00.to_money }
+    specify { expect(price_change.price_change).to eq(-260.00.to_money) }
     specify do
       expect(price_change.reservation_price_changes.count).to(
         eq booking.reservations.count)
@@ -43,9 +43,9 @@ describe 'Booking price_changes' do
 
       specify { expect(price_change.target.type).to eq 'Reservation' }
       specify { expect(price_change.target.id).to eq reservation.id }
-      specify { expect(price_change.original_price).to eq 400.00 }
-      specify { expect(price_change.changed_price).to eq 200.00 }
-      specify { expect(price_change.price_change).to eq(-200) }
+      specify { expect(price_change.original_price).to eq 400.00.to_money }
+      specify { expect(price_change.changed_price).to eq 200.00.to_money }
+      specify { expect(price_change.price_change).to eq(-200.to_money) }
     end
 
     context 'the total price_change applied on the top level reservation' do
@@ -57,9 +57,9 @@ describe 'Booking price_changes' do
 
       specify { expect(price_change.target.type).to eq 'Reservation' }
       specify { expect(price_change.target.id).to eq reservation.id }
-      specify { expect(price_change.original_price).to eq 640.00 }
-      specify { expect(price_change.changed_price).to eq 380.00 }
-      specify { expect(price_change.price_change).to eq(-260.00) }
+      specify { expect(price_change.original_price).to eq 640.00.to_money }
+      specify { expect(price_change.changed_price).to eq 380.00.to_money }
+      specify { expect(price_change.price_change).to eq(-260.00.to_money) }
     end
 
     context 'the price_change applied on the first extra pick' do
@@ -71,9 +71,9 @@ describe 'Booking price_changes' do
 
       specify { expect(price_change.target.type).to eq 'Reservation' }
       specify { expect(price_change.target.id).to eq extra_pick_without_price_change.id }
-      specify { expect(price_change.original_price).to eq 120 }
-      specify { expect(price_change.changed_price).to eq 120 }
-      specify { expect(price_change.price_change).to eq 0 }
+      specify { expect(price_change.original_price).to eq 120.to_money }
+      specify { expect(price_change.changed_price).to eq 120.to_money }
+      specify { expect(price_change.price_change).to eq 0.to_money }
     end
 
     context 'the total price_change applied on the first extra pick' do
@@ -85,9 +85,9 @@ describe 'Booking price_changes' do
 
       specify { expect(price_change.target.type).to eq 'Reservation' }
       specify { expect(price_change.target.id).to eq extra_pick_without_price_change.id }
-      specify { expect(price_change.original_price).to eq 120 }
-      specify { expect(price_change.changed_price).to eq 120 }
-      specify { expect(price_change.price_change).to eq 0 }
+      specify { expect(price_change.original_price).to eq 120.to_money }
+      specify { expect(price_change.changed_price).to eq 120.to_money }
+      specify { expect(price_change.price_change).to eq 0.to_money }
     end
 
     context 'the price_change applied on second extra pick' do
@@ -99,9 +99,9 @@ describe 'Booking price_changes' do
 
       specify { expect(price_change.target.type).to eq 'Reservation' }
       specify { expect(price_change.target.id).to eq extra_pick_with_price_change.id }
-      specify { expect(price_change.original_price).to eq 120.00 }
-      specify { expect(price_change.changed_price).to eq 60.00 }
-      specify { expect(price_change.price_change).to eq(-60.00) }
+      specify { expect(price_change.original_price).to eq 120.00.to_money }
+      specify { expect(price_change.changed_price).to eq 60.00.to_money }
+      specify { expect(price_change.price_change).to eq(-60.00.to_money) }
     end
 
     context 'the total price_change applied on second extra pick' do
@@ -113,9 +113,9 @@ describe 'Booking price_changes' do
 
       specify { expect(price_change.target.type).to eq 'Reservation' }
       specify { expect(price_change.target.id).to eq extra_pick_with_price_change.id }
-      specify { expect(price_change.original_price).to eq 120.00 }
-      specify { expect(price_change.changed_price).to eq 60.00 }
-      specify { expect(price_change.price_change).to eq(-60.00) }
+      specify { expect(price_change.original_price).to eq 120.00.to_money }
+      specify { expect(price_change.changed_price).to eq 60.00.to_money }
+      specify { expect(price_change.price_change).to eq(-60.00.to_money) }
     end
   end
 end

data/spec/spec_helper.rb

@@ -27,6 +27,11 @@ end
 require 'quick_travel/connection_error'
 require 'quick_travel/cache'
 
+Money.default_currency = Money::Currency.new("AUD")
+Money.locale_backend = nil
+Money.default_formatting_rules = { thousands_separator: "," }
+Money.rounding_mode = BigDecimal::ROUND_HALF_UP
+
 class HashCache
   def initialize
     @cache = {}

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: quicktravel_client
 version: !ruby/object:Gem::Version
-  version: 3.9.0
+  version: 4.0.0
 platform: ruby
 authors:
 - Michael Noack
@@ -10,24 +10,10 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-26 00:00:00.000000000 Z
+date: 2020-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.17'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.17'
-- !ruby/object:Gem::Dependency
-  name: json
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -46,14 +32,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 5.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 5.0.0
 - !ruby/object:Gem::Dependency
   name: facets
   requirement: !ruby/object:Gem::Requirement
@@ -73,9 +59,6 @@ dependencies:
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
-      - !ruby/object:Gem::Version
-        version: '3.0'
-    - - "<"
       - !ruby/object:Gem::Version
         version: '6.0'
   type: :runtime
@@ -83,11 +66,22 @@ dependencies:
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
-      - !ruby/object:Gem::Version
-        version: '3.0'
-    - - "<"
       - !ruby/object:Gem::Version
         version: '6.0'
+- !ruby/object:Gem::Dependency
+  name: money_extensions
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: will_paginate
   requirement: !ruby/object:Gem::Requirement
@@ -108,14 +102,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -288,8 +282,8 @@ files:
 - Rakefile
 - examples/example_config.rb
 - examples/login_example.rb
-- gemfiles/rails4.gemfile
 - gemfiles/rails5.gemfile
+- gemfiles/rails6.gemfile
 - lib/extensions.rb
 - lib/quick_travel.rb
 - lib/quick_travel/accommodation.rb
@@ -463,8 +457,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Booking process using QuickTravel API

data/gemfiles/rails4.gemfile

@@ -1,9 +0,0 @@
-source 'https://rubygems.org'
-gemspec :path => '../'
-
-group :development, :test do
-  gem 'activesupport', '~> 4.2'
-  gem 'activerecord', '~> 4.2'
-  gem 'actionpack', '~> 4.2'
-end
-