twitter_retry 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0a52f9cf9e286f7d0fc9a4ae895116c0418efacb
-  data.tar.gz: 038b5a727684f0b990c49043320382e173cab72b
+  metadata.gz: ff0eab75ac0dbd4ea33d5f2eb2840ef01e617b6f
+  data.tar.gz: 2c3dab70979cb3fecb5a4d81360b925ccf532efd
 SHA512:
-  metadata.gz: 2506d6e93b4fd3ec4c57e3e7d1e76916d0b951022d260607ba2689056783958ebe846116e6f03efd30023fb982faa1613e61784f863ebdcbcbd447c8b0fe5fe9
-  data.tar.gz: 4c8b0f4b4ab43640879aca7e51578d6bb2a37901cecb226b0524ceb9e43f5ebf473c3bdf67e4d68f18eda786ef9c95f0ef29f5346273455be70c6634eea1c283
+  metadata.gz: 53971d3ac57744fbc9ccaadae99a6ed109ed6ce1ce2317a317cbc734c8855003520af22bee0010a4f8e6203e948daecef4c311a7623a80bdd94652a42f54ae92
+  data.tar.gz: 3ee9a129f5c78ed08b0b6dca148ffb6dea9b7093c2eb34dd0840fd60a523aea50bc56b12ea0baed74cc0c093703a5efc4bf734ce81ac234abf572088f43d50bc

data/.travis.yml

@@ -2,6 +2,8 @@ language: ruby
 rvm:
   - 2.1
   - 2.2
+  - 2.3.3
+  - 2.4.0
   - ruby-head
 before_install: gem install bundler -v 1.10.5
 before_script:

data/CHANGELOG.md

@@ -0,0 +1,13 @@
+## Unreleased
+[full changelog](https://github.com/sue445/twitter_retry/compare/v0.2.0...master)
+
+## v0.2.0
+[full changelog](https://github.com/sue445/twitter_retry/compare/v0.1.0...v0.2.0)
+
+* Support regexp pattern on `retryable_errors?` and `ignorable_errors?`
+  * https://github.com/sue445/twitter_retry/pull/9
+* Support: check only error class
+  * https://github.com/sue445/twitter_retry/pull/11
+
+## v0.1.0
+* first release

data/Gemfile

@@ -2,3 +2,8 @@ source "https://rubygems.org"
 
 # Specify your gem's dependencies in twitter_retry.gemspec
 gemspec
+
+if Gem::Version.create(RUBY_VERSION) < Gem::Version.create("2.2.2")
+  # NOTE: activesupport 5.x supports only ruby 2.2.2+
+  gem "activesupport", "< 5.0.0"
+end

data/README.md

@@ -2,11 +2,15 @@
 
 Twitter api awesome handling with retry
 
+[![Gem Version](https://badge.fury.io/rb/twitter_retry.svg)](http://badge.fury.io/rb/twitter_retry)
 [![Build Status](https://travis-ci.org/sue445/twitter_retry.svg?branch=master)](https://travis-ci.org/sue445/twitter_retry)
 [![Code Climate](https://codeclimate.com/github/sue445/twitter_retry/badges/gpa.svg)](https://codeclimate.com/github/sue445/twitter_retry)
 [![Coverage Status](https://coveralls.io/repos/sue445/twitter_retry/badge.svg?branch=master&service=github)](https://coveralls.io/github/sue445/twitter_retry?branch=master)
 [![Dependency Status](https://gemnasium.com/sue445/twitter_retry.svg)](https://gemnasium.com/sue445/twitter_retry)
 
+## Requirements
+* Ruby 2.1+
+
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -52,6 +56,12 @@ TwitterRetry.configure do |config|
   config.max_retry_count  = 3
   config.retryable_errors << [Twitter::Error, "some error message"]
   config.ignorable_errors << [Twitter::Error, "some error message"]
+
+  # Check whether error message matched with regexp
+  config.retryable_errors << [Twitter::Error::ServiceUnavailable, /something/]
+
+  # Check whether only error class matches (error message can be anything)
+  config.ignorable_errors << [Twitter::Error::ServiceUnavailable]
 end
 ```
 

data/lib/twitter_retry/retryable.rb

@@ -30,17 +30,13 @@ module TwitterRetry
     # whether retryable error
     # @param error [Exception]
     def retryable?(error)
-      TwitterRetry.config.retryable_errors.any? do |error_class, message|
-        error.is_a?(error_class) && error.message.include?(message)
-      end
+      match_any_error?(error, TwitterRetry.config.retryable_errors)
     end
 
     # whether ignorable error
     # @param error [Exception]
     def ignorable?(error)
-      TwitterRetry.config.ignorable_errors.any? do |error_class, message|
-        error.is_a?(error_class) && error.message.include?(message)
-      end
+      match_any_error?(error, TwitterRetry.config.ignorable_errors)
     end
 
     # whether suspended user error
@@ -48,5 +44,26 @@ module TwitterRetry
       error.is_a?(Twitter::Error::Forbidden) &&
         error.message.include?("Your account is suspended and is not permitted to access this feature.")
     end
+
+    private
+
+    def match_any_error?(source_error, check_errors)
+      check_errors.any? do |error_class, message|
+        match_error?(source_error, error_class, message)
+      end
+    end
+
+    def match_error?(source_error, error_class, message)
+      return false unless source_error.is_a?(error_class)
+
+      # Check only error class
+      return true unless message
+
+      if message.is_a?(Regexp)
+        source_error.message =~ message
+      else
+        source_error.message.include?(message)
+      end
+    end
   end
 end

data/lib/twitter_retry/version.rb

@@ -1,3 +1,3 @@
 module TwitterRetry
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/twitter_retry.gemspec

@@ -27,6 +27,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "rspec-parameterized"
-  spec.add_development_dependency "rubocop", "0.31.0"
   spec.add_development_dependency "yard"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: twitter_retry
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - sue445
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-07-19 00:00:00.000000000 Z
+date: 2017-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -108,20 +108,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.31.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.31.0
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -145,12 +131,10 @@ extra_rdoc_files: []
 files:
 - ".coveralls.yml"
 - ".gitignore"
-- ".hound.yml"
 - ".rspec"
-- ".rubocop.yml"
-- ".rubocop_standard.yml"
 - ".travis.yml"
 - ".yardopts"
+- CHANGELOG.md
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -182,9 +166,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Twitter api awesome handling with retry
 test_files: []
-has_rdoc: 

data/.hound.yml

@@ -1,3 +0,0 @@
-ruby:
-  enabled: true
-  config_file: .rubocop.yml

data/.rubocop.yml

@@ -1 +0,0 @@
-inherit_from: .rubocop_standard.yml

data/.rubocop_standard.yml

@@ -1,196 +0,0 @@
-# via. https://gist.github.com/onk/38bfbd78899d892e0e83
-
-# target_version:
-#   rubocop v0.31.0
-
-# 自動生成されるものはチェック対象から除外する
-AllCops:
-  Exclude:
-    - "vendor/**/*" # rubocop config/default.yml
-    - "db/schema.rb"
-  DisplayCopNames: true
-
-##################### Style ##################################
-
-# redirect_to xxx and return のイディオムを維持したい
-Style/AndOr:
-  EnforcedStyle: conditionals
-
-# 日本語のコメントを許可する
-Style/AsciiComments:
-  Enabled: false
-
-# option 等、明示的にハッシュにした方が分かりやすい場合もある
-Style/BracesAroundHashParameters:
-  Enabled: false
-
-# Style/CollectionMethods 自体は無効になっているのだが、
-#   https://github.com/bbatsov/rubocop/issues/1084
-#   https://github.com/bbatsov/rubocop/issues/1334
-# Performance/Detect がこの設定値を見るので PreferredMethods だけ変更しておく。
-#
-# デフォルト値から変えたのは
-#   find -> detect
-#     ActiveRecord の find と間違えやすいため
-#   reduce -> inject
-#     detect, reject, select と並べたときに韻を踏んでいるため。
-#     collect -> map を維持しているのは文字数が圧倒的に少ないため。
-Style/CollectionMethods:
-  PreferredMethods:
-    detect: "detect"
-    find: "detect"
-    inject: "inject"
-    reduce: "inject"
-
-# Hash#has_key? は許可したい
-Style/DeprecatedHashMethods:
-  Enabled: false
-
-# ドキュメントの無い public class を許可する
-Style/Documentation:
-  Enabled: false
-
-# !! のイディオムは積極的に使う
-Style/DoubleNegation:
-  Enabled: false
-
-# メソッドチェーンの改行は末尾に . を入れる
-# REPL に貼り付けた際の暴発を防ぐため
-Style/DotPosition:
-  EnforcedStyle: trailing
-
-# 明示的に else で nil を返すのは分かりやすいので許可する
-Style/EmptyElse:
-  EnforcedStyle: empty
-
-# いずれかに揃えるのならば `sprintf` や `format` より String#% が好きです
-Style/FormatString:
-  EnforcedStyle: percent
-
-# if 文の中に 3 行程度のブロックを書くぐらいは許容した方が現実的
-Style/GuardClause:
-  MinBodyLength: 5
-
-# 同じ hash 内で記法を混ぜない
-# rake タスクの順序の hash は rocket を許可する
-Style/HashSyntax:
-  EnforcedStyle: ruby19_no_mixed_keys
-  Exclude:
-    - "**/*.rake"
-    - "Rakefile"
-
-# 条件式の方を意識させたい場合には後置の if/unless を使わない方が分かりやすい
-Style/IfUnlessModifier:
-  Enabled: false
-
-# ({ と hash を開始した場合に ( の位置にインデントさせる
-# そもそも {} が必要ない可能性が高いが Style/BracesAroundHashParameters はチェックしないことにしたので
-Style/IndentHash:
-  EnforcedStyle: consistent
-
-# private/protected は一段深くインデントする
-Style/IndentationConsistency:
-  EnforcedStyle: rails
-
-# scope 等は複数行でも lambda ではなく ->{} で揃えた方が見た目が綺麗
-Style/Lambda:
-  Enabled: false
-
-# 1_000_000 と区切り文字が 2 個以上必要になる場合のみ _ 区切りを必須にする
-Style/NumericLiterals:
-  MinDigits: 7
-
-# has_ から始まるメソッドは許可する
-Style/PredicateName:
-  NamePrefixBlacklist:
-    - "is_"
-    - "have_"
-  NamePrefix:
-    - "is_"
-    - "have_"
-
-# 特に model 内において、ローカル変数とメソッド呼び出しの区別をつけた方が分かりやすい場合が多い
-Style/RedundantSelf:
-  Enabled: false
-
-# 受け取り側で multiple assignment しろというのを明示
-Style/RedundantReturn:
-  AllowMultipleReturnValues: true
-
-# fail と使い分ける必要ナシ
-Style/SignalException:
-  EnforcedStyle: only_raise
-
-# `||` も align に使うことがあるので追加する
-Style/SpaceAroundOperators:
-  MultiSpaceAllowedForOperators:
-    - "="
-    - "=>"
-    - "||"
-
-# * 式展開したい場合に書き換えるのが面倒
-# * 文章ではダブルクォートよりもシングルクォートの方が頻出する
-# ことから EnforcedStyle: double_quotes 推奨
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
-
-# auto-correct 時に Style/StringLiterals とカニバって無限ループになる (v0.28.0)
-Style/StringLiteralsInInterpolation:
-  Enabled: false
-
-# いくらなんでも inject { |a, e| } は短すぎるので分かりやすい名前をつけたい
-Style/SingleLineBlockParams:
-  Enabled: false
-
-# * migrate
-# * jbuilder
-# * model の association
-# * controller の callback
-# 辺りの桁揃えで引っかかるので全体的にチェックしない
-Style/SingleSpaceBeforeFirstArg:
-  Enabled: false
-
-# 複数行の場合はケツカンマを入れる
-Style/TrailingComma:
-  EnforcedStyleForMultiline: comma
-
-##################### Lint ##################################
-
-# * 同名のメソッドがある場合にローカル変数に `_` を付ける
-# * 一時変数として `_` を付ける
-# というテクニックは頻出する
-Lint/UnderscorePrefixedVariableName:
-  Enabled: false
-
-# 子クラスで実装させるつもりのメソッドで引っかかるので
-Lint/UnusedMethodArgument:
-  Enabled: false
-
-##################### Metrics ##################################
-
-# 30 まではギリギリ許せる範囲だった
-Metrics/AbcSize:
-  Max: 30
-
-# 6 は強すぎるので緩める
-Metrics/CyclomaticComplexity:
-  Max: 10
-
-# * 警告 120文字
-# * 禁止 160文字
-# のイメージ
-Metrics/LineLength:
-  Max: 160
-  Exclude:
-    - "db/migrate/*.rb"
-
-# 20 行超えるのは migration ファイル以外滅多に無い
-Metrics/MethodLength:
-  Max: 20
-  Exclude:
-    - "db/migrate/*.rb"
-
-# 分岐の数。ガード句を多用しているとデフォルト 7 だと厳しい
-Metrics/PerceivedComplexity:
-  Max: 8
-