cucumber_statistics 2.1.1 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 907013082e484671f77d0cbf28fa4852d534f0d9
-  data.tar.gz: c3cdaa1c2838e73ecf63a03fd72a965e7168cfe3
+  metadata.gz: f092f72aee7ed2a12f831f6121609828b7779efa
+  data.tar.gz: 168d3905bad7a316082ef66318e69d614842b305
 SHA512:
-  metadata.gz: f492e8bf780dcf55339e82f242a774afc4e1f907f227f9bd973f18f59ef8803f4b5a1d48b660e62b863a214e73d2b5b2127cb02bf40392515b3ff30bfba819eb
-  data.tar.gz: 618be5d5530b12dcb4786ddfea2099252a76bb7f353886537f91bb01903907d41f7e598570e656b9edfb3b283ec93bb7a9bda8debc4bbf1b6451f5989d5450cb
+  metadata.gz: 3a59558e79b0de5fc6057510d3b64e58358e5df8b091811a9ed5079849b7f484ef1ef6979c97b8b8f26924dec76479ec69d2329ff9343f793f5d30471ce568aa
+  data.tar.gz: 0ce6a14d9fb2fb8b008e3896da559b27381d378c3f7955622ab3dfee02a80200ee1176d8672662ed18e4663e9dfd0b45e9d2e62d9dceb43493b3d3a42b7a34c1

data/.ruby-version

@@ -1 +1 @@
-ruby-2.2.2
+ruby-2.3.1

data/lib/cucumber_statistics.rb

@@ -1,6 +1,8 @@
 require 'cucumber_statistics/configuration'
 require 'cucumber_statistics/step_statistics'
+require 'cucumber_statistics/scenario_statistics'
+require 'cucumber_statistics/feature_statistics'
 require 'cucumber_statistics/overall_statistics'
 require 'cucumber_statistics/unused_steps'
 require 'cucumber_statistics/formatter'
-require 'cucumber_statistics/renderer'
+require 'cucumber_statistics/renderer'

data/lib/cucumber_statistics/configuration.rb

@@ -26,6 +26,13 @@ module CucumberStatistics
         tmp_file('step_statistics.html')
       end
 
+      def result_scenario_statistics
+        tmp_file('scenario_statistics.html')
+      end
+
+      def result_feature_statistics
+        tmp_file('feature_statistics.html')
+      end
 
       def resolve_path_from_root(relative_path)
         if defined?(Rails)
@@ -54,4 +61,4 @@ module CucumberStatistics
       #end
     end
   end
-end
+end

data/lib/cucumber_statistics/feature_statistics.rb

@@ -0,0 +1,28 @@
+module CucumberStatistics
+  class FeatureStatistics
+    def initialize
+      @all = Hash.new
+    end
+
+    def record feature_name, duration, file
+      short_file = file[file.index('features').to_i..-1]
+
+      feature_result = @all[short_file]
+      feature_result ||= Hash.new
+      feature_result[:duration] = duration
+      feature_result[:feature_name] = feature_name
+
+      @all[short_file] ||= feature_result
+    end
+
+    def all
+      @all
+    end
+
+    def sort_by_property property
+      result = @all.sort {|a,b| a.last[property.to_sym] <=> b.last[property.to_sym]}
+      result
+    end
+
+  end
+end

data/lib/cucumber_statistics/formatter.rb

@@ -7,6 +7,8 @@ module CucumberStatistics
 
       @overall_statistics = OverallStatistics.new
       @step_statistics = StepStatistics.new
+      @scenario_statistics = ScenarioStatistics.new
+      @feature_statistics = FeatureStatistics.new
       @unused_steps = UnusedSteps.new
     end
 
@@ -29,6 +31,29 @@ module CucumberStatistics
     end
 
 
+    #----------------------------------------------------
+    # Feature Element callbacks
+    #----------------------------------------------------
+    def before_feature_element(feature_element)
+      @scenario_name = ''
+      @scenario_file_colon_line = ''
+      @scenario_start_time = Time.now
+    end
+
+    def after_feature_element(feature_element)
+      @scenario_duration = Time.now - @scenario_start_time
+      @scenario_statistics.record @scenario_name, @scenario_duration, @scenario_file_colon_line
+    end
+
+    #----------------------------------------------------
+    # Feature callbacks
+    #----------------------------------------------------
+    def before_feature(feature)
+      @feature_name = ''
+      @feature_file = ''
+      @feature_start_time = Time.now
+    end
+
     #----------------------------------------------------
     # Overall callbacks
     #----------------------------------------------------
@@ -36,6 +61,13 @@ module CucumberStatistics
     #end
     def scenario_name(keyword, name, file_colon_line, source_indent)
       @overall_statistics.scenario_count_inc
+      @scenario_name = name
+      @scenario_file_colon_line = file_colon_line
+    end
+
+    def feature_name(keyword, name)
+      @overall_statistics.feature_count_inc
+      @feature_name = name
     end
 
     def after_step(step)
@@ -43,7 +75,9 @@ module CucumberStatistics
     end
 
     def after_feature(feature)
-      @overall_statistics.feature_count_inc
+      @feature_file = feature.location.to_s
+      @feature_duration = Time.now - @feature_start_time
+      @feature_statistics.record @feature_name, @feature_duration, @feature_file
     end
 
     def before_features(features)
@@ -62,7 +96,8 @@ module CucumberStatistics
       @step_statistics.calculate
 
       file = Renderer.render_step_statistics @step_statistics, @overall_statistics
-      puts "Cucumber statistics generated to #{file.to_s}"
+      Renderer.render_scenario_statistics @scenario_statistics, @overall_statistics
+      Renderer.render_feature_statistics @feature_statistics, @overall_statistics
     end
   end
-end
+end

data/lib/cucumber_statistics/renderer.rb

@@ -20,6 +20,30 @@ module CucumberStatistics
 
         absolute_file_name
       end
+
+      def render_scenario_statistics(scenario_statistics, overall_statistics)
+        template = Tilt::HamlTemplate.new(File.expand_path('../view/scenario_statistics.html.haml', __FILE__))
+        rendered_content = template.render(RendererHelper.new, scenario_statistics: scenario_statistics, overall_statistics: overall_statistics)
+
+        absolute_file_name = Configuration.result_scenario_statistics
+        File.open(absolute_file_name, 'w') do |f|
+          f.write rendered_content
+        end
+
+        absolute_file_name
+      end
+
+      def render_feature_statistics(feature_statistics, overall_statistics)
+        template = Tilt::HamlTemplate.new(File.expand_path('../view/feature_statistics.html.haml', __FILE__))
+        rendered_content = template.render(RendererHelper.new, feature_statistics: feature_statistics, overall_statistics: overall_statistics)
+
+        absolute_file_name = Configuration.result_feature_statistics
+        File.open(absolute_file_name, 'w') do |f|
+          f.write rendered_content
+        end
+
+        absolute_file_name
+      end
     end
   end
-end
+end

data/lib/cucumber_statistics/renderer_helper.rb

@@ -2,22 +2,38 @@ module CucumberStatistics
 
   class RendererHelper
 
-    def name_td(step_results)
-      %{<td title="#{step_results[1][:file]}">#{step_results[0]}</td>}
+    def name_td(results)
+      %{<td title="#{results[1][:file]}">#{results[0]}</td>}
     end
 
-    def time_td(step_results, metric, *warning_step_results)
-      duration = step_results[1][metric]
+    def scenario_file_td(name, scenario_name)
+      %{<td title="#{scenario_name}">#{name}</td>}
+    end
+
+    def std_file_td(file_name, name)
+      %{<td title="#{name}">#{file_name}</td>}
+    end
+
+    def time_td(results, metric, *warning_results)
+      duration = results[1][metric]
+
+      %{<td #{warning_class(results, warning_results)} data-value="#{duration}" title="#{duration}">#{format(duration)}</td>}
+    end
+
+    def scenario_time_td(duration)
+      %{<td data-value="#{duration}" title="#{duration}">#{format(duration)}</td>}
+    end
 
-      %{<td #{warning_class(step_results, warning_step_results)} data-value="#{duration}" title="#{duration}">#{format(duration)}</td>}
+    def std_time_td(duration)
+      %{<td data-value="#{duration}" title="#{duration}">#{format(duration)}</td>}
     end
 
-    def warning_class(step_results, warning_step_results)
+    def warning_class(results, warning_results)
 
-      if warning_step_results.nil? || warning_step_results.empty?
+      if warning_results.nil? || warning_results.empty?
         should_warn = false
       else
-        should_warn = (step_results[0].eql? warning_step_results[0][0])
+        should_warn = (results[0].eql? warning_results[0][0])
       end
       if should_warn
         %{class="danger"}
@@ -26,8 +42,8 @@ module CucumberStatistics
       end
     end
 
-    def count_td(step_results, metric)
-      value = step_results[1][metric]
+    def count_td(results, metric)
+      value = results[1][metric]
       %{<td data-value="#{value}">#{value}</td>}
     end
 

data/lib/cucumber_statistics/scenario_statistics.rb

@@ -0,0 +1,28 @@
+module CucumberStatistics
+  class ScenarioStatistics
+    def initialize
+      @all = Hash.new
+    end
+
+    def record scenario_name, duration, file_colon_line
+      short_file_colon_line = file_colon_line[file_colon_line.index('features').to_i..-1]
+
+      scenario_result = @all[short_file_colon_line]
+      scenario_result ||= Hash.new
+      scenario_result[:duration] = duration
+      scenario_result[:scenario_name] = scenario_name
+
+      @all[short_file_colon_line] ||= scenario_result
+    end
+
+    def all
+      @all
+    end
+
+    def sort_by_property property
+      result = @all.sort {|a,b| a.last[property.to_sym] <=> b.last[property.to_sym]}
+      result
+    end
+
+  end
+end

data/lib/cucumber_statistics/version.rb

@@ -1,3 +1,3 @@
 module CucumberStatistics
-  VERSION = '2.1.1'
+  VERSION = '2.2.0'
 end

data/lib/cucumber_statistics/view/feature_statistics.html.haml

@@ -0,0 +1,116 @@
+!!!
+%html{:lang => "en"}
+  %head
+    %meta{:charset => "utf-8"}/
+    %meta{:content => "IE=edge", "http-equiv" => "X-UA-Compatible"}/
+    %meta{:content => "width=device-width, initial-scale=1", :name => "viewport"}/
+    %meta{:content => "Cucumber Feature Statistics", :name => "description"}/
+    %meta{:content => "AlienFast", :name => "author"}/
+    %title Cucumber Feature Statistics
+    / Bootstrap core CSS
+    %link{:href => "http://netdna.bootstrapcdn.com/bootstrap/3.1.1/css/bootstrap.min.css", :rel => "stylesheet"}/
+    / Custom styles for this layout
+    :css
+      body {
+          min-height: 2000px;
+          padding-top: 70px;
+      }
+
+      td {
+          white-space: nowrap;
+          /*max-width: 100px;*/
+      }
+      tr td:first-child {
+          overflow: hidden;
+          text-overflow: ellipsis;
+          max-width: 300px;
+      }
+    %link{:href => "https://raw.githack.com/drvic10k/bootstrap-sortable/master/Contents/bootstrap-sortable.css", :rel => "stylesheet"}/
+    / HTML5 shim and Respond.js IE8 support of HTML5 elements and media queries
+    /[if lt IE 9]
+      <script src="https://oss.maxcdn.com/libs/html5shiv/3.7.0/html5shiv.js"></script>
+      <script src="https://oss.maxcdn.com/libs/respond.js/1.4.2/respond.min.js"></script>
+  %body
+    / Fixed navbar
+    .navbar.navbar-default.navbar-fixed-top{:role => "navigation"}
+      .container
+        .navbar-header
+          %button.navbar-toggle{"data-target" => ".navbar-collapse", "data-toggle" => "collapse", :type => "button"}
+            %span.sr-only Toggle navigation
+            %span.icon-bar
+            %span.icon-bar
+            %span.icon-bar
+          %a.navbar-brand{:href => "#"} cucumber_statistics
+        .navbar-collapse.collapse
+          -#%ul.nav.navbar-nav
+            %li.active
+              %a{:href => "#"} Home
+            %li
+              %a{:href => "#about"} About
+            %li
+              %a{:href => "#contact"} Contact
+            %li.dropdown
+              %a.dropdown-toggle{"data-toggle" => "dropdown", :href => "#"}
+                Dropdown
+                %b.caret
+              %ul.dropdown-menu
+                %li
+                  %a{:href => "#"} Action
+                %li
+                  %a{:href => "#"} Another action
+                %li
+                  %a{:href => "#"} Something else here
+                %li.divider
+                %li.dropdown-header Nav header
+                %li
+                  %a{:href => "#"} Separated link
+                %li
+                  %a{:href => "#"} One more separated link
+          %ul.nav.navbar-nav.navbar-right
+            -#%li
+            -#  %a{:href => "../navbar/"} Default
+            -#%li
+            -#  %a{:href => "../navbar-static-top/"} Static top
+            %li
+              %a{:href => "https://github.com/alienfast/cucumber_statistics", :target => 'other'} Github
+        / /.nav-collapse
+    .container
+      / Main component for a primary marketing message or call to action
+      .jumbotron
+        %h1 Cucumber feature statistics
+        %p ... find those slow features.
+
+      .alert.alert-info
+        %span
+          #{overall_statistics.feature_count} Features,
+          #{overall_statistics.scenario_count} Scenarios,
+          #{overall_statistics.step_count} Steps completed in #{format(overall_statistics.duration)}.
+        %span.text-muted.pull-right.small
+          Finished on #{format_date_time(overall_statistics.end_time)}
+
+      %table.table.table-bordered.table-striped.sortable
+        %thead
+          %tr
+            %th Feature File
+            %th{"data-defaultsort" => "desc"} Time
+        %tbody
+
+          - feature_statistics.all.each do |file_name, feature_result|
+
+            %tr
+              = std_file_td file_name, feature_result[:feature_name]
+              = std_time_td feature_result[:duration]
+    / /container
+    /
+      Bootstrap core JavaScript
+      \==================================================
+    / Placed at the end of the document so the pages load faster
+    %script{:src => "https://ajax.googleapis.com/ajax/libs/jquery/1.11.0/jquery.min.js"}
+    %script{:src => "http://netdna.bootstrapcdn.com/bootstrap/3.1.1/js/bootstrap.min.js"}
+    %script{:src => "https://raw.githack.com/drvic10k/bootstrap-sortable/master/Scripts/moment.min.js"}
+    %script{:src => "https://raw.githack.com/drvic10k/bootstrap-sortable/master/Scripts/bootstrap-sortable.js"}
+    :javascript
+      // Initialise on DOM ready
+      $(function() {
+          $.bootstrapSortable(true, 'reversed');
+      });

data/lib/cucumber_statistics/view/scenario_statistics.html.haml

@@ -0,0 +1,116 @@
+!!!
+%html{:lang => "en"}
+  %head
+    %meta{:charset => "utf-8"}/
+    %meta{:content => "IE=edge", "http-equiv" => "X-UA-Compatible"}/
+    %meta{:content => "width=device-width, initial-scale=1", :name => "viewport"}/
+    %meta{:content => "Cucumber Scenario Statistics", :name => "description"}/
+    %meta{:content => "AlienFast", :name => "author"}/
+    %title Cucumber Scenario Statistics
+    / Bootstrap core CSS
+    %link{:href => "http://netdna.bootstrapcdn.com/bootstrap/3.1.1/css/bootstrap.min.css", :rel => "stylesheet"}/
+    / Custom styles for this layout
+    :css
+      body {
+          min-height: 2000px;
+          padding-top: 70px;
+      }
+
+      td {
+          white-space: nowrap;
+          /*max-width: 100px;*/
+      }
+      tr td:first-child {
+          overflow: hidden;
+          text-overflow: ellipsis;
+          max-width: 300px;
+      }
+    %link{:href => "https://raw.githack.com/drvic10k/bootstrap-sortable/master/Contents/bootstrap-sortable.css", :rel => "stylesheet"}/
+    / HTML5 shim and Respond.js IE8 support of HTML5 elements and media queries
+    /[if lt IE 9]
+      <script src="https://oss.maxcdn.com/libs/html5shiv/3.7.0/html5shiv.js"></script>
+      <script src="https://oss.maxcdn.com/libs/respond.js/1.4.2/respond.min.js"></script>
+  %body
+    / Fixed navbar
+    .navbar.navbar-default.navbar-fixed-top{:role => "navigation"}
+      .container
+        .navbar-header
+          %button.navbar-toggle{"data-target" => ".navbar-collapse", "data-toggle" => "collapse", :type => "button"}
+            %span.sr-only Toggle navigation
+            %span.icon-bar
+            %span.icon-bar
+            %span.icon-bar
+          %a.navbar-brand{:href => "#"} cucumber_statistics
+        .navbar-collapse.collapse
+          -#%ul.nav.navbar-nav
+            %li.active
+              %a{:href => "#"} Home
+            %li
+              %a{:href => "#about"} About
+            %li
+              %a{:href => "#contact"} Contact
+            %li.dropdown
+              %a.dropdown-toggle{"data-toggle" => "dropdown", :href => "#"}
+                Dropdown
+                %b.caret
+              %ul.dropdown-menu
+                %li
+                  %a{:href => "#"} Action
+                %li
+                  %a{:href => "#"} Another action
+                %li
+                  %a{:href => "#"} Something else here
+                %li.divider
+                %li.dropdown-header Nav header
+                %li
+                  %a{:href => "#"} Separated link
+                %li
+                  %a{:href => "#"} One more separated link
+          %ul.nav.navbar-nav.navbar-right
+            -#%li
+            -#  %a{:href => "../navbar/"} Default
+            -#%li
+            -#  %a{:href => "../navbar-static-top/"} Static top
+            %li
+              %a{:href => "https://github.com/alienfast/cucumber_statistics", :target => 'other'} Github
+        / /.nav-collapse
+    .container
+      / Main component for a primary marketing message or call to action
+      .jumbotron
+        %h1 Cucumber scenario statistics
+        %p ... find those slow scenarios.
+
+      .alert.alert-info
+        %span
+          #{overall_statistics.feature_count} Features,
+          #{overall_statistics.scenario_count} Scenarios,
+          #{overall_statistics.step_count} Steps completed in #{format(overall_statistics.duration)}.
+        %span.text-muted.pull-right.small
+          Finished on #{format_date_time(overall_statistics.end_time)}
+
+      %table.table.table-bordered.table-striped.sortable
+        %thead
+          %tr
+            %th Scenario File
+            %th{"data-defaultsort" => "desc"} Time
+        %tbody
+
+          - scenario_statistics.all.each do |file_name, scenario_result|
+
+            %tr
+              = std_file_td file_name, scenario_result[:scenario_name]
+              = std_time_td scenario_result[:duration]
+    / /container
+    /
+      Bootstrap core JavaScript
+      \==================================================
+    / Placed at the end of the document so the pages load faster
+    %script{:src => "https://ajax.googleapis.com/ajax/libs/jquery/1.11.0/jquery.min.js"}
+    %script{:src => "http://netdna.bootstrapcdn.com/bootstrap/3.1.1/js/bootstrap.min.js"}
+    %script{:src => "https://raw.githack.com/drvic10k/bootstrap-sortable/master/Scripts/moment.min.js"}
+    %script{:src => "https://raw.githack.com/drvic10k/bootstrap-sortable/master/Scripts/bootstrap-sortable.js"}
+    :javascript
+      // Initialise on DOM ready
+      $(function() {
+          $.bootstrapSortable(true, 'reversed');
+      });

data/spec/cucumber_statistics/configuration_spec.rb

@@ -10,11 +10,11 @@ module CucumberStatistics
     it 'should auto create tmp_dir' do
 
       tmp_dir = Configuration.tmp_dir
-      Dir.exists?(tmp_dir).should == true
+      expect(Dir.exists?(tmp_dir)).to be_truthy
 
       Dir.delete tmp_dir
       tmp_dir = Configuration.tmp_dir
-      Dir.exists?(tmp_dir).should == true
+      expect(Dir.exists?(tmp_dir)).to be_truthy
     end
   end
 end

data/spec/cucumber_statistics/feature_statistics_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+module CucumberStatistics
+  describe FeatureStatistics do
+
+    subject(:feature_statistics) { FeatureStatistics.new }
+    subject(:overall_statistics) { OverallStatistics.new } #not sure if really need
+
+    describe 'record' do
+      it 'should create a record' do
+        record 'my feature', 50, 'features/admin_cancel_account.feature'
+
+        expect(feature_statistics.all.count).to eq 1
+        expect(feature_statistics.all['features/admin_cancel_account.feature'][:duration]).to eq 50
+      end
+
+      it 'should support multiple scenarios with the same name' do
+        record 'my feature', 50, 'admin_cancel_account.feature'
+        record 'my feature', 75, 'noise/features/super_admin/admin_cancel_account.feature'
+
+        expect(feature_statistics.all.count).to eq 2
+        expect(feature_statistics.all['admin_cancel_account.feature'][:duration]).to eq 50
+        expect(feature_statistics.all['features/super_admin/admin_cancel_account.feature'][:duration]).to eq 75
+      end
+
+      it 'chops off file path up to features/' do
+        record 'my feature', 50, '/User/some/file/path/that/doesnt/matter really/features/admin_cancel_account.feature'
+        expect(feature_statistics.all.keys.first).to eq 'features/admin_cancel_account.feature'
+      end
+
+      it 'uses the whole string if the filepath doesnt include the features folder' do
+        record 'my feature', 50, 'admin_cancel_account.feature'
+        expect(feature_statistics.all.keys.first).to eq 'admin_cancel_account.feature'
+      end
+    end
+
+    describe 'all' do
+      before(:each) do
+        record 'my feature', 24, 'admin_cancel_account.feature'
+        record 'my feature', 50, 'admin_edit_account.feature'
+      end
+
+      it 'should return all records' do
+        expect(feature_statistics.all.count).to eq 2
+        feature_statistics.all.each_with_index do |file_colon_line, data, index|
+          case index
+            when 1
+              expect(file_colon_line).to eq 'admin_cancel_account.feature'
+            when 2
+              expect(file_colon_line).to eq 'admin_edit_account.feature'
+          end
+        end
+      end
+    end
+
+    describe 'set operations' do
+      before(:each) do
+        record 'a', 25, 'admin_cancel_account.feature'
+        record 'c', 51, 'view_account.feature'
+        record 'b', 75, 'edit_account.feature'
+      end
+
+      describe 'sort_by_property' do
+        context 'should sort all records by any property' do
+          it { expect(feature_statistics.sort_by_property(:feature_name).first[0]).to eq 'admin_cancel_account.feature' }
+          it { expect(feature_statistics.sort_by_property(:feature_name).last[0]).to eq 'view_account.feature' }
+
+          it { expect(feature_statistics.sort_by_property(:duration).first[0]).to eq 'admin_cancel_account.feature' }
+          it { expect(feature_statistics.sort_by_property(:duration).last[0]).to eq 'edit_account.feature' }
+        end
+      end
+    end
+
+    def record(feature_name, duration, file)
+      feature_statistics.record feature_name, duration, file
+    end
+  end
+end

data/spec/cucumber_statistics/renderer_spec.rb

@@ -4,6 +4,8 @@ module CucumberStatistics
   describe Renderer do
 
     subject(:step_statistics) { StepStatistics.new }
+    subject(:scenario_statistics) { ScenarioStatistics.new }
+    subject(:feature_statistics) { FeatureStatistics.new }
     subject(:overall_statistics) { OverallStatistics.new }
 
     before(:each) do
@@ -23,21 +25,37 @@ module CucumberStatistics
       record 'my step 4', 4.21
       record 'my step 4', 4.21
 
+      record_scenario 'my scenario 1', 10.2, '/Users/kross/alienfast/acme/features/account management/admin_cancel_account.feature:8'
+      record_scenario 'my scenario 2', 30.2, '/Users/kross/alienfast/acme/features/account management/admin_cancel_account.feature:13'
+      record_scenario 'my scenario 3', 17.342, '/Users/kross/alienfast/acme/features/user experience/view.feature:2'
+      record_scenario 'my scenario 3', 3.2, '/Users/kross/alienfast/acme/features/user experience/view.feature:23'
+
       overall_statistics.end_time = Time.now
       step_statistics.calculate
     end
 
 
-    describe 'render_step_statistics' do
-      it 'should render content' do
+    describe 'render statistics' do
+      context 'should render content' do
+
+        it 'renders step results file' do
+          expect(File.exists?(Configuration.result_step_statistics)).to eql false
+          absolute_file_name = Renderer.render_step_statistics step_statistics, overall_statistics
+          expect(File.exists?(absolute_file_name)).to eql true
+        end
 
-        expect(File.exists?(Configuration.result_step_statistics)).to eql false
+        it 'renders scenario results file' do
+          expect(File.exists?(Configuration.result_scenario_statistics)).to eql false
+          absolute_file_name = Renderer.render_scenario_statistics scenario_statistics, overall_statistics
+          expect(File.exists?(absolute_file_name)).to eql true
+        end
 
-        absolute_file_name = Renderer.render_step_statistics step_statistics, overall_statistics
-        expect(File.exists?(absolute_file_name)).to eql true
+        it 'renders feature results file' do
+          expect(File.exists?(Configuration.result_feature_statistics)).to eql false
+          absolute_file_name = Renderer.render_feature_statistics feature_statistics, overall_statistics
+          expect(File.exists?(absolute_file_name)).to eql true
+        end
 
-        #file = File.open(absolute_file_name, 'rb')
-        #file_contents = file.read
       end
     end
 
@@ -46,5 +64,11 @@ module CucumberStatistics
       step_statistics.record step_name, duration, '/Users/kross/alienfast/acme/features/account management/admin_cancel_account.feature:8'
       overall_statistics.step_count_inc
     end
+
+    def record_scenario(scenario_name, duration, file_colon_line)
+      # fake a source for convenience
+      scenario_statistics.record scenario_name, duration, file_colon_line
+      overall_statistics.scenario_count_inc
+    end
   end
 end

data/spec/cucumber_statistics/scenario_statistics_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+module CucumberStatistics
+  describe ScenarioStatistics do
+
+    subject(:scenario_statistics) { ScenarioStatistics.new }
+    subject(:overall_statistics) { OverallStatistics.new } #not sure if really need
+
+    describe 'record' do
+      it 'should create a record' do
+        record 'my scenario', 50, 'features/admin_cancel_account.feature:8'
+
+        expect(scenario_statistics.all.count).to eq 1
+        expect(scenario_statistics.all['features/admin_cancel_account.feature:8'][:duration]).to eq 50
+      end
+
+      it 'should support multiple scenarios with the same name' do
+        record 'my scenario', 50, 'admin_cancel_account.feature:8'
+        record 'my scenario', 75, 'noise/features/super_admin/admin_cancel_account.feature:8'
+
+        expect(scenario_statistics.all.count).to eq 2
+        expect(scenario_statistics.all['admin_cancel_account.feature:8'][:duration]).to eq 50
+        expect(scenario_statistics.all['features/super_admin/admin_cancel_account.feature:8'][:duration]).to eq 75
+      end
+
+      it 'chops off file path up to features/' do
+        record 'my scenario', 50, '/User/some/file/path/that/doesnt/matter really/features/admin_cancel_account.feature:8'
+        expect(scenario_statistics.all.keys.first).to eq 'features/admin_cancel_account.feature:8'
+      end
+
+      it 'uses the whole string if the filepath doesnt include the features folder' do
+        record 'my scenario', 50, 'admin_cancel_account.feature:8'
+        expect(scenario_statistics.all.keys.first).to eq 'admin_cancel_account.feature:8'
+      end
+    end
+
+    describe 'all' do
+      before(:each) do
+        record 'my scenario 1', 24, 'admin_cancel_account.feature:8'
+        record 'my scenario 1', 50, 'admin_cancel_account.feature:18'
+        record 'my scenario 2', 75, 'admin_cancel_account.feature:38'
+      end
+
+      it 'should return all records' do
+        expect(scenario_statistics.all.count).to eq 3
+        scenario_statistics.all.each_with_index do |file_colon_line, data, index|
+          case index
+            when 1
+              expect(file_colon_line).to eq 'admin_cancel_account.feature:8'
+            when 2
+              expect(file_colon_line).to eq 'admin_cancel_account.feature:18'
+            when 3
+              expect(file_colon_line).to eq 'admin_cancel_account.feature:38'
+          end
+        end
+      end
+    end
+
+    describe 'set operations' do
+      before(:each) do
+        record 'my scenario 1', 25, 'admin_cancel_account.feature:8'
+        record 'my scenario 2', 50, 'admin_cancel_account.feature:13'
+        record 'my scenario 3', 49, 'admin_cancel_account.feature:15'
+        record 'my scenario 1', 51, 'view_account.feature:3'
+        record 'my scenario 2', 75, 'view_account.feature:6'
+        record 'my scenario 3', 10, 'view_account.feature:22'
+      end
+
+      describe 'sort_by_property' do
+        context 'should sort all records by any property' do
+          it { expect(scenario_statistics.sort_by_property(:scenario_name).first[0]).to eq 'admin_cancel_account.feature:8' }
+          it { expect(scenario_statistics.sort_by_property(:scenario_name).last[0]).to eq 'view_account.feature:22' }
+
+          it { expect(scenario_statistics.sort_by_property(:duration).first[0]).to eq 'view_account.feature:22' }
+          it { expect(scenario_statistics.sort_by_property(:duration).last[0]).to eq 'view_account.feature:6' }
+        end
+      end
+    end
+
+    def record(scenario_name, duration, file_colon_line)
+      # fake a source for convenience
+      scenario_statistics.record scenario_name, duration, file_colon_line
+      #overall_statistics.step_count_inc
+    end
+  end
+end

data/spec/cucumber_statistics/step_statistics_spec.rb

@@ -8,81 +8,81 @@ module CucumberStatistics
 
     describe 'record' do
       it 'should create a record' do
-        record "my step", 50
+        record 'my step', 50
 
-        step_statistics.all.count.should == 1
-        step_statistics.all['my step'][:instances].count.should == 1
-        step_statistics.all['my step'][:instances].first.should == 50
+        expect(step_statistics.all.count).to eq 1
+        expect(step_statistics.all['my step'][:instances].count).to eq 1
+        expect(step_statistics.all['my step'][:instances].first).to eq 50
       end
 
       it 'should support multiple instances of record' do
-        record "my step", 50
-        record "my step", 75
+        record 'my step', 50
+        record 'my step', 75
 
-        step_statistics.all.count.should == 1
-        step_statistics.all['my step'][:instances].count.should == 2
-        step_statistics.all['my step'][:instances].first.should == 50
-        step_statistics.all['my step'][:instances].last.should == 75
+        expect(step_statistics.all.count).to eq 1
+        expect(step_statistics.all['my step'][:instances].count).to eq 2
+        expect(step_statistics.all['my step'][:instances].first).to eq 50
+        expect(step_statistics.all['my step'][:instances].last).to eq 75
       end
     end
 
     describe 'calculate' do
       before(:each) do
-        record "my step", 25
-        record "my step", 50
-        record "my step", 75
+        record 'my step', 25
+        record 'my step', 50
+        record 'my step', 75
 
         step_statistics.calculate
       end
 
       it 'should calculate the total elapsed time' do
-        step_statistics.all["my step"][:total].should == 150
+        expect(step_statistics.all['my step'][:total]).to eq 150
       end
 
       it 'should calculate the number of count' do
-        step_statistics.all["my step"][:count].should == 3
+        expect(step_statistics.all['my step'][:count]).to eq 3
       end
 
       it 'should calculate the average time' do
-        step_statistics.all["my step"][:average].should == 50
+        expect(step_statistics.all['my step'][:average]).to eq 50
       end
 
       it 'should calculate the fastest step' do
-        step_statistics.all["my step"][:fastest].should == 25
+        expect(step_statistics.all['my step'][:fastest]).to eq 25
       end
-      
+
       it 'should calculate the slowest step' do
-        step_statistics.all["my step"][:slowest].should == 75
+        expect(step_statistics.all['my step'][:slowest]).to eq 75
       end
 
       it 'should calculate the variation' do
-        step_statistics.all["my step"][:variation].should == 50
+        expect(step_statistics.all['my step'][:variation]).to eq 50
       end
 
       it 'should calculate the standard deviation' do
-        step_statistics.all["my step"][:standard_deviation].should == 25
+        expect(step_statistics.all['my step'][:standard_deviation]).to eq 25
       end
 
       it 'should calculate the variance' do
-        step_statistics.all["my step"][:variance].should == 625
+        expect(step_statistics.all['my step'][:variance]).to eq 625
       end
     end
 
     describe 'all' do
       before(:each) do
-        record "my step 1", 24
-        record "my step 1", 50
-        record "my step 2", 75
+        record 'my step 1', 24
+        record 'my step 1', 50
+        record 'my step 2', 75
       end
 
       it 'should return all records' do
-        step_statistics.all.count.should == 2
+        expect(step_statistics.all.count).to eq 2
         step_statistics.all.each_with_index do |step_name, data, index|
           case index
-          when 1
-            step_name.should == "my_step 2"
-          when 2
-            step_name.should == "my_step 1"
+            when 1
+              expect(step_name).to eq 'my_step 2'
+            when 2
+              expect(step_name).to eq 'my_step 1'
           end
         end
       end
@@ -90,103 +90,103 @@ module CucumberStatistics
 
     describe 'set operations' do
       before(:each) do
-        record "my step 1", 25
-        record "my step 1", 50
-        record "my step 2", 49
-        record "my step 2", 51
-        record "my step 3", 75
-        record "my step 3", 10
+        record 'my step 1', 25
+        record 'my step 1', 50
+        record 'my step 2', 49
+        record 'my step 2', 51
+        record 'my step 3', 75
+        record 'my step 3', 10
 
         step_statistics.calculate
       end
 
       describe 'sort_by_property' do
         it 'should sort all records by any property' do
-          step_statistics.sort_by_property(:total).first.first.should == "my step 1"
-          step_statistics.sort_by_property(:total).last.first.should == "my step 2"
+          expect(step_statistics.sort_by_property(:total).first.first).to eq 'my step 1'
+          expect(step_statistics.sort_by_property(:total).last.first).to eq 'my step 2'
 
-          step_statistics.sort_by_property(:fastest).first.first.should == "my step 3"
-          step_statistics.sort_by_property(:fastest).last.first.should == "my step 2"
+          expect(step_statistics.sort_by_property(:fastest).first.first).to eq 'my step 3'
+          expect(step_statistics.sort_by_property(:fastest).last.first).to eq 'my step 2'
         end
       end
 
       describe 'highest_average' do
         it 'should return the record with the highest average' do
-          step_statistics.highest_average.first.should == "my step 2"
+          expect(step_statistics.highest_average.first).to eq 'my step 2'
         end
       end
 
       describe 'highest_elapsed_time' do
         it 'should return the record with the highest elapsed time' do
-          step_statistics.highest_total.first.should == "my step 2"
+          expect(step_statistics.highest_total.first).to eq 'my step 2'
         end
       end
 
       describe 'greatest_variation' do
         it 'should return the record with the greatest variation between slow and fast' do
-          step_statistics.highest_variation.first.should == "my step 3"
+          expect(step_statistics.highest_variation.first).to eq 'my step 3'
         end
       end
 
       describe 'step_part_of_total' do
         it 'should return the total times of each step from largest to smallest' do
-          step_statistics.step_part_of_total.should == [100.0, 85.0, 75.0]
+          expect(step_statistics.step_part_of_total).to eq [100.0, 85.0, 75.0]
         end
       end
 
       describe 'total_elapsed_time' do
         it 'should return the count of all steps' do
-          step_statistics.total_elapsed_time.should == 260
+          expect(step_statistics.total_elapsed_time).to eq 260
         end
       end
 
       describe 'average_times_plot_data' do
         it 'should return all the averages sorted by average amount descending' do
-          record "my step 1", 25
-          record "my step 1", 50
-          record "my step 2", 49
-          record "my step 2", 51
-          record "my step 3", 75
-          record "my step 3", 10
+          record 'my step 1', 25
+          record 'my step 1', 50
+          record 'my step 2', 49
+          record 'my step 2', 51
+          record 'my step 3', 75
+          record 'my step 3', 10
 
           step_statistics.calculate
-          
-          step_statistics.average_times_plot_data.should == [50.0, 42.5, 37.5]
+
+          expect(step_statistics.average_times_plot_data).to eq [50.0, 42.5, 37.5]
         end
       end
 
       describe 'total_times_plot_data' do
         it 'should return the total times of each step sorted by average amount descending' do
-          record "my step 1", 25
-          record "my step 1", 50
-          record "my step 3", 75
-          record "my step 3", 10
+          record 'my step 1', 25
+          record 'my step 1', 50
+          record 'my step 3', 75
+          record 'my step 3', 10
 
           step_statistics.calculate
 
-          step_statistics.total_times_plot_data.should == [100.0, 170, 150.0]
+          expect(step_statistics.total_times_plot_data).to eq [100.0, 170, 150.0]
         end
       end
     end
 
     describe 'sample_variance' do
       it 'should calculate the variance' do
-        step_statistics.sample_variance([1,2,3,4,5,6]).should be_within(0.1).of(3.5)
-        step_statistics.sample_variance([2,4,4,4,5,5,7,9]).should be_within(0.1).of(4.57)
-        step_statistics.sample_variance([25,50,75]).should be_within(0.1).of(625)
+        expect(step_statistics.sample_variance([1, 2, 3, 4, 5, 6])).to be_within(0.1).of(3.5)
+        expect(step_statistics.sample_variance([2, 4, 4, 4, 5, 5, 7, 9])).to be_within(0.1).of(4.57)
+        expect(step_statistics.sample_variance([25, 50, 75])).to be_within(0.1).of(625)
       end
     end
 
     describe 'standard_deviation' do
       it 'should calculate the standard deviation' do
-        sample_variance = step_statistics.sample_variance([1,2,3,4,5,6])
-        step_statistics.standard_deviation(sample_variance).should be_within(0.1).of(1.87)
-        
-        sample_variance = step_statistics.sample_variance([2,4,4,4,5,5,7,9])
-        step_statistics.standard_deviation(sample_variance).should be_within(0.1).of(2.13)
-
-        sample_variance = step_statistics.sample_variance([25,50,75])
-        step_statistics.standard_deviation(sample_variance).should be_within(0.1).of(25)
+        sample_variance = step_statistics.sample_variance([1, 2, 3, 4, 5, 6])
+        expect(step_statistics.standard_deviation(sample_variance)).to be_within(0.1).of(1.87)
+
+        sample_variance = step_statistics.sample_variance([2, 4, 4, 4, 5, 5, 7, 9])
+        expect(step_statistics.standard_deviation(sample_variance)).to be_within(0.1).of(2.13)
+
+        sample_variance = step_statistics.sample_variance([25, 50, 75])
+        expect(step_statistics.standard_deviation(sample_variance)).to be_within(0.1).of(25)
       end
     end
 

data/spec/cucumber_statistics/unused_steps_spec.rb

@@ -8,29 +8,29 @@ module CucumberStatistics
 
     describe 'record' do
       it 'should create a record' do
-        subject.record "my_step", "some code somewhere"
-        subject.all['my_step'].should == "some code somewhere"
+        subject.record 'my_step', 'some code somewhere'
+        expect(subject.all['my_step']).to eq 'some code somewhere'
       end
     end
 
     describe 'all' do
       it 'should return all records sorted alphabetically' do
-        subject.record "my_step 3", "some code somewhere 3"
-        subject.record "my_step 2", "some code somewhere 2"
-        subject.record "my_step 1", "some code somewhere 1"
+        subject.record 'my_step 3', 'some code somewhere 3'
+        subject.record 'my_step 2', 'some code somewhere 2'
+        subject.record 'my_step 1', 'some code somewhere 1'
 
-        subject.all.count.should == 3
+        expect(subject.all.count).to eq 3
         subject.all.each_with_index do |step_name, where, index|
           case index
-          when 1
-            step_name.should == "my_step 1"
-            where.should == "some code somewhere 1"
-          when 2
-            step_name.should == "my_step 2"
-            where.should == "some code somewhere 1"
-          when 3
-            step_name.should == "my_step 3"
-            where.should == "some code somewhere 1"
+            when 1
+              expect(step_name).to eq 'my_step 1'
+              expect(where).to eq 'some code somewhere 1'
+            when 2
+              expect(step_name).to eq 'my_step 2'
+              expect(where).to eq 'some code somewhere 1'
+            when 3
+              expect(step_name).to eq 'my_step 3'
+              expect(where).to eq 'some code somewhere 1'
           end
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cucumber_statistics
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.0
 platform: ruby
 authors:
 - Kevin Ross
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-15 00:00:00.000000000 Z
+date: 2016-08-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -108,8 +108,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: |2
-      Want to know what is slowing down your build?
+description: "    Want to know what is slowing down your build?\n"
 email:
 - kevin.ross@alienfast.com.com
 executables: []
@@ -128,19 +127,25 @@ files:
 - lib/cucumber_statistics.rb
 - lib/cucumber_statistics/autoload.rb
 - lib/cucumber_statistics/configuration.rb
+- lib/cucumber_statistics/feature_statistics.rb
 - lib/cucumber_statistics/formatter.rb
 - lib/cucumber_statistics/overall_statistics.rb
 - lib/cucumber_statistics/renderer.rb
 - lib/cucumber_statistics/renderer_helper.rb
+- lib/cucumber_statistics/scenario_statistics.rb
 - lib/cucumber_statistics/step_statistics.rb
 - lib/cucumber_statistics/unused_steps.rb
 - lib/cucumber_statistics/version.rb
+- lib/cucumber_statistics/view/feature_statistics.html.haml
+- lib/cucumber_statistics/view/scenario_statistics.html.haml
 - lib/cucumber_statistics/view/step_statistics.html
 - lib/cucumber_statistics/view/step_statistics.html.haml
 - notes.txt
 - spec/cucumber_statistics/configuration_spec.rb
+- spec/cucumber_statistics/feature_statistics_spec.rb
 - spec/cucumber_statistics/renderer_helper_spec.rb
 - spec/cucumber_statistics/renderer_spec.rb
+- spec/cucumber_statistics/scenario_statistics_spec.rb
 - spec/cucumber_statistics/step_statistics_spec.rb
 - spec/cucumber_statistics/unused_steps_spec.rb
 - spec/spec_helper.rb
@@ -164,15 +169,17 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: An cucumber formatter that will gather statistics and generate a single page
   showing step time metrics.
 test_files:
 - spec/cucumber_statistics/configuration_spec.rb
+- spec/cucumber_statistics/feature_statistics_spec.rb
 - spec/cucumber_statistics/renderer_helper_spec.rb
 - spec/cucumber_statistics/renderer_spec.rb
+- spec/cucumber_statistics/scenario_statistics_spec.rb
 - spec/cucumber_statistics/step_statistics_spec.rb
 - spec/cucumber_statistics/unused_steps_spec.rb
 - spec/spec_helper.rb