Merge cd3c50b into c4750c9

.travis.yml

@@ -4,6 +4,7 @@ rvm:
   - 2.3
   - 2.4
   - 2.5
+before_install: gem install bundler -v "~>1.0"
 script: bundle && bundle exec rake spec
 gemfile:
   - gemfiles/rails4.gemfile

CHANGELOG.md

@@ -7,6 +7,7 @@ This changelog adheres to [Keep a CHANGELOG](http://keepachangelog.com/).
 ### Added
 - [DC-1418] Port NRMA portal availability cache back to EcomEngine
 - [TT-4850] Added client templates
+- [DC-1692] Fix error that can be thrown when checking if a product configuration can be priced
 
 ### Changed
 - [DC-1767] include long/lat changes to Stop

lib/quick_travel/product_configuration.rb

@@ -31,7 +31,7 @@ def available?
     end
 
     def priced?
-      @product.pricing_details.present?
+      @product.try(:pricing_details).present?
     end
 
     def price

quicktravel_client.gemspec

@@ -23,7 +23,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'money', '>= 3.0', '< 6.0' # 6.0 starts to deprecate/split
   spec.add_dependency 'will_paginate'
 
-  spec.add_development_dependency 'bundler', '~> 1.7'
+  spec.add_development_dependency 'bundler', '>= 1.3.0', '< 2.0'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec'
   spec.add_development_dependency 'rspec-its'

spec/product_configuration_spec.rb

@@ -127,6 +127,17 @@
       let(:pricing_details) { nil }
       it { is_expected.not_to be_priced }
     end
+
+    context 'when pricing details are omitted' do
+      let(:product) do
+        double(
+          extras: extras,
+          available?: available
+        )
+      end
+
+      it { is_expected.not_to be_priced }
+    end
   end
 
   context '#price' do