hot_or_not 0.2.2 → 0.2.3

data/Gemfile

@@ -16,6 +16,6 @@ end
 gem 'rake', '~> 0.9.2'
 gem 'rest-client', '~> 1.6.1'
 gem 'diffy', '~> 2.0.3'
-gem 'facets', '~> 2.9.1'
+gem 'facets', '= 2.9.1'
 gem 'json', '~> 1.5.2'
 gem 'hpricot', '~> 0.8.4'

data/Gemfile.lock

@@ -2,7 +2,7 @@ GEM
   remote: http://rubygems.org/
   specs:
     diffy (2.0.4)
-    facets (2.9.2)
+    facets (2.9.1)
     git (1.2.5)
     hpricot (0.8.4)
     jeweler (1.6.4)
@@ -26,7 +26,7 @@ PLATFORMS
 DEPENDENCIES
   bundler (~> 1.0.0)
   diffy (~> 2.0.3)
-  facets (~> 2.9.1)
+  facets (= 2.9.1)
   hpricot (~> 0.8.4)
   jeweler (~> 1.6.0)
   json (~> 1.5.2)

data/VERSION

@@ -1 +1 @@
-0.2.2
+0.2.3

data/lib/hot_or_not/announcers/inline_announcer.rb

@@ -42,7 +42,7 @@ module HotOrNot
     private
     def say(what, result)
       what += ':'
-      puts "%-10s #{result.full_name}" % [what]
+      puts "%-10s %-50s %.4fs, %.4fs" % [what, result.full_name, result.side_a_latency, result.side_b_latency]
     end
 
     def results_dir

data/lib/hot_or_not/comparison_result.rb

@@ -19,6 +19,14 @@ module HotOrNot
       init_message unless success?
     end
 
+    def side_a_latency
+      @side_a_results.latency
+    end
+
+    def side_b_latency
+      @side_b_results.latency
+    end
+
     def success?
       !error? &&
         @side_a_results.code == @side_b_results.code &&

data/lib/hot_or_not/url_result.rb

@@ -9,21 +9,22 @@ module HotOrNot
         options = DEFAULT_OPTIONS.merge options
         options[:url] = url
         response, error = nil, nil
+        start = Time.now
         begin
           response = RestClient::Request.execute(options)
         rescue RestClient::Exception => e
           error = e
         end
 
-        new url, response, error
+        new url, response, error, Time.now - start
       end
     end
 
-    attr_reader :url
+    attr_reader :url, :latency
     def_delegators :@response, :code, :body, :headers
 
-    def initialize(url, response, error)
-      @url, @error = url, error
+    def initialize(url, response, error, latency)
+      @url, @error, @latency = url, error, latency
       @response = response || error.response
     end
 

data/test/announcers/inline_announcer_test.rb

@@ -18,7 +18,7 @@ module HotOrNot
           urls = [mock_compare_url('Passing Test', '/api/foo', 'foo', 'foo')]
           Runner.new(urls, @announcer).run!
 
-          expected_output = /^Hot:\s+Passing Test$/
+          expected_output = /^Hot:\s+Passing Test\s+1.0000s, 1.0000s$/
           assert_match expected_output, test_output[1].chomp
         end
 
@@ -26,7 +26,7 @@ module HotOrNot
           urls = [mock_compare_url('Failing Test', '/api/foo', 'foo', 'bar')]
           Runner.new(urls, @announcer).run!
 
-          expected_output = /^Not Hot:\s+Failing Test$/
+          expected_output = /^Not Hot:\s+Failing Test\s+1.0000s, 1.0000s$/
           assert_match expected_output, test_output[1].chomp
         end
 
@@ -34,7 +34,7 @@ module HotOrNot
           urls = [mock_compare_url('Error test', '/api/foo', 'foo', 'bar', 404)]
           Runner.new(urls, @announcer).run!
 
-          assert_match /^Error:\s+Error test$/, test_output[1].chomp
+          assert_match /^Error:\s+Error test\s+1.0000s, 1.0000s$/, test_output[1].chomp
         end
       end
     end

data/test/comparison_result_test.rb

@@ -25,8 +25,8 @@ module HotOrNot
           context "and are strings" do
             setup do
               response = FakeResponse.new 'foo'
-              side_a_results = UrlResult.new @compare_url.side_a, response, nil
-              side_b_results = UrlResult.new @compare_url.side_b, response, nil
+              side_a_results = UrlResult.new @compare_url.side_a, response, nil, 1
+              side_b_results = UrlResult.new @compare_url.side_b, response, nil, 1
               @result = ComparisonResult.new @compare_url, side_a_results, side_b_results
             end
 
@@ -41,8 +41,8 @@ module HotOrNot
               hash2 = {'a'=>3,'b'=>4}
               response_a = FakeResponse.new [hash1, hash2].to_json, 200, :content_type => 'application/json'
               response_b = FakeResponse.new [hash2, hash1].to_json, 200, :content_type => 'application/json'
-              side_a_results = UrlResult.new @compare_url.side_a, response_a, nil
-              side_b_results = UrlResult.new @compare_url.side_b, response_b, nil
+              side_a_results = UrlResult.new @compare_url.side_a, response_a, nil, 1
+              side_b_results = UrlResult.new @compare_url.side_b, response_b, nil, 1
               @result = ComparisonResult.new @compare_url, side_a_results, side_b_results
             end
 
@@ -58,8 +58,8 @@ module HotOrNot
               body_b = "<div>bar</div><div id='content'>foo</div>"
               response_a = FakeResponse.new body_a, 200
               response_b = FakeResponse.new body_b, 200
-              side_a_results = UrlResult.new compare_url.side_a, response_a, nil
-              side_b_results = UrlResult.new compare_url.side_b, response_b, nil
+              side_a_results = UrlResult.new compare_url.side_a, response_a, nil, 1
+              side_b_results = UrlResult.new compare_url.side_b, response_b, nil, 1
               @result = ComparisonResult.new compare_url, side_a_results, side_b_results
             end
 
@@ -75,8 +75,8 @@ module HotOrNot
               body_b = " foo\nbar"
               response_a = FakeResponse.new body_a, 200
               response_b = FakeResponse.new body_b, 200
-              side_a_results = UrlResult.new compare_url.side_a, response_a, nil
-              side_b_results = UrlResult.new compare_url.side_b, response_b, nil
+              side_a_results = UrlResult.new compare_url.side_a, response_a, nil, 1
+              side_b_results = UrlResult.new compare_url.side_b, response_b, nil, 1
               @result = ComparisonResult.new compare_url, side_a_results, side_b_results
             end
 
@@ -90,8 +90,8 @@ module HotOrNot
           setup do
             response_a = FakeResponse.new 'side_a'
             response_b = FakeResponse.new 'side_b'
-            side_a_results = UrlResult.new @compare_url.side_a, response_a, nil
-            side_b_results = UrlResult.new @compare_url.side_b, response_b, nil
+            side_a_results = UrlResult.new @compare_url.side_a, response_a, nil, 1
+            side_b_results = UrlResult.new @compare_url.side_b, response_b, nil, 1
             @result = ComparisonResult.new @compare_url, side_a_results, side_b_results
           end
 
@@ -114,8 +114,8 @@ module HotOrNot
               body_b = " foo\nbaz"
               response_a = FakeResponse.new body_a, 200
               response_b = FakeResponse.new body_b, 200
-              side_a_results = UrlResult.new compare_url.side_a, response_a, nil
-              side_b_results = UrlResult.new compare_url.side_b, response_b, nil
+              side_a_results = UrlResult.new compare_url.side_a, response_a, nil, 1
+              side_b_results = UrlResult.new compare_url.side_b, response_b, nil, 1
               @result = ComparisonResult.new compare_url, side_a_results, side_b_results
             end
 
@@ -129,8 +129,8 @@ module HotOrNot
           setup do
             response_a = FakeResponse.new 'side_a'
             error_b = RestClient::InternalServerError.new
-            side_a_results = UrlResult.new @compare_url.side_a, response_a, nil
-            side_b_results = UrlResult.new @compare_url.side_b, nil, error_b
+            side_a_results = UrlResult.new @compare_url.side_a, response_a, nil, 1
+            side_b_results = UrlResult.new @compare_url.side_b, nil, error_b, 1
             @result = ComparisonResult.new @compare_url, side_a_results, side_b_results
           end
 

data/test/test_helper.rb

@@ -41,8 +41,8 @@ class Test::Unit::TestCase
     HotOrNot::CompareUrl.new(name, url, 'http://side_a', 'http://side_b').tap do |compare_url|
       response_a, error_a = response_and_error_for body_a, code_a
       response_b, error_b = response_and_error_for body_b, code_b
-      side_a_result = HotOrNot::UrlResult.new compare_url.side_a, response_a, error_a
-      side_b_result = HotOrNot::UrlResult.new compare_url.side_b, response_b, error_b
+      side_a_result = HotOrNot::UrlResult.new compare_url.side_a, response_a, error_a, 1
+      side_b_result = HotOrNot::UrlResult.new compare_url.side_b, response_b, error_b, 1
       HotOrNot::ComparisonResult.expects(:for).with(compare_url).returns HotOrNot::ComparisonResult.new(compare_url, side_a_result, side_b_result)
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hot_or_not
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-09-19 00:00:00.000000000Z
+date: 2012-03-14 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70361786977860 !ruby/object:Gem::Requirement
+  requirement: &70139178274740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 0.9.2
   type: :runtime
   prerelease: false
-  version_requirements: *70361786977860
+  version_requirements: *70139178274740
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &70361786975660 !ruby/object:Gem::Requirement
+  requirement: &70139178273980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 1.6.1
   type: :runtime
   prerelease: false
-  version_requirements: *70361786975660
+  version_requirements: *70139178273980
 - !ruby/object:Gem::Dependency
   name: diffy
-  requirement: &70361786974860 !ruby/object:Gem::Requirement
+  requirement: &70139178273120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,21 +43,21 @@ dependencies:
         version: 2.0.3
   type: :runtime
   prerelease: false
-  version_requirements: *70361786974860
+  version_requirements: *70139178273120
 - !ruby/object:Gem::Dependency
   name: facets
-  requirement: &70361786973720 !ruby/object:Gem::Requirement
+  requirement: &70139178271960 !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - =
       - !ruby/object:Gem::Version
         version: 2.9.1
   type: :runtime
   prerelease: false
-  version_requirements: *70361786973720
+  version_requirements: *70139178271960
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &70361786972580 !ruby/object:Gem::Requirement
+  requirement: &70139178271100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,10 +65,10 @@ dependencies:
         version: 1.5.2
   type: :runtime
   prerelease: false
-  version_requirements: *70361786972580
+  version_requirements: *70139178271100
 - !ruby/object:Gem::Dependency
   name: hpricot
-  requirement: &70361786948600 !ruby/object:Gem::Requirement
+  requirement: &70139178269700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -76,10 +76,10 @@ dependencies:
         version: 0.8.4
   type: :runtime
   prerelease: false
-  version_requirements: *70361786948600
+  version_requirements: *70139178269700
 - !ruby/object:Gem::Dependency
   name: shoulda
-  requirement: &70361786946180 !ruby/object:Gem::Requirement
+  requirement: &70139178268220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70361786946180
+  version_requirements: *70139178268220
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70361786945240 !ruby/object:Gem::Requirement
+  requirement: &70139178267260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -98,10 +98,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70361786945240
+  version_requirements: *70139178267260
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70361786943660 !ruby/object:Gem::Requirement
+  requirement: &70139178266380 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -109,10 +109,10 @@ dependencies:
         version: 1.6.0
   type: :development
   prerelease: false
-  version_requirements: *70361786943660
+  version_requirements: *70139178266380
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &70361786894020 !ruby/object:Gem::Requirement
+  requirement: &70139178265280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,10 +120,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70361786894020
+  version_requirements: *70139178265280
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &70361786892180 !ruby/object:Gem::Requirement
+  requirement: &70139178264260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -131,7 +131,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70361786892180
+  version_requirements: *70139178264260
 description: Run this with a new version of your site comparing to current version
   to verify no break changes are made to your api's
 email: asher.friedman@gmail.com
@@ -183,7 +183,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2576571352742383337
+      hash: -2710309296133271667
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -192,7 +192,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.6
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: A tool for side-by-side comparsion of web apis