perf_check 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3ea85bab0194c6138083f20d54ddccd20024c886
-  data.tar.gz: 9479060df80ba93d13695da8953eed296f24fa9e
+  metadata.gz: a0044f60e2a24034b680d872ba644802609544bd
+  data.tar.gz: a6af1d7e8a2fa0407874e4c0a37081da8de5941c
 SHA512:
-  metadata.gz: 71f853f4b2adb6f2a713f71f8f687417339a6435f376bc6310840f66f28eef64a4efea4665494ccf0836580e813f0ef0c34b89ff2c287f74622d3690b8d86a7d
-  data.tar.gz: c76944ca5d33b1dc1d84eea2ea7d7a376b7871a1b19f0cd77be70650f83efa69dfccb8e9be0e54c6f320fbfe79003b1a76d23be9d1171c3fb9d63c472e6267aa
+  metadata.gz: 7cefb6316cea89e4f891a6430407001a22bf44757e08e857247bdda84eaa75f2fe9c1b545c2556a1d1e3f2ec8b24941d34d3b41518572217a6bcb375075e0f46
+  data.tar.gz: 8212ec527aa899f2a57efbbba827eee7a01751112377055f98d31599fcb50aa0476dd092e9aa71cb76021b88c8720af9086089849b80f67b312f4cb11f53d053

data/lib/perf_check/git.rb

@@ -32,6 +32,7 @@ class PerfCheck
         system('git stash -q >/dev/null')
         abort("Problem with git stash! Bailing...") unless $?.success?
         at_exit do
+          system('git checkout .')
           Git.pop
         end
       end

data/lib/perf_check/railtie.rb

@@ -21,6 +21,8 @@ class PerfCheck
         config = ActiveSupport::Configurable::Configuration
         config.send(:define_method, :perform_caching){ fragment_caching }
       end
+      
+      PerfCheck::Server.seed_random!
     end
   end
 end

data/lib/perf_check/server.rb

@@ -35,6 +35,29 @@ class PerfCheck
       end
     end
 
+    def self.seed_random!
+      # Seed random, for --verify purposes
+      srand(1)
+
+      # SecureRandom cannot be seeded, so we have to monkey patch it instead
+      
+      def SecureRandom.hex(n=16)
+        '4' * n
+      end
+      def SecureRandom.random_bytes(n=16)
+        '4' * n
+      end
+      def SecureRandom.random_number(n=0)
+        n > 4 ? 4 : n
+      end
+      def SecureRandom.urlsafe_base64(n=16, padding=false)
+        '4' * (4*n / 3)
+      end
+      def SecureRandom.uuid
+        "00000000-0000-0000-0000-000000000004"
+      end
+    end
+
     def self.sign_cookie_data(key, data, opts={})
       opts[:serializer] ||= Marshal
       secret = Rails.application.config.secret_token
@@ -80,7 +103,7 @@ class PerfCheck
       mp_link
     end
 
-    def profile
+    def profile      
       http = Net::HTTP.new(host, port).tap{ |http| http.read_timeout = 1000 }
       response = nil
       prepare_to_profile

data/lib/perf_check/test_case.rb

@@ -3,7 +3,7 @@
 class PerfCheck
   class TestCase
     attr_accessor :resource, :controller, :action, :format
-    attr_accessor :latencies, :cookie
+    attr_accessor :cookie, :this_response, :reference_response
     attr_accessor :this_latencies, :reference_latencies
 
     def initialize(route)
@@ -11,7 +11,6 @@ class PerfCheck
 
       self.this_latencies = []
       self.reference_latencies = []
-      self.latencies = this_latencies
 
       self.controller = params[:controller].split('/')[-1]
       self.action = params[:action]
@@ -19,17 +18,23 @@ class PerfCheck
       self.resource = route
     end
 
+    def switch_to_reference_context
+      @context = :reference
+    end
+
     def run(server, options)
       print("\t"+'request #'.underline)
       print("  "+'latency'.underline)
       print("   "+'server rss'.underline)
       puts("   "+'profiler data'.underline)
 
+      latencies = (@context == :reference) ? reference_latencies : this_latencies
+
       headers = {'Cookie' => "#{cookie}"}
       unless self.format
         headers['Accept'] = 'text/html,application/xhtml+xml,application/xml'
       end
-      options.number_of_requests.times do |i|
+      (options.number_of_requests+1).times do |i|
         profile = server.profile do |http|
           http.get(resource, headers)
         end
@@ -45,10 +50,20 @@ class PerfCheck
           exit(1)
         end
 
+        next if i.zero?
+
+        if i == 1
+          if @context == :reference
+            self.reference_response = profile.response_body
+          else
+            self.this_response = profile.response_body
+          end
+        end
+
         printf("\t%2i:\t   %.1fms   %4dMB\t  %s\n",
                i, profile.latency, server.mem, profile.profile_url)
 
-        self.latencies << profile.latency
+        latencies << profile.latency
       end
       puts
     end

data/lib/perf_check.rb

@@ -57,7 +57,7 @@ class PerfCheck
       if i == 1
         Git.stash_if_needed
         Git.checkout_reference(options.reference)
-        test_cases.each{ |x| x.latencies = x.reference_latencies }
+        test_cases.each{ |x| x.switch_to_reference_context }
       end
 
       test_cases.each do |test|
@@ -110,6 +110,16 @@ class PerfCheck
       puts("master: ".rjust(15)     + "#{master_latency}")
       puts("your branch: ".rjust(15)+ "#{this_latency}")
       puts(("change: ".rjust(15)     + "#{formatted_change}").bold.send(color))
+      
+      Tempfile.new('this_response').tap do |this_response|
+        Tempfile.new('reference_response').tap do |reference_response|
+          this_response.write(test.this_response)
+          reference_response.write(test.reference_response)
+          this_response.close
+          reference_response.close
+          system('diff', '-U0', this_response.path, reference_response.path)
+        end
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: perf_check
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - rubytune