perf_check 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2d34aaf3aa40817cfe8ac6f6e00357da051c6855
-  data.tar.gz: 158f5996b44d75d5999909952715068f1f5f3984
+  metadata.gz: 2fa536f7c7af14d67a6489e83df8d214198d26f5
+  data.tar.gz: f6cc644442eb0e543487ffd502b3d721c2288565
 SHA512:
-  metadata.gz: ff7b9258d900af3f851ffa546b05851b49995861e31c3e866d82c47141ba21a54a694bf08d2da8bebd615fddcf601d710521d44aee1e10baa811c3b0d3ca98b0
-  data.tar.gz: edd58cb96e37aa5dfbb1dd3125d737f42a04f0db52d52e0880289131dd0e4a347ee68eb91fba25a3f9622e485880e0760a959080195b1015890551ba6f7f7dcf
+  metadata.gz: 04c51faf695bbb85bc11fd9b7240cd84dc7ccc01b0caefdeb9ab1d5c709a8934072f1204e28d1e9a9f655539fe91706a8906713de84c0d38d4b922aae31ed824
+  data.tar.gz: 8004ceeb790ee66f23950acd15f3d894ad646b82bbe015443fb3bbce3eee149941ff0346572f9bc4281548d32f39a23b4eed98e562d117d68f8fd4845255ba5d

data/bin/perf_check

@@ -51,8 +51,8 @@ opts = OptionParser.new do |opts|
     options.reference = nil
   end
 
-  opts.on('--clear-cache', 'Call Rails.cache.clear before running benchmark') do
-    options.clear_cache = true
+  opts.on('--no-caching', 'Do not enable fragment caching') do
+    ENV['PERF_CHECK_NOCACHING'] = true
   end
 
   opts.on('--302-success', 'Consider HTTP 302 code a successful request') do

data/lib/perf_check/railtie.rb

@@ -17,8 +17,9 @@ class PerfCheck
         config = Rails::Application::Configuration
         config.send(:define_method, :cache_classes){ true }
 
+        fragment_caching = !ENV['PERF_CHECK_NOCACHING']
         config = ActiveSupport::Configurable::Configuration
-        config.send(:define_method, :perform_caching){ true }
+        config.send(:define_method, :perform_caching){ fragment_caching }
       end
     end
   end

data/lib/perf_check.rb

@@ -86,11 +86,17 @@ class PerfCheck
 
       master_latency = sprintf('%.1fms', test.reference_latency)
       this_latency = sprintf('%.1fms', test.this_latency)
-
       difference = sprintf('%+.1fms', test.latency_difference)
-      percent_change = 100 * (test.latency_difference / test.reference_latency).abs
-      formatted_change = sprintf('%d%%', percent_change)
-      if percent_change < 5
+
+      if test.latency_difference < 0
+        change_factor = test.reference_latency / test.this_latency
+      else
+        change_factor = test.this_latency / test.reference_latency
+      end
+      formatted_change = sprintf('%.1fx', change_factor)
+
+      percent_change = 100*(test.latency_difference / test.reference_latency).abs
+      if percent_change < 10
         formatted_change = "yours is about the same"
         color = :blue
       elsif test.latency_difference < 0

metadata

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