amtrak 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ded5ede2e0ccd7bc11b93632610adf0c8fb16f34
-  data.tar.gz: b6aaa52969153fad2b4ff9d72dd9c8e7f45bf9fb
+  metadata.gz: 7bfefa0ae21671dd8f1c99c07c6bd049f4af4846
+  data.tar.gz: b4472693448ac1dd807623727b35f72b948b5a35
 SHA512:
-  metadata.gz: 01848e66414956a446a827e170fa0020cdd9f50c07ccb4e1c0122c2d9ad9915b32d935c68b9fd5d4959f6298a08bf4640fc07f30e33eb0d1102f78ffbe03664f
-  data.tar.gz: 159f1a415e2b49d7f19cc25176ae5cc08e0f0dbf48fcc0d05ee663baca7f2ff38d556d53706a5bc19b9fbe3b39ee7baa6d1cf2055e303e8c2773094d150c555f
+  metadata.gz: 9880dcc9313a95f4a4d795ed5c07ba527f68b5e2d585e4892ccc1a8306800332fd8a2717a37ea6b15f6a812360beea89f56f86d2fcf86f509e47b04ee34461f5
+  data.tar.gz: 1371237f1f683d90589602c4b392827a555aad340ea6fa3262c1cb5f64951af7590d3392ec448adb0475e78f0fb2db8c51995b9f739640f504ca298d605cad16

data/lib/amtrak/train_fetcher/main_page.rb

@@ -13,8 +13,16 @@ module Amtrak
         @date = date
       end
 
-      def page
-        @page ||= Excon.post(
+      def request
+        retries ||= 3
+        _request
+      rescue SocketError, TimeoutError
+        retries -= 1
+        retry unless retries.zero?
+      end
+
+      def _request
+        @request ||= Excon.post(
           'http://tickets.amtrak.com/itd/amtrak',
           headers: headers,
           body: URI.encode_www_form(body),
@@ -53,7 +61,7 @@ module Amtrak
       end
 
       def session_id
-        page.headers['Set-Cookie'].match(/JSESSIONID=([^;]*)/)[1]
+        request.headers['Set-Cookie'].match(/JSESSIONID=([^;]*)/)[1]
       end
 
       def total_pages
@@ -61,7 +69,7 @@ module Amtrak
       end
 
       def extract_listing_length
-        page.body.match(/var availabilityLength = '(\d+)';/)[1]
+        request.body.match(/var availabilityLength = '(\d+)';/)[1]
       end
     end
   end

data/lib/amtrak/train_fetcher/train_page.rb

@@ -20,6 +20,14 @@ module Amtrak
       end
 
       def request
+        retries ||= 3
+        _request
+      rescue SocketError, TimeoutError
+        retries -= 1
+        retry unless retries.zero?
+      end
+
+      def _request
         @request ||= Excon.get(
           'https://tickets.amtrak.com/itd/amtrak/TrainStatusRequest',
           headers: headers,

data/lib/amtrak/version.rb

@@ -1,6 +1,6 @@
 module Amtrak
   # This module holds the Amtrak version
   module Version
-    VERSION = '1.0.1'
+    VERSION = '1.0.2'
   end
 end

data/spec/amtrak/train_fetcher/main_page_spec.rb

@@ -57,18 +57,18 @@ describe Amtrak::TrainFetcher::MainPage do
     end
   end
 
-  describe '#page' do
+  describe '#request' do
     context 'when Excon raises' do
       subject { described_class.new('', '') }
 
       it 'returns a TrainFetcher::Error' do
         expect(Excon).to receive(:post) { fail Excon::Errors::ClientError, '' }
-        expect { subject.page }.to raise_error(Amtrak::TrainFetcher::Error)
+        expect { subject.request }.to raise_error(Amtrak::TrainFetcher::Error)
       end
 
       it 'returns a TrainFetcher::Error' do
         expect(Excon).to receive(:post) { fail Excon::Errors::ServerError, '' }
-        expect { subject.page }.to raise_error(Amtrak::TrainFetcher::Error)
+        expect { subject.request }.to raise_error(Amtrak::TrainFetcher::Error)
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: amtrak
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Todd Lunter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-30 00:00:00.000000000 Z
+date: 2015-04-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: excon