Merge pull request #54 from sue445/feature/integration_retry

Add retryable to all API call

spec/integration_spec.rb

@@ -8,7 +8,9 @@
   end
 
   it "successful real http connection" do # rubocop:disable RSpec/ExampleLength
-    category_and_boards = Itest5ch::Board.all
+    category_and_boards = Retryable.with_context(:default) do
+      Itest5ch::Board.all
+    end
 
     expect(category_and_boards).to be_an_instance_of(Hash)
 
@@ -21,7 +23,13 @@
 
     board = boards.sample
 
-    threads = board.threads
+    thread_exception_cb = proc do |_exception|
+      puts "Error: #{board.json_url}"
+    end
+
+    threads = Retryable.with_context(:default, exception_cb: thread_exception_cb) do
+      board.threads
+    end
 
     aggregate_failures do
       expect(threads).not_to be_empty
@@ -30,14 +38,13 @@
 
     thread = threads.sample
 
-    exception_cb = proc do |_exception|
+    comment_exception_cb = proc do |_exception|
       puts "Error: #{thread.json_url}"
     end
 
-    comments =
-      Retryable.retryable(tries: 5, on: JSON::ParserError, exception_cb: exception_cb) do
-        thread.comments
-      end
+    comments = Retryable.with_context(:default, exception_cb: comment_exception_cb) do
+      thread.comments
+    end
 
     aggregate_failures do
       expect(comments).not_to be_empty

spec/spec_helper.rb

@@ -18,6 +18,7 @@
 require "active_support/time"
 require "rspec-parameterized"
 require "retryable"
+require "open-uri"
 
 Dir["#{__dir__}/support/**/*.rb"].each {|f| require f }
 
@@ -137,3 +138,18 @@ def spec_dir
     c.syntax = :expect
   end
 end
+
+Retryable.configure do |config|
+  retryable_errors = [
+    JSON::ParserError,
+    Net::OpenTimeout,
+    Net::ReadTimeout,
+    OpenURI::HTTPError,
+  ]
+
+  config.contexts[:default] = {
+    on: retryable_errors,
+    sleep: 1,
+    tries: 5,
+  }
+end