limited_red 0.3.6 → 0.3.7

data/lib/limited_red/client.rb

@@ -18,8 +18,8 @@ module LimitedRed
                          :token => token_for(data.merge(:build_id => build_id))})
 
       @thread_pool.with_a_thread_run do
-        result = @adapter.post("/projects/#{@project_id}/builds/#{build_id}/results", :body => data)
-        @out.puts error_message(result) if error?(result)
+        response = @adapter.post("/projects/#{@project_id}/builds/#{build_id}/results", :body => data)
+        log(response) if error?(response)
       end
     end
 
@@ -30,8 +30,8 @@ module LimitedRed
                          :build_id => build_id, :token => token_for(data)})
         
       @thread_pool.with_a_thread_run do
-        result = @adapter.post("/projects/#{@project_id}/builds", :body => data)
-        @out.puts error_message(result) if error?(result)
+        response = @adapter.post("/projects/#{@project_id}/builds", :body => data)
+        log(response) if error?(response)
       end
     end
       
@@ -40,8 +40,11 @@ module LimitedRed
 
       response = @adapter.get("/projects/#{@project_id}/features/fails?user=#{@username}")
 
-      return [] if response.nil? || response.empty?
-      response.code == 200 ? response.body.split(" ") : []
+      if response.nil? || response.empty? || error?(response)
+        return []
+      else
+        response.body.split(" ")
+      end
     end
             
     private
@@ -64,18 +67,16 @@ module LimitedRed
       data == "" || data.nil? ? "" : data.join("")
     end
 
-    def error?(result)
-      result && !result.body.empty?
+    def error?(response)
+      response.code != 200
+    end
+    
+    def log(response)
+      @out.puts(error_message(response))
     end
       
-    def error_message(error_msg)
-      if (ENV['LIMITED_RED_ENV'] == 'test') || (ENV['LIMITED_RED_ENV'] == 'development')
-        message = error_msg
-      else
-        message = ""
-      end
-      
-      "\nLimited Red had a problem logging your test results.\n#{message}"
+    def error_message(response)
+      "\nLimited Red had a problem logging your test results: #{response ? response.body : ''}"
     end
   end
 end

data/lib/limited_red/version.rb

@@ -1,5 +1,5 @@
 module LimitedRed # :nodoc:
   module Version # :nodoc:
-    STRING = '0.3.6'
+    STRING = '0.3.7'
   end
 end

metadata

@@ -1,47 +1,60 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: limited_red
-version: !ruby/object:Gem::Version
-  version: 0.3.6
-  prerelease: 
+version: !ruby/object:Gem::Version 
+  prerelease: false
+  segments: 
+  - 0
+  - 3
+  - 7
+  version: 0.3.7
 platform: ruby
-authors:
+authors: 
 - Joseph Wilk
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-19 00:00:00.000000000Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2012-02-20 00:00:00 +00:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
   name: httparty
-  requirement: &70244157382760 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - =
-      - !ruby/object:Gem::Version
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - "="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        - 8
+        - 1
         version: 0.8.1
   type: :runtime
-  prerelease: false
-  version_requirements: *70244157382760
-- !ruby/object:Gem::Dependency
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
   name: cucumber
-  requirement: &70244157381320 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 1
+        - 1
+        - 4
         version: 1.1.4
   type: :runtime
-  prerelease: false
-  version_requirements: *70244157381320
-description: Learn and adapt from you test metrics. Used with the www.limited-red.com
-  service. Tests are priorited by the probability of failure
+  version_requirements: *id002
+description: Learn and adapt from you test metrics. Used with the www.limited-red.com service. Tests are priorited by the probability of failure
 email: joe@josephwilk.net
 executables: []
+
 extensions: []
-extra_rdoc_files:
+
+extra_rdoc_files: 
 - LICENSE
 - README.rdoc
-files:
+files: 
 - lib/limited_red/adapter/httparty.rb
 - lib/limited_red/client.rb
 - lib/limited_red/config.rb
@@ -55,31 +68,35 @@ files:
 - lib/limited_red.rb
 - LICENSE
 - README.rdoc
+has_rdoc: true
 homepage: http://www.limited-red.com
-licenses:
+licenses: 
 - MIT
 post_install_message: 
-rdoc_options:
+rdoc_options: 
 - --charset=UTF-8
-require_paths:
+require_paths: 
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
 requirements: []
+
 rubyforge_project: 
-rubygems_version: 1.8.6
+rubygems_version: 1.3.6
 signing_key: 
 specification_version: 3
 summary: Client for www.limited-red.com service
 test_files: []
-has_rdoc: 
+