perf_check 0.5.5 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f5a9897a251c91e07e9e20082a6992fcd952964b
-  data.tar.gz: cd11654f7464453b90bb57f52bba33a5f51cfd0a
+  metadata.gz: 4889da43eb0fdd05ddda82e2ac5ce98f39fa2104
+  data.tar.gz: 37f15e1100fbdb95b386921da50a2be65a4daec3
 SHA512:
-  metadata.gz: 097aec836afed6820b9cbdb5b5a9f94797f2a0511d3797b22bbf3c730064ddf2e616402037f10379cb035bb45804453e57d3dcb0403a4a3ee15363af25c4ef2a
-  data.tar.gz: 6d8467a860d950e173f9e419c19533bbb033230e8b9588c29eb119b8c6a4d0d960bf8a4730b46ab87b546b99a9df783f39365fe62ba66c1058348605225d41d2
+  metadata.gz: ded87cf9dd12de4ebd09d4305008ccfc7a9111e61cd51ff682da0cee52f5c97912382ead9a87832c1c86fe069f1c850893be4270e8533733e4c6602e47be66de
+  data.tar.gz: 9d0960078d6740851e1c79f8d577a137b733a3c08d22c6d5c690a56540dfa3eb3bdf56289e71b149fa937de19280f9df7f81e42993ba1fba391049bc0921adc3

data/bin/perf_check

@@ -46,8 +46,6 @@ perf_check.run
 
 if perf_check.options.brief
   perf_check.print_brief_results
-elsif perf_check.options.json
-  perf_check.print_json_results
 else
   perf_check.print_full_results
 end

data/lib/perf_check/config.rb

@@ -47,10 +47,6 @@ class PerfCheck
         options.headers[key.strip] = value.strip
       end
 
-      opts.on('--json', '-j') do
-        options.json = true
-      end
-
       opts.on('--input FILE', '-i') do |input|
         File.readlines(input).each do |resource|
           ARGV << resource.strip
@@ -66,12 +62,12 @@ class PerfCheck
         options.brief = true
       end
 
-#      opts.on('--diff') do
-#        options.diff = true
-#        options.brief = true
-#        options.verify_responses = true
-#        options.number_of_requests = 1
-#      end
+     opts.on('--diff') do
+       options.diff = true
+       options.brief = true
+       options.verify_responses = true
+       options.number_of_requests = 1
+     end
 
       opts.separator ''
       opts.separator <<EOF
@@ -101,8 +97,8 @@ EOF
     end
   end
 
-  def self.diff_options
-    @@diff_options ||=
+  def diff_options
+    @diff_options ||=
       ['-U3', '--ignore-matching-lines=/mini-profiler-resources/includes.js']
   end
 end

data/lib/perf_check/output.rb

@@ -66,47 +66,4 @@ class PerfCheck
       print_diff_results(test.response_diff) if options.verify_responses
     end
   end
-
-  def print_json_results
-    results = []
-    test_cases.each do |test|
-      results.push(
-        route: test.resource,
-        latency: test.this_latency,
-        query_count: test.this_query_count,
-        requests: []
-      )
-
-      test.this_profiles.each do |profile|
-        results[-1][:requests].push(
-          latency: profile.latency,
-          query_count: profile.query_count,
-          server_memory: profile.server_memory,
-          response_code: profile.response_code,
-          miniprofiler_url: profile.profile_url
-        )
-      end
-
-      if options.reference
-        results[-1].merge!(
-          reference_latency: test.reference_latency,
-          latency_difference: test.latency_difference,
-          speedup_factor: test.speedup_factor,
-          reference_query_count: test.reference_query_count,
-          reference_requests: []
-        )
-
-        test.reference_profiles.each do |profile|
-          results[-1][:reference_requests].push(
-            latency: profile.latency,
-            query_count: profile.query_count,
-            server_memory: profile.server_memory,
-            response_code: profile.response_code,
-            miniprofiler_url: profile.profile_url
-          )
-        end
-      end
-    end
-    puts JSON.pretty_generate(results)
-  end
 end

data/lib/perf_check/test_case.rb

@@ -80,13 +80,13 @@ class PerfCheck
     end
 
     def response_diff
-      diff = Diffy::Diff.new(this_response, reference_response,
-                             :diff => PerfCheck.diff_options)
+      diff = Diffy::Diff.new(reference_response, this_response,
+                             diff: perf_check.diff_options)
       if diff.to_s.empty?
         OpenStruct.new(:changed? => false)
       else
-        FileUtils.mkdir_p("#{Rails.root}/tmp/perf_check/diffs")
-        file = `mktemp -u "#{Rails.root}/tmp/perf_check/diffs/XXXXXXXXXX"`.strip
+        FileUtils.mkdir_p("#{perf_check.app_root}/tmp/perf_check/diffs")
+        file = `mktemp -u "#{perf_check.app_root}/tmp/perf_check/diffs/XXXXXXXXX"`.strip
         [:text, :html].each do |format|
           ext = {:text => 'diff', :html => 'html'}[format]
           File.open("#{file}.#{ext}", 'w'){ |f| f.write(diff.to_s(format)) }

data/lib/perf_check.rb

@@ -4,7 +4,6 @@ require 'fileutils'
 require 'benchmark'
 require 'ostruct'
 require 'colorize'
-require 'json'
 require 'logger'
 
 class PerfCheck

metadata

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