profiler 0.0.2 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ba83126bb2d73a6124c20eced9b92fea62620f04
-  data.tar.gz: d3a6dcc7351a199057f7a1f67bec34fd47ca9aa3
+  metadata.gz: b2b5b4acdc7c27039b160612f45651bdefc73f44
+  data.tar.gz: 685344f67903c29b1a882b7fa482143e1527f50c
 SHA512:
-  metadata.gz: dec533b92111776a251d221bc6a750c6f311cebeb22f0acc07736084f180cd4519594b80342bcad0558691732add7c8efac1e672557ac06a82ccb12b807ff3e1
-  data.tar.gz: e6ee3de1590e5eff0d0314a1c3f95794ff5b31b42f12a5094fbbbbbe53d63cd1b6e2ee492892b8cdea7dbae001e011d4cff526c05dd8c0075e00543f90289e75
+  metadata.gz: eba24738f62b575c4b187d18b6707a55304a59046fecf85a6be5e82775d861f541bad9eaed2ae7c5bd8cb5ada7f9f915a1c71ddabe631c4d3398b240a22f8c6e
+  data.tar.gz: b419dc218e2f96a1761cc01cd7aad6e0e69ad7c9311fd2a5a3fb42212700401035af9ce94182159c37cac2a6acd419b1b728e6539a5cf55a33105572a51bb4bb

data/lib/profiler.rb

@@ -21,6 +21,7 @@ module Profiler
     end
 
     def check(caller_offset=0)
+      raise "Profiler hasn't started yet. Please call Profiler.start first." if self.profile.nil?
       return if is_paused
       profile.check(caller_offset)
     end
@@ -28,5 +29,9 @@ module Profiler
     def stop
       profile.stop
     end
+
+    def result
+      profile.result
+    end
   end
 end

data/lib/profiler/printers/flat_printer.rb

@@ -25,7 +25,9 @@ module Profiler
 
     def get_body
       column_lengths = get_column_lengths
-      result.paths_durations.map do |path, duration|
+      paths_durations = result.paths_durations
+      paths_durations = paths_durations.sort_by(&:last).reverse
+      paths_durations.map do |path, duration|
         count = result.paths_counts[path]
         columns = [
           duration.round(5),

data/lib/profiler/profile.rb

@@ -1,6 +1,6 @@
 module Profiler
   class Profile
-    attr_accessor :previous_line, :previous_time, :paths_counts, :paths_durations
+    attr_accessor :previous_line, :previous_time, :paths_counts, :paths_durations, :result
 
     def initialize
       self.paths_counts = Hash.new(0)
@@ -20,9 +20,13 @@ module Profiler
     end
 
     def stop
-      result = Result.new(self)
-      FlatPrinter.new(result).print
-      result
+      self.result = Result.new(self)
+      print
+      nil
+    end
+
+    def print
+      FlatPrinter.new(self.result).print
     end
   end
 end

data/lib/profiler/version.rb

@@ -1,3 +1,3 @@
 module Profiler
-  VERSION = '0.0.2'
+  VERSION = '0.1.0'
 end

data/spec/printers/flat_printer_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe Profiler::FlatPrinter do
+  it "sorts by longest duration" do
+    Profiler.start
+    Profiler.check
+    Profiler.check
+    Profiler.check
+    expect(STDOUT).to receive(:puts) do |string|
+      lines = string.split("\n")
+      durations = lines[1..3].map { |line| line.split(/\s/).first.to_f }
+      expect(durations).to eq(durations.sort.reverse)
+    end
+    Profiler.stop
+  end
+end

data/spec/profiler_spec.rb

@@ -9,7 +9,8 @@ describe Profiler do
     it "records one path" do
       Profiler.start
       Profiler.check
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths.length).to eq(1)
     end
 
@@ -17,14 +18,16 @@ describe Profiler do
       Profiler.start
       Profiler.check
       Profiler.check
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths.length).to eq(2)
     end
 
     it "records one call" do
       Profiler.start
       Profiler.check
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths_counts.values).to eq([1])
     end
 
@@ -33,14 +36,16 @@ describe Profiler do
       5.times do
         Profiler.check
       end
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths_counts.values).to eq([1, 4])
     end
 
     it "records paths" do
       Profiler.start
       Profiler.check
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths.length).to eq(1)
       expect(result.paths.first).to start_with(' -> profiler_spec.rb:')
     end
@@ -48,7 +53,8 @@ describe Profiler do
     it "records durations" do
       Profiler.start
       Profiler.check
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths_durations.length).to eq(1)
       expect(result.paths_durations.values.first).to be_between(0, 0.1)
     end
@@ -60,7 +66,8 @@ describe Profiler do
       Profiler.check
       Profiler.resume
       Profiler.check
-      result = Profiler.stop
+      Profiler.stop
+      result = Profiler.result
       expect(result.paths.length).to eq(2)
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: profiler
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.1.0
 platform: ruby
 authors:
 - Tom Benner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-04 00:00:00.000000000 Z
+date: 2015-07-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -43,6 +43,7 @@ files:
 - lib/profiler/result.rb
 - lib/profiler/version.rb
 - profiler.gemspec
+- spec/printers/flat_printer_spec.rb
 - spec/profiler_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/tombenner/profiler