square-cucumber 0.3.12.2 → 0.3.93.1

data/spec/cucumber/cli/main_spec.rb

@@ -13,13 +13,13 @@ module Cucumber
         @out = StringIO.new
         @err = StringIO.new
         Kernel.stub!(:exit).and_return(nil)
+        File.stub!(:exist?).and_return(false) # When Configuration checks for cucumber.yml
       end
 
       describe "verbose mode" do
 
         before(:each) do
           @empty_feature = Ast::Feature.new(nil, Ast::Comment.new(''), Ast::Tags.new(2, []), "Feature", [])
-          Dir.stub!(:[])
         end
 
         it "should show ruby files required" do
@@ -94,6 +94,19 @@ module Cucumber
         end
       end
 
+    [ProfilesNotDefinedError, YmlLoadError, ProfileNotFound].each do |exception_klass|
+
+      it "rescues #{exception_klass}, prints the message to the error stream and returns true" do
+        Configuration.stub!(:new).and_return(configuration = mock('configuration'))
+        configuration.stub!(:parse!).and_raise(exception_klass.new("error message"))
+
+        main = Main.new('', out = StringIO.new, error = StringIO.new)
+        main.execute!(Object.new.extend(StepMother)).should be_true
+        error.string.should == "error message\n"
+      end
+    end
+
+
       context "--drb" do
         before(:each) do
           @configuration = mock('Configuration', :drb? => true, :null_object => true)
@@ -129,10 +142,6 @@ module Cucumber
             @err.string.should include("WARNING: error message. Running features locally:")
           end
 
-          it "reparses the configuration since the --drb flag causes the initial parsing to short circuit" do
-            @configuration.should_receive(:parse!).exactly(:twice)
-            @cli.execute!(@step_mother)
-          end
         end
       end
     end

data/spec/cucumber/cli/options_spec.rb

@@ -0,0 +1,311 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+require 'yaml'
+
+module Cucumber
+module Cli
+  describe Options do
+
+    def given_cucumber_yml_defined_as(hash_or_string)
+      File.stub!(:exist?).and_return(true)
+      cucumber_yml = hash_or_string.is_a?(Hash) ? hash_or_string.to_yaml : hash_or_string
+      IO.stub!(:read).with('cucumber.yml').and_return(cucumber_yml)
+    end
+
+    before(:each) do
+      File.stub!(:exist?).and_return(false) # Meaning, no cucumber.yml exists
+      Kernel.stub!(:exit).and_return(nil)
+    end
+
+    def output_stream
+      @output_stream ||= StringIO.new
+    end
+
+    def error_stream
+      @error_stream ||= StringIO.new
+    end
+
+    def options
+      @options ||= Options.new(output_stream, error_stream)
+    end
+
+    def prepare_args(args)
+      args.is_a?(Array) ? args : args.split(' ')
+    end
+
+    describe 'parsing' do
+
+      def when_parsing(args)
+        yield
+        options.parse!(prepare_args(args))
+      end
+
+      def after_parsing(args)
+        options.parse!(prepare_args(args))
+        yield
+      end
+
+      context '-r or --require' do
+        it "collects all specified files into an array" do
+          after_parsing ('--require some_file.rb -r another_file.rb') do
+            options[:require].should == ['some_file.rb', 'another_file.rb']
+          end
+        end
+      end
+
+      context '-l LANG or --language LANG' do
+        context "with LANG specified as 'help'" do
+          it "lists all known langues" do
+            when_parsing '-l help' do
+              LanguageHelpFormatter.should_receive(:list_languages).with(output_stream)
+            end
+          end
+          it "exits the program" do
+            when_parsing('--language help') { Kernel.should_receive(:exit) }
+          end
+        end
+        it "sets the langauge" do
+          after_parsing('-l en') { options[:lang].should == 'en' }
+        end
+      end
+
+      context '-f FORMAT or --format FORMAT' do
+        it "defaults the output for the formatter to the output stream (STDOUT)" do
+          after_parsing('-f pretty') { options[:formats].should == [['pretty', output_stream]] }
+        end
+      end
+
+      context '-o [FILE|DIR] or --out [FILE|DIR]' do
+        it "defaults the formatter to 'pretty' when not specified earlier" do
+          after_parsing('-o file.txt') { options[:formats].should == [['pretty', 'file.txt']] }
+        end
+        it "sets the output for the formatter defined immediatly before it" do
+          after_parsing('-f profile --out file.txt -f pretty -o file2.txt') do 
+            options[:formats].should == [['profile', 'file.txt'], ['pretty', 'file2.txt']]
+          end
+        end
+      end
+
+      context '-t TAGS --tags TAGS' do
+        it "removes the @ prefix" do
+          after_parsing('-t @foo,bar') { options[:include_tags].should == ['foo','bar'] }
+        end
+        it "designates tags prefixed with ~ as tags to be excluded" do
+          after_parsing('--tags ~@foo,bar') { options[:exclude_tags].should == ['foo'] }
+        end
+      end
+
+      context '-n NAME or --name NAME' do
+        it "stores the provided names as regular expressions" do
+          after_parsing('-n foo --name bar') { options[:name_regexps].should == [/foo/,/bar/] }
+        end
+      end
+
+      context '-e PATTERN or --exclude PATTERN' do
+        it "stores the provided exclusions as regular expressions" do
+          after_parsing('-e foo --exclude bar') { options[:excludes].should == [/foo/,/bar/] }
+        end
+      end
+
+      context '-p PROFILE or --profile PROFILE' do
+
+        it "notifies the user that an individual profile is being used" do
+          given_cucumber_yml_defined_as({'foo' => [1,2,3]})
+          options.parse!(%w{--profile foo})
+          output_stream.string.should =~ /Using the foo profile...\n/
+        end
+
+        it "notifies the user when multiple profiles are being used" do
+          given_cucumber_yml_defined_as({'foo' => [1,2,3], 'bar' => ['v'], 'dog' => ['v']})
+          options.parse!(%w{--profile foo --profile bar --profile dog})
+          output_stream.string.should =~ /Using the foo, bar and dog profiles...\n/
+        end
+
+        it "notifies the user of all profiles being used, even when they are nested" do
+          given_cucumber_yml_defined_as('foo' => '-p bar', 'bar' => 'features')
+          after_parsing('-p foo') do
+            output_stream.string.should =~ /Using the foo and bar profiles.../
+          end
+        end
+
+        it "uses the default profile passed in during initialization if none are specified by the user" do
+          given_cucumber_yml_defined_as({'default' => '--require some_file'})
+
+          options = Options.new(output_stream, error_stream, :default_profile => 'default')
+          options.parse!(%w{--format progress})
+          options[:require].should include('some_file')
+        end
+
+        it "merges all uniq values from both cmd line and the profile" do
+          given_cucumber_yml_defined_as('foo' => %w[--verbose])
+          options.parse!(%w[--wip --profile foo])
+          options[:wip].should be_true
+          options[:verbose].should be_true
+        end
+
+        it "gives precendene to the origianl options' paths" do
+          given_cucumber_yml_defined_as('foo' => %w[features])
+          options.parse!(%w[my.feature -p foo])
+          options[:paths].should == %w[my.feature]
+        end
+
+        it "combines the require files of both" do
+          given_cucumber_yml_defined_as('bar' => %w[--require features -r dog.rb])
+          options.parse!(%w[--require foo.rb -p bar])
+          options[:require].should == %w[foo.rb features dog.rb]
+        end
+
+        it "combines the include_tags of both" do
+          given_cucumber_yml_defined_as('baz' => %w[-t bar])
+          options.parse!(%w[--tags foo -p baz])
+          options[:include_tags].should == %w[foo bar]
+        end
+
+        it "combines the exclude_tags of both" do
+          given_cucumber_yml_defined_as('baz' => %w[-t ~bar])
+          options.parse!(%w[--tags ~foo -p baz])
+          options[:exclude_tags].should == %w[foo bar]
+        end
+
+        it "only takes the paths from the original options, and disgregards the profiles" do
+          given_cucumber_yml_defined_as('baz' => %w[features])
+          options.parse!(%w[my.feature -p baz])
+          options[:paths].should == ['my.feature']
+        end
+
+        it "uses the paths from the profile when none are specified originally" do
+          given_cucumber_yml_defined_as('baz' => %w[some.feature])
+          options.parse!(%w[-p baz])
+          options[:paths].should == ['some.feature']
+        end
+
+        it "combines environment variables from the profile but gives precendene to cmd line args" do
+          given_cucumber_yml_defined_as('baz' => %w[FOO=bar CHEESE=swiss])
+          options.parse!(%w[-p baz CHEESE=cheddar BAR=foo])
+          options[:env_vars].should == {'BAR' => 'foo', 'FOO' => 'bar', 'CHEESE' => 'cheddar'}
+        end
+
+        it "disregards STDOUT formatter defined in profile when another is passed in (via cmd line)" do
+          given_cucumber_yml_defined_as({'foo' => %w[--format pretty]})
+          options.parse!(%w{--format progress --profile foo})
+          options[:formats].should == [['progress', output_stream]]
+        end
+
+        it "includes any non-STDOUT formatters from the profile" do
+          given_cucumber_yml_defined_as({'html' => %w[--format html -o features.html]})
+          options.parse!(%w{--format progress --profile html})
+          options[:formats].should == [['progress', output_stream], ['html', 'features.html']]
+        end
+
+        it "assumes all of the formatters defined in the profile when none are specified on cmd line" do
+          given_cucumber_yml_defined_as({'html' => %w[--format progress --format html -o features.html]})
+          options.parse!(%w{--profile html})
+          options[:formats].should == [['progress', output_stream], ['html', 'features.html']]
+        end
+
+        it "respects --quiet when defined in the profile" do
+          given_cucumber_yml_defined_as('foo' => '-q')
+          options.parse!(%w[-p foo])
+          options[:snippets].should be_false
+          options[:source].should be_false
+        end
+
+      end
+
+      context '-P or --no-profile' do
+
+        it "disables profiles" do
+          given_cucumber_yml_defined_as({'default' => '-v --require file_specified_in_default_profile.rb'})
+
+          after_parsing("-P --require some_file.rb") do
+            options[:require].should == ['some_file.rb']
+          end
+        end
+
+        it "notifies the user that the profiles are being disabled" do
+          given_cucumber_yml_defined_as({'default' => '-v'})
+
+          after_parsing("--no-profile --require some_file.rb") do
+            output_stream.string.should =~ /Disabling profiles.../
+          end
+        end
+
+      end
+
+      context '-b or --backtrace' do
+        it "turns on cucumber's full backtrace" do
+          when_parsing("-b") do
+            Exception.should_receive(:cucumber_full_backtrace=).with(true)
+          end
+        end
+      end
+
+      context '--version' do
+        it "displays Cucumber's version" do
+          after_parsing('--version') do
+            output_stream.string.should =~ /#{VERSION::STRING}/
+          end
+        end
+        it "exits the program" do
+          when_parsing('--version') { Kernel.should_receive(:exit) }
+        end
+      end
+
+      context 'environment variables (i.e. MODE=webrat)' do
+        it "places all of the environment variables into a hash" do
+          after_parsing('MODE=webrat FOO=bar') do
+            options[:env_vars].should == {'MODE' => 'webrat', 'FOO' => 'bar'}
+          end
+        end
+      end
+
+      it "assigns any extra arguments as paths to features" do
+        after_parsing('-f pretty my_feature.feature my_other_features') do
+          options[:paths].should == ['my_feature.feature', 'my_other_features']
+        end
+      end
+
+      it "does not mistake environment variables as feature paths" do
+        after_parsing('my_feature.feature FOO=bar') do
+          options[:paths].should == ['my_feature.feature']
+        end
+      end
+    end
+
+    describe '#expanded_args_without_drb' do
+      it "returns the orginal args in additon to the args from any profiles" do
+        given_cucumber_yml_defined_as('foo' => '-v', 
+                                      'bar' => '--wip -p baz',
+                                      'baz' => '-r some_file.rb')
+        options.parse!(%w[features -p foo --profile bar])
+
+        options.expanded_args_without_drb.should == %w[features -v --wip -r some_file.rb]
+      end
+
+      it "removes the --drb flag so that the args can be safely passed to the drb server" do
+        given_cucumber_yml_defined_as('default' => 'features -f pretty --drb')
+        options.parse!(%w[--profile default])
+
+        options.expanded_args_without_drb.should == %w[features -f pretty]
+      end
+
+      it "contains the environment variables" do
+        options.parse!(%w[features FOO=bar])
+        options.expanded_args_without_drb.should == %w[features FOO=bar]
+      end
+
+      it "ignores the paths from the profiles if one was specified on the command line" do
+        given_cucumber_yml_defined_as('foo' => 'features --drb')
+        options.parse!(%w[some_feature.feature -p foo])
+        options.expanded_args_without_drb.should == %w[some_feature.feature]
+      end
+
+
+
+
+    end
+
+  end
+
+end
+end
+

data/spec/cucumber/cli/profile_loader_spec.rb

@@ -0,0 +1,10 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+require 'yaml'
+
+module Cucumber
+module Cli
+  describe ProfileLoader do
+    it "should have all of its specs migrated from configuration_spec"
+  end
+end
+end

data/spec/cucumber/core_ext/proc_spec.rb

@@ -36,7 +36,7 @@ describe Proc do
   end
 
   if Cucumber::RUBY_1_9
-    it "should allow varargs" do
+    it "should allow varargs (expecting 0+)" do
       lambda {
         Object.new.cucumber_instance_exec(true, 'foo', 1) do |*args|
         end
@@ -44,11 +44,25 @@ describe Proc do
     end
   else
     # Ruby 1.8
-    it "should not allow varargs because Ruby 1.8 reports same arity as with no args, so we can't really tell the difference." do
+    it "should not allow varargs 0+ because Ruby 1.8 reports same arity as with no args, so we can't really tell the difference." do
       lambda {
         Object.new.cucumber_instance_exec(true, 'foo', 1) do |*args|
         end
       }.should raise_error(Cucumber::ArityMismatchError, "Your block takes 0 arguments, but the Regexp matched 1 argument.")
     end
   end
+
+  it "should allow varargs (expecting 1+)" do
+    lambda {
+      Object.new.cucumber_instance_exec(true, 'foo', 1) do |arg,*args|
+      end
+    }.should_not raise_error(Cucumber::ArityMismatchError)
+  end
+
+  it "should raise ArityMismatchError for too few required args when using varargs (expecting 1+)" do
+    lambda {
+      Object.new.cucumber_instance_exec(true, nil) do |arg,*args|
+      end
+    }.should raise_error(Cucumber::ArityMismatchError, "Your block takes 1+ arguments, but the Regexp matched 0 arguments.")
+  end
 end

data/spec/cucumber/formatter/html_spec.rb

@@ -0,0 +1,18 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+require 'cucumber/formatter/html'
+
+module Cucumber
+  module Formatter
+    describe Html do
+      before(:each) do
+        @out = StringIO.new
+        @html = Html.new(mock("step mother"), @out, {})
+      end
+
+      it "should not raise an error when visiting a blank feature name" do
+        lambda { @html.visit_feature_name("") }.should_not raise_error
+      end
+    end
+  end
+end
+

data/spec/cucumber/formatter/progress_spec.rb

@@ -17,7 +17,7 @@ module Cucumber
       describe "visiting a table cell value without a status" do
         it "should take the status from the last run step" do
           @progress.visit_step_result('', '', nil, :failed, nil, 10, nil)
-          @progress.visit_table_cell_value('value', 10, nil)
+          @progress.visit_table_cell_value('value', nil)
 
           @out.string.should == "FF"
         end
@@ -25,7 +25,7 @@ module Cucumber
 
       describe "visiting a table cell which is a table header" do
         it "should not output anything" do
-          @progress.visit_table_cell_value('value', 10, :skipped_param)
+          @progress.visit_table_cell_value('value', :skipped_param)
 
           @out.string.should == ""
         end

data/spec/cucumber/parser/table_parser_spec.rb

@@ -45,4 +45,4 @@ module Cucumber
       end
     end
   end
-end
+end