cucumber 2.99.0 → 3.0.0

data/spec/cucumber/filters/tag_limits/verifier_spec.rb

@@ -1,53 +1,54 @@
-require "cucumber/filters/tag_limits"
+# frozen_string_literal: true
+require 'cucumber/filters/tag_limits'
 
 describe Cucumber::Filters::TagLimits::Verifier do
-  describe "#verify!" do
+  describe '#verify!' do
     subject(:verifier) { Cucumber::Filters::TagLimits::Verifier.new(tag_limits) }
     let(:test_case_index) { double(:test_case_index) }
 
-    context "the tag counts exceed the tag limits" do
+    context 'the tag counts exceed the tag limits' do
       let(:tag_limits) do
         {
-          "@exceed_me" => 1
+          '@exceed_me' => 1
         }
       end
 
       let(:locations) do
         [
-          double(:location, to_s: "path/to/some.feature:3"),
-          double(:location, to_s: "path/to/some/other.feature:8"),
+          double(:location, to_s: 'path/to/some.feature:3'),
+          double(:location, to_s: 'path/to/some/other.feature:8')
         ]
       end
 
       before do
-        allow(test_case_index).to receive(:count_by_tag_name).with("@exceed_me") { 2 }
-        allow(test_case_index).to receive(:locations_of_tag_name).with("@exceed_me") { locations }
+        allow(test_case_index).to receive(:count_by_tag_name).with('@exceed_me') { 2 }
+        allow(test_case_index).to receive(:locations_of_tag_name).with('@exceed_me') { locations }
       end
 
-      it "raises a TagLimitExceeded error with the locations of the tags" do
+      it 'raises a TagLimitExceeded error with the locations of the tags' do
         expect {
           verifier.verify!(test_case_index)
         }.to raise_error(
           Cucumber::Filters::TagLimitExceededError,
           "@exceed_me occurred 2 times, but the limit was set to 1\n" +
           "  path/to/some.feature:3\n" +
-          "  path/to/some/other.feature:8"
+          '  path/to/some/other.feature:8'
         )
       end
     end
 
-    context "the tag counts do not exceed the tag limits" do
+    context 'the tag counts do not exceed the tag limits' do
       let(:tag_limits) do
         {
-          "@dont_exceed_me" => 2
+          '@dont_exceed_me' => 2
         }
       end
 
       before do
-        allow(test_case_index).to receive(:count_by_tag_name).with("@dont_exceed_me") { 1 }
+        allow(test_case_index).to receive(:count_by_tag_name).with('@dont_exceed_me') { 1 }
       end
 
-      it "does not raise an error" do
+      it 'does not raise an error' do
         expect {
           verifier.verify!(test_case_index)
         }.to_not raise_error

data/spec/cucumber/filters/tag_limits_spec.rb

@@ -1,4 +1,5 @@
-require "cucumber/filters/tag_limits"
+# frozen_string_literal: true
+require 'cucumber/filters/tag_limits'
 
 describe Cucumber::Filters::TagLimits do
   subject(:filter) { Cucumber::Filters::TagLimits.new(tag_limits, receiver) }
@@ -15,24 +16,24 @@ describe Cucumber::Filters::TagLimits do
     allow(Cucumber::Filters::TagLimits::TestCaseIndex).to receive(:new) { test_case_index }
   end
 
-  describe "#test_case" do
+  describe '#test_case' do
     before do
       allow(test_case_index).to receive(:add)
       allow(gated_receiver).to receive(:test_case)
     end
 
-    it "indexes the test case" do
+    it 'indexes the test case' do
       expect(test_case_index).to receive(:add).with(test_case)
       filter.test_case(test_case)
     end
 
-    it "adds the test case to the gated receiver" do
+    it 'adds the test case to the gated receiver' do
       expect(gated_receiver).to receive(:test_case).with(test_case)
       filter.test_case(test_case)
     end
   end
 
-  describe "#done" do
+  describe '#done' do
     let(:verifier) { double(:verifier) }
 
     before do
@@ -40,17 +41,17 @@ describe Cucumber::Filters::TagLimits do
       allow(gated_receiver).to receive(:done)
     end
 
-    it "verifies tag limits have not been exceeded" do
+    it 'verifies tag limits have not been exceeded' do
       expect(verifier).to receive(:verify!).with(test_case_index)
       filter.done
     end
 
-    context "the verifier verifies successfully" do
+    context 'the verifier verifies successfully' do
       before do
         allow(verifier).to receive(:verify!).with(test_case_index)
       end
 
-      it "calls done on the receiver gate" do
+      it 'calls done on the receiver gate' do
         expect(gated_receiver).to receive(:done)
         filter.done
       end

data/spec/cucumber/formatter/ansicolor_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 require 'cucumber/formatter/ansicolor'
 
@@ -6,26 +7,26 @@ module Cucumber
     describe ANSIColor do
       include ANSIColor
 
-      it "wraps passed_param with bold green and reset to green" do
-        expect(passed_param("foo")).to eq "\e[32m\e[1mfoo\e[0m\e[0m\e[32m"
+      it 'wraps passed_param with bold green and reset to green' do
+        expect(passed_param('foo')).to eq "\e[32m\e[1mfoo\e[0m\e[0m\e[32m"
       end
 
-      it "wraps passed in green" do
-        expect(passed("foo")).to eq "\e[32mfoo\e[0m"
+      it 'wraps passed in green' do
+        expect(passed('foo')).to eq "\e[32mfoo\e[0m"
       end
 
-      it "does not reset passed if there are no arguments" do
+      it 'does not reset passed if there are no arguments' do
         expect(passed).to eq "\e[32m"
       end
 
-      it "wraps comments in grey" do
-        expect(comment("foo")).to eq "\e[90mfoo\e[0m"
+      it 'wraps comments in grey' do
+        expect(comment('foo')).to eq "\e[90mfoo\e[0m"
       end
 
-      it "does not generate ansi codes when colors are disabled" do
+      it 'does not generate ansi codes when colors are disabled' do
         ::Cucumber::Term::ANSIColor.coloring = false
 
-        expect(passed("foo")).to eq "foo"
+        expect(passed('foo')).to eq 'foo'
       end
     end
   end

data/spec/cucumber/formatter/backtrace_filter_spec.rb

@@ -0,0 +1,32 @@
+require 'cucumber/formatter/backtrace_filter'
+
+module Cucumber
+  module Formatter
+    describe BacktraceFilter do
+      context '#exception' do
+        before do
+          trace = %w(a b
+                     _anything__/vendor/rails__anything_
+                     _anything__lib/cucumber__anything_
+                     _anything__bin/cucumber:__anything_
+                     _anything__lib/rspec__anything_
+                     _anything__gems/__anything_
+                     _anything__minitest__anything_
+                     _anything__test/unit__anything_
+                     _anything__Xgem/ruby__anything_
+                     _anything__lib/ruby/__anything_
+                     _anything__.rbenv/versions/2.3/bin/bundle__anything_)
+          @exception = Exception.new
+          @exception.set_backtrace(trace)
+        end
+
+        it 'filters unnecessary traces' do
+          BacktraceFilter.new(@exception).exception
+          expect(@exception.backtrace).to eql %w(a b)
+        end
+      end
+    end
+  end
+end
+
+

data/spec/cucumber/formatter/console_counts_spec.rb

@@ -0,0 +1,14 @@
+require 'cucumber/configuration'
+require 'cucumber/formatter/console_counts'
+
+module Cucumber
+  module Formatter
+    describe ConsoleCounts do
+      it 'works for zero' do
+        config = Configuration.new
+        counts = ConsoleCounts.new(config)
+        expect(counts.to_s).to eq "0 scenarios\n0 steps"
+      end
+    end
+  end
+end

data/spec/cucumber/formatter/duration_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 require 'cucumber/formatter/duration'
 
@@ -6,15 +7,15 @@ module Cucumber
     describe Duration do
       include Duration
 
-      it "formats ms" do
+      it 'formats ms' do
         expect(format_duration(0.002103)).to eq '0m0.002s'
       end
 
-      it "formats m" do
+      it 'formats m' do
         expect(format_duration(61.002503)).to eq'1m1.003s'
       end
 
-      it "formats h" do
+      it 'formats h' do
         expect(format_duration(3661.002503)).to eq '61m1.003s'
       end
     end

data/spec/cucumber/formatter/fail_fast_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'cucumber/formatter/fail_fast'
 require 'cucumber/core'
 require 'cucumber/core/gherkin/writer'
@@ -8,77 +9,76 @@ require 'cucumber'
 require 'support/standard_step_actions'
 
 module Cucumber::Formatter
-  describe FailFast do 
+  describe FailFast do
     include Cucumber::Core
     include Cucumber::Core::Gherkin::Writer
 
     let(:configuration) { Cucumber::Configuration.new }
     before { FailFast.new(configuration) }
-    let(:report) { EventBusReport.new(configuration) }
 
-    context 'failing scenario' do 
-      before(:each) do 
-        @gherkin = gherkin('foo.feature') do 
-          feature do 
-            scenario do 
+    context 'failing scenario' do
+      before(:each) do
+        @gherkin = gherkin('foo.feature') do
+          feature do
+            scenario do
               step 'failing'
             end
 
-            scenario do 
+            scenario do
               step 'failing'
             end
           end
         end
       end
 
-      after(:each) do 
+      after(:each) do
         Cucumber.wants_to_quit = false
       end
 
-      it 'sets Cucumber.wants_to_quit' do 
-        execute([@gherkin], report, [StandardStepActions.new])
+      it 'sets Cucumber.wants_to_quit' do
+        execute [@gherkin], [StandardStepActions.new], configuration.event_bus
         expect(Cucumber.wants_to_quit).to be true
       end
     end
 
-    context 'passing scenario' do 
-      before(:each) do 
-        @gherkin = gherkin('foo.feature') do 
-          feature do 
-            scenario do 
+    context 'passing scenario' do
+      before(:each) do
+        @gherkin = gherkin('foo.feature') do
+          feature do
+            scenario do
               step 'passing'
             end
           end
         end
       end
 
-      it 'doesn\'t set Cucumber.wants_to_quit' do 
-        execute([@gherkin], report, [StandardStepActions.new])
+      it 'doesn\'t set Cucumber.wants_to_quit' do
+        execute [@gherkin], [StandardStepActions.new], configuration.event_bus
         expect(Cucumber.wants_to_quit).to be_falsey
       end
     end
 
     context 'undefined scenario' do
-      before(:each) do 
-        @gherkin = gherkin('foo.feature') do 
-          feature do 
-            scenario do 
+      before(:each) do
+        @gherkin = gherkin('foo.feature') do
+          feature do
+            scenario do
               step 'undefined'
             end
           end
         end
       end
 
-      it 'doesn\'t set Cucumber.wants_to_quit' do 
-        execute([@gherkin], report, [StandardStepActions.new])
+      it 'doesn\'t set Cucumber.wants_to_quit' do
+        execute [@gherkin], [StandardStepActions.new], configuration.event_bus
         expect(Cucumber.wants_to_quit).to be_falsey
       end
 
       context 'in strict mode' do
-        let(:configuration) { Cucumber::Configuration.new strict: true }
+        let(:configuration) { Cucumber::Configuration.new strict: Cucumber::Core::Test::Result::StrictConfiguration.new([:undefined]) }
 
-        it 'sets Cucumber.wants_to_quit' do 
-          execute([@gherkin], report, [StandardStepActions.new])
+        it 'sets Cucumber.wants_to_quit' do
+          execute [@gherkin], [StandardStepActions.new], configuration.event_bus
           expect(Cucumber.wants_to_quit).to be_truthy
         end
       end