spinach 0.1.2.1 → 0.1.3

data/features/reporting/display_run_summary.feature

@@ -0,0 +1,9 @@
+Feature: Display run summary
+  As a developer
+  I want spinach to display a summary of steps statuses
+  So I can easyly know general features status
+
+  Scenario: Display run summary at the end of features run
+    Given I have a feature that has some successful, undefined, failed and error steps
+    When I run it
+    Then I should see a summary with steps status information

data/features/steps/reporting/display_run_summary.rb

@@ -0,0 +1,62 @@
+require 'aruba/api'
+
+Feature "Display run summary" do
+  include Integration::SpinachRunner
+
+  Given "I have a feature that has some successful, undefined, failed and error steps" do
+    write_file('features/test_feature.feature',
+     'Feature: A test feature
+
+      Scenario: Undefined scenario
+        Given I am a fool
+        When I jump from Codegrams roof
+        Then I must be pwned by floor
+
+      # Scenario: Failed scenario
+      #   Given I love risk
+      #   When I jump from Codegrams roof
+      #   Then my parachute must open
+      #   Then I must not pwned by floor
+
+      # Scenario: Error scenario
+      #   Given I am not a fool
+      #   When I go downstairs
+      #   Then I must succeed
+     ')
+    write_file('features/steps/test_feature.rb',
+    'Feature "A test feature" do
+      Given "I am a fool" do
+      end
+      When "I jump from Codegrams roof" do
+      end
+      # Given "I love risk" do
+      # end
+      # And "my parachute must open" do
+      #   false
+      # end
+      # Given I am a fool
+      #   djkadlaksaldkajd
+      # end
+      # Given I am not a fool
+      # end
+      # When I go downstairs
+      # end
+      # Then I must succeed
+      #   true
+      # end
+     end')
+    @feature = "features/test_feature.feature"
+  end
+
+  When "I run it" do
+    run_feature @feature
+  end
+
+  Then "I should see a summary with steps status information" do
+    all_stdout.must_match(
+      /Summary:.*2.*Successful.*1.*Undefined.*0.*Failed.*0.*Error/
+    )
+  end
+end
+
+

data/features/support/spinach_runner.rb

@@ -8,10 +8,10 @@ module Integration
       base.class_eval do
         before_scenario do
           in_current_dir do
-            FileUtils.rmdir('features')
+            run "rm -fR features"
           end
         end
-        before do
+        before_scenario do
           @aruba_timeout_seconds = 6
         end
       end

data/lib/spinach/dsl.rb

@@ -15,8 +15,6 @@ module Spinach
         extend ClassMethods
         include Hooks
 
-        define_hook :before
-        define_hook :after
         define_hook :before_scenario
         define_hook :after_scenario
         define_hook :before_step

data/lib/spinach/reporter.rb

@@ -11,6 +11,7 @@ module Spinach
       @errors = []
       @options = options
       @undefined_features = []
+      @successful_steps = []
       @undefined_steps = []
       @failed_steps = []
       @error_steps = []
@@ -22,7 +23,7 @@ module Spinach
 
     attr_accessor :current_feature, :current_scenario
 
-    attr_reader :undefined_steps, :failed_steps, :error_steps, :undefined_features
+    attr_reader :undefined_steps, :failed_steps, :error_steps, :undefined_features, :successful_steps
 
     def bind
       runner.after_run method(:after_run)

data/lib/spinach/reporter/stdout.rb

@@ -11,6 +11,7 @@ module Spinach
 
       # The last scenario error
       attr_accessor :scenario_error
+      attr_accessor :scenario
 
       # Initialitzes the runner
       #
@@ -65,6 +66,8 @@ module Spinach
       #
       def on_successful_step(step, step_location)
         output_step('✔', step, :green, step_location)
+        self.scenario = [current_feature, current_scenario, step]
+        successful_steps << scenario
       end
 
       # Adds a failing step to the output buffer.
@@ -167,12 +170,25 @@ module Spinach
       end
 
       # It prints the error summary if the run has failed
+      # It always print feature success summary
       #
       # @param [True,False] success
       #   whether the run has succeed or not
       #
       def after_run(success)
         error_summary unless success
+        out.puts ""
+        run_summary
+      end
+
+      # Prints the feature success summaryfor ths run.
+      #
+      def run_summary
+        successful_summary = "(".colorize(:green)+successful_steps.length.to_s.colorize(:light_green)+") Successful".colorize(:green)
+        undefined_summary = "(".colorize(:yellow)+undefined_steps.length.to_s.colorize(:light_yellow)+") Undefined".colorize(:yellow)
+        failed_summary = "(".colorize(:red)+failed_steps.length.to_s.colorize(:light_red)+") Failed".colorize(:red)
+        error_summary = "(".colorize(:red)+error_steps.length.to_s.colorize(:light_red)+") Error".colorize(:red)
+        out.puts "Steps Summary: #{successful_summary}, #{undefined_summary}, #{failed_summary}, #{error_summary}\n\n"
       end
 
       # Prints the errors for ths run.

data/lib/spinach/runner/feature.rb

@@ -31,14 +31,6 @@ module Spinach
       #
       attr_reader :filename
 
-      # @return [Feature]
-      #   The feature object used to run this scenario.
-      #
-      # @api public
-      def feature
-        @feature ||= Spinach.find_feature(feature_name).new
-      end
-
       # @return [Hash]
       #   The parsed data for this feature.
       #
@@ -71,15 +63,13 @@ module Spinach
       # @api public
       def run
         run_hook :before_run, data
-        feature.run_hook :before, data
 
         scenarios.each do |scenario|
           if !@scenario_line || scenario['line'].to_s == @scenario_line
-            @success = Scenario.new(feature_name, feature, scenario).run
+            @success = Scenario.new(feature_name, scenario).run
           end
         end
 
-        feature.run_hook :after, data
         run_hook :after_run, data
 
       rescue Spinach::FeatureStepsNotFoundException => e

data/lib/spinach/runner/scenario.rb

@@ -24,16 +24,23 @@ module Spinach
       #   The parsed feature data.
       #
       # @api public
-      def initialize(feature_name, feature, data)
+      def initialize(feature_name, data)
         @feature_name = feature_name
         @data = data
-        @feature = feature
       end
 
       def steps
         @steps ||= data['steps']
       end
 
+      # @return [Feature]
+      #   The feature object used to run this scenario.
+      #
+      # @api public
+      def feature
+        @feature ||= Spinach.find_feature(feature_name).new
+      end
+
       # Runs this scenario
       # @return [True, False]
       #   true if this scenario succeeded, false if not

data/lib/spinach/version.rb

@@ -1,4 +1,4 @@
 module Spinach
   # Spinach version.
-  VERSION = "0.1.2.1"
+  VERSION = "0.1.3"
 end

data/test/spinach/capybara_test.rb

@@ -48,7 +48,8 @@ describe Spinach::FeatureSteps::Capybara do
           Then Goodbye
     ').at_least_once
 
-    @feature_runner.stubs(feature: @feature).at_least_once
+    Spinach::Runner::Scenario.any_instance.stubs(feature: @feature)
+
     Capybara.current_session.expects(:reset!).twice
 
     @feature_runner.run

data/test/spinach/reporter/stdout_test.rb

@@ -207,16 +207,20 @@ describe Spinach::Reporter::Stdout do
 
   describe '#after_run' do
     describe 'when the run has succeed' do
-      it 'does nothing' do
-        @reporter.after_run(true).must_equal nil
+      it 'display run summary' do
+        @reporter.expects(:error_summary).never
+        @reporter.expects(:run_summary)
+
+        @reporter.after_run(true)
       end
     end
 
-    describe 'when the run has succeed' do
-      it 'does nothing' do
-        @reporter.stubs(:error_summary).returns('Error summary')
+    describe 'when the run has failed' do
+      it 'display run and error summaries' do
+        @reporter.expects(:error_summary)
+        @reporter.expects(:run_summary)
 
-        @reporter.after_run(false).must_equal 'Error summary'
+        @reporter.after_run(false)
       end
     end
   end

data/test/spinach/runner/feature_test.rb

@@ -17,16 +17,6 @@ describe Spinach::Runner::Feature do
     end
   end
 
-  describe '#feature' do
-    it 'finds the feature given a feature name' do
-      @feature = stub_everything
-      feature.stubs(feature_name: 'A cool feature')
-
-      Spinach.expects(:find_feature).with('A cool feature').returns(@feature)
-      feature.feature
-    end
-  end
-
   describe '#data' do
     it 'returns the parsed data' do
       parsed_data = {name: 'A cool feature'}
@@ -56,14 +46,12 @@ describe Spinach::Runner::Feature do
 
     it 'calls the steps as expected' do
       seq = sequence('feature')
-      feature.feature.expects(:run_hook).with(:before, kind_of(Hash))
       3.times do
         Spinach::Runner::Scenario.
           expects(:new).
           returns(stub_everything).
           in_sequence(seq)
       end
-      feature.feature.expects(:run_hook).with(:after, kind_of(Hash))
       feature.run
     end
 
@@ -90,7 +78,7 @@ describe Spinach::Runner::Feature do
       })
       @feature.stubs(feature: stub_everything)
 
-      Spinach::Runner::Scenario.expects(:new).with(anything, anything, @feature.scenarios[1], anything).once.returns(stub_everything)
+      Spinach::Runner::Scenario.expects(:new).with(anything, @feature.scenarios[1], anything).once.returns(stub_everything)
       @feature.run
     end
 
@@ -98,7 +86,7 @@ describe Spinach::Runner::Feature do
       feature = Spinach::Runner::Feature.new(filename)
       data = mock
       exception = Spinach::FeatureStepsNotFoundException.new([anything, anything])
-      feature.stubs(:feature).raises(exception)
+      feature.stubs(:scenarios).raises(exception)
       feature.stubs(:data).returns(data)
       not_found_called = false
       feature.class.when_not_found do |data, exception|

data/test/spinach/runner/scenario_test.rb

@@ -14,7 +14,11 @@ describe Spinach::Runner::Scenario do
 
   let(:feature) { stub_everything }
   let(:feature_name) { 'My feature' }
-  let(:scenario) { Spinach::Runner::Scenario.new(feature_name, feature, data) }
+  let(:scenario) { 
+    scenario = Spinach::Runner::Scenario.new(feature_name, data) 
+    scenario.stubs(feature: feature)
+    scenario
+  }
 
   describe '#initialize' do
     it 'lists all the steps' do
@@ -26,6 +30,16 @@ describe Spinach::Runner::Scenario do
     end
   end
 
+  describe '#feature' do
+    it 'finds the feature given a feature name' do
+      scenario.unstub(:feature)
+      @feature = stub_everything
+      scenario.stubs(feature_name: 'A cool feature')
+      Spinach.expects(:find_feature).with('A cool feature').returns(@feature)
+      scenario.feature
+    end
+  end
+
   describe '#run' do
     it 'calls the appropiate feature steps' do
       feature.expects(:execute_step).with('I herd you like steps')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spinach
 version: !ruby/object:Gem::Version
-  version: 0.1.2.1
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
@@ -16,7 +16,7 @@ date: 2011-10-10 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gherkin
-  requirement: &2164889160 !ruby/object:Gem::Requirement
+  requirement: &2158625180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -24,10 +24,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2164889160
+  version_requirements: *2158625180
 - !ruby/object:Gem::Dependency
   name: minitest
-  requirement: &2164888740 !ruby/object:Gem::Requirement
+  requirement: &2158624760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -35,10 +35,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2164888740
+  version_requirements: *2158624760
 - !ruby/object:Gem::Dependency
   name: colorize
-  requirement: &2164888320 !ruby/object:Gem::Requirement
+  requirement: &2158624340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -46,10 +46,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2164888320
+  version_requirements: *2158624340
 - !ruby/object:Gem::Dependency
   name: hooks
-  requirement: &2164887900 !ruby/object:Gem::Requirement
+  requirement: &2158623920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -57,10 +57,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2164887900
+  version_requirements: *2158623920
 - !ruby/object:Gem::Dependency
   name: purdytest
-  requirement: &2164887480 !ruby/object:Gem::Requirement
+  requirement: &2158623500 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -68,10 +68,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164887480
+  version_requirements: *2158623500
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2164887060 !ruby/object:Gem::Requirement
+  requirement: &2158623080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -79,10 +79,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164887060
+  version_requirements: *2158623080
 - !ruby/object:Gem::Dependency
   name: mocha
-  requirement: &2164886640 !ruby/object:Gem::Requirement
+  requirement: &2158622660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -90,10 +90,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164886640
+  version_requirements: *2158622660
 - !ruby/object:Gem::Dependency
   name: sinatra
-  requirement: &2164886220 !ruby/object:Gem::Requirement
+  requirement: &2158622240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -101,10 +101,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164886220
+  version_requirements: *2158622240
 - !ruby/object:Gem::Dependency
   name: capybara
-  requirement: &2164885800 !ruby/object:Gem::Requirement
+  requirement: &2158621820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -112,10 +112,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164885800
+  version_requirements: *2158621820
 - !ruby/object:Gem::Dependency
   name: aruba
-  requirement: &2164885380 !ruby/object:Gem::Requirement
+  requirement: &2158621400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -123,10 +123,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164885380
+  version_requirements: *2158621400
 - !ruby/object:Gem::Dependency
   name: pry
-  requirement: &2164884960 !ruby/object:Gem::Requirement
+  requirement: &2158620960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -134,10 +134,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164884960
+  version_requirements: *2158620960
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &2164884540 !ruby/object:Gem::Requirement
+  requirement: &2158620540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -145,10 +145,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164884540
+  version_requirements: *2158620540
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2164884120 !ruby/object:Gem::Requirement
+  requirement: &2158620120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -156,7 +156,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2164884120
+  version_requirements: *2158620120
 description: Spinach is a BDD framework on top of gherkin
 email:
 - info@codegram.com
@@ -179,12 +179,14 @@ files:
 - bin/spinach
 - features/exit_status.feature
 - features/feature_name_guessing.feature
+- features/reporting/display_run_summary.feature
 - features/reporting/error_reporting.feature
 - features/reporting/show_step_source_location.feature
 - features/reporting/undefined_feature_reporting.feature
 - features/rspec_compatibility.feature
 - features/steps/exit_status.rb
 - features/steps/feature_name_guessing.rb
+- features/steps/reporting/display_run_summary.rb
 - features/steps/reporting/error_reporting.rb
 - features/steps/reporting/show_step_source_location.rb
 - features/steps/reporting/undefined_feature_reporting.rb
@@ -241,7 +243,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1175741634638937209
+      hash: 317238591053879963
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -250,7 +252,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1175741634638937209
+      hash: 317238591053879963
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.6
@@ -260,12 +262,14 @@ summary: Spinach is a BDD framework on top of gherkin
 test_files:
 - features/exit_status.feature
 - features/feature_name_guessing.feature
+- features/reporting/display_run_summary.feature
 - features/reporting/error_reporting.feature
 - features/reporting/show_step_source_location.feature
 - features/reporting/undefined_feature_reporting.feature
 - features/rspec_compatibility.feature
 - features/steps/exit_status.rb
 - features/steps/feature_name_guessing.rb
+- features/steps/reporting/display_run_summary.rb
 - features/steps/reporting/error_reporting.rb
 - features/steps/reporting/show_step_source_location.rb
 - features/steps/reporting/undefined_feature_reporting.rb