little_monster 0.1.7 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a5fe4a4fb228c4c34cf4084d3e679ea3338423f5
-  data.tar.gz: 33d5b1ab1f908282da6d82a34775393e7db9c357
+  metadata.gz: 11689957621c432567c76283d4d534c4489ca928
+  data.tar.gz: ad919ed6292c894d52d473c04103061c88625451
 SHA512:
-  metadata.gz: 84ceda610dfd1ee6e0d6edff7ad5a3de3a81f0e6b14771cadf7c5f66d56125162988ad7bc7ccf3e049eea48056cc25083155281f8fa1c8c8d215bd836f77f19e
-  data.tar.gz: bcb0ce751eb61f6f03219d7774e27cd5a1450b26788ba9485e6bdafb2abf9f4e7fc44f936dc3b26cc2a12a4e54ff6ac56d8ffefe46e8a3f99cf23e40dd342b87
+  metadata.gz: 1133c898fe47757c8f1828947d2c37a447594f57626db250a6eb6ee3b06a2cb4086fbfe32df9a893c7bd257e38c8ad11b84b5f2d29130653b3a82a318cac2a03
+  data.tar.gz: 31a3cda7700de906ff6a52aade9ffb2c13fcbdbb201dbe645483c240ac8bf6cb17ee8acaedec861aadce12f9d475fc893eac52e76f6bbcc85a7efcffaa52e807

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    little_monster (0.1.7)
+    little_monster (0.1.9)
       activesupport
       multi_json
       thor

data/README.md

@@ -3,3 +3,90 @@
 [![Code Climate](https://codeclimate.com/github/mercadolibre/fury-little_monster-gem/badges/gpa.svg)](https://codeclimate.com/github/mercadolibre/fury-little_monster-gem)
 # fury-little_monster
 
+##RSPEC Matchers
+
+###Installation
+
+in your spec_helper add the following line
+
+```ruby
+require 'little_monster/rspec'
+```
+
+###helpers
+
+### generate_job
+it takes a job and a hash of parameters and returns a fully configured job instance
+
+#### generate_job with data
+```ruby
+generate_job :my_job, data: { a: :b }
+```
+
+#### generate_job with task mocked to fail
+```ruby
+generate_job :my_job, data: { a: :b }, fails: { task: :my_task, error: MyError.new }
+```
+
+#### generate_job with multiples tasks mocked to fail
+```ruby
+generate_job :my_job, data: { a: :b }, fails: [{ task: :my_task, error: MyError.new }, { task: :my_other_task, error: MyError.new }]
+```
+
+### run_job
+given a generated job, it returns a JobResult object to make expectation about the run
+
+### generate_task
+it takes a task class or symbol and returns a fully configured task instance
+
+#### generate_task with data
+```ruby
+generate_task MyJob::MyTask, data: { a: :b }
+```
+
+###matchers
+
+###have_run
+
+given a JobResult object expects the job to run that list of tasks
+```ruby
+expect(run_job(:my_job)).to have_run(:my_task, :my_other_task)
+```
+
+###have_run_task
+
+given a JobResult object expects the job to run the a given task with a certain data
+```ruby
+expect(run_job(:my_job)).to have_run_task(:my_task).with_data(a: :b)
+```
+
+###have_ended_with_status
+
+given a JobResult object expects the job have a given status after the run
+```ruby
+expect(run_job(:my_job)).to have_ended_with_status(:success)
+```
+
+###have_data
+
+given checks a job or JobResult instance data
+```ruby
+expect(run_job(:my_job)).to have_ended_with_status(:success)
+```
+```ruby
+expect(generate_job(:my_job)).to have_ended_with_status(:success)
+```
+
+###have_retries
+
+given job instance, a class or a class symbol it expects the retries for that class
+```ruby
+expect(:my_job).to have_retries(3)
+```
+
+###have_retries
+
+given job instance, a class or a class symbol it expects the callback retries for that class
+```ruby
+expect(my_job).to have_callback_retries(10)
+```

data/lib/little_monster/core/api.rb

@@ -1,5 +1,6 @@
 require 'typhoeus'
 require 'multi_json'
+require 'securerandom'
 
 module LittleMonster::Core
   class API
@@ -29,20 +30,27 @@ module LittleMonster::Core
       def request(method, path, params = {}, retries: LittleMonster.default_request_retries,
                   retry_wait: LittleMonster.default_request_retry_wait,
                   critical: false)
+
+        request_id = SecureRandom.uuid
         ret = 0
         res = nil
         url = [LittleMonster.api_url.chomp('/'), path.sub(/\//, '')].join '/'
 
         params[:body] = MultiJson.dump params.fetch(:body, {}) unless params[:body].is_a? String
+
         params[:headers] ||= {}
         params[:headers]['Content-Type'] = 'application/json' unless params[:headers]['Content-Type']
+        params[:headers]['X-Request-ID'] = request_id
+
         params[:timeout] = LittleMonster.request_timeout
 
         begin
           res = Typhoeus.public_send method, url, params
           if res.code >= 500 || res.code.zero?
-            raise FuryHttpApiError, "request to #{res.effective_url} failed with status #{res.code} retry #{ret}"
+            raise FuryHttpApiError, "[type:request_failed][request_id:#{request_id}] request to #{res.effective_url} failed with status #{res.code} retry #{ret}"
           end
+
+          logger.info "[type:request_log][request_id:#{request_id}] request made to #{url} with [status:#{res.code}]"
         rescue StandardError => e
           logger.error e.message
           if ret < retries
@@ -51,11 +59,11 @@ module LittleMonster::Core
             retry
           end
 
-          logger.error "[type:request_max_retries_reached][url:#{url}][retries:#{ret}] request has reached max retries"
+          logger.error "[type:request_max_retries_reached][request_id:#{request_id}][url:#{url}][retries:#{ret}] request has reached max retries"
 
           if critical
-            logger.error "[type:critical_request_failed][url:#{url}][retries:#{ret}] request has reached max retries"
-            raise APIUnreachableError, "critical request to #{url} has fail, check little monster api"
+            logger.error "[type:critical_request_failed][request_id:#{request_id}][url:#{url}][retries:#{ret}] request has reached max retries"
+            raise APIUnreachableError, "[request_id:#{request_id}] critical request to #{url} has fail, check little monster api"
           end
         end
 

data/lib/little_monster/core/counters.rb

@@ -13,7 +13,7 @@ module LittleMonster::Core::Counters
       logger.error "Could not increase counter #{counter_name}, Api unreachable"
       raise e
     end
-    raise DuplicatedCounterError if resp.code == 412
+    #raise DuplicatedCounterError if resp.code == 412
     true
   end
 

data/lib/little_monster/rspec/matchers/have_run.rb

@@ -2,7 +2,7 @@ module LittleMonster::RSpec::Matchers
   class HaveRun
     def initialize(*expected_tasks)
       @expected_tasks = if expected_tasks.length == 1
-                          expected_tasks.first
+                          [expected_tasks.first].flatten
                         else
                           expected_tasks
                         end

data/lib/little_monster/version.rb

@@ -1,3 +1,3 @@
 module LittleMonster
-  VERSION = '0.1.7'.freeze
+  VERSION = '0.1.9'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: little_monster
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.9
 platform: ruby
 authors:
 - arq
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-09-23 00:00:00.000000000 Z
+date: 2016-12-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport