aslakhellesoy-cucumber 0.3.92 → 0.3.93

data/lib/cucumber/parser/feature.tt

@@ -21,7 +21,11 @@ module Cucumber
           def has_tags?(tag_names)
             tags.has_tags?(tag_names)
           end
-          
+
+          def has_all_tags?(tag_names)
+            tags.has_all_tags?(tag_names)
+          end
+
           def build(filter)
             if(filter.nil? || feature_elements.accept?(filter) || (!bg.empty? && filter.accept?(bg)))
               background = bg.respond_to?(:build) ? bg.build : nil      
@@ -47,6 +51,10 @@ module Cucumber
             (tag_names & tags).any?
           end
 
+          def has_all_tags?(tags)
+            (tags & tag_names) == tags
+          end
+
           def build
             Ast::Tags.new(ts.line, tag_names)
           end
@@ -89,6 +97,11 @@ module Cucumber
             feature_tags.has_tags?(tag_names)
           end
 
+          def has_all_tags?(tag_names)
+            feature_tags = self.parent.tags
+            feature_tags.has_all_tags?(tag_names)
+          end
+
           def build
             Ast::Background.new(
               comment.build, 
@@ -130,6 +143,11 @@ module Cucumber
             tags.has_tags?(tag_names) || feature_tags.has_tags?(tag_names)
           end
 
+          def has_all_tags?(tag_names)
+            feature_tags = self.parent.parent.tags
+            tags.has_all_tags?(tag_names) || feature_tags.has_all_tags?(tag_names)
+          end
+
           def matches_name?(regexp_to_match)
             name.build =~ regexp_to_match
           end
@@ -166,6 +184,11 @@ module Cucumber
             tags.has_tags?(tag_names) || feature_tags.has_tags?(tag_names)
           end
 
+          def has_all_tags?(tag_names)
+            feature_tags = self.parent.parent.tags
+            tags.has_all_tags?(tag_names) || feature_tags.has_all_tags?(tag_names)
+          end
+
           def matches_name?(regexp_to_match)
             outline_matches_name?(regexp_to_match) || examples_sections.matches_name?(regexp_to_match)
           end
@@ -249,6 +272,10 @@ module Cucumber
             true
           end
 
+          def has_all_tags?(tag_names)
+            true
+          end
+
           def outline_at_line?(line)
             true
           end

data/lib/cucumber/parser/i18n/language.rb

@@ -47,6 +47,10 @@ module Cucumber
           Treetop.load_from_string(grammar)
           self.class.alias_step_definitions(@keywords)
           @parser = Parser::I18n.const_get("#{@keywords['grammar_name']}Parser").new
+          def @parser.inspect
+            "#<#{self.class.name}>"
+          end
+          @parser
         end
 
         def parse(source, path, filter)

data/lib/cucumber/parser/table.rb

@@ -49,7 +49,7 @@ module Cucumber
           end
         end
         if s0.empty?
-          self.index = i0
+          @index = i0
           r0 = nil
         else
           r0 = instantiate_node(SyntaxNode,input, i0...index, s0)
@@ -58,7 +58,7 @@ module Cucumber
 
         node_cache[:table][start_index] = r0
 
-        return r0
+        r0
       end
 
       module TableRow0
@@ -116,7 +116,7 @@ module Cucumber
         r1 = instantiate_node(SyntaxNode,input, i1...index, s1)
         s0 << r1
         if r1
-          if input.index('|', index) == index
+          if has_terminal?('|', false, index)
             r3 = instantiate_node(SyntaxNode,input, index...(index + 1))
             @index += 1
           else
@@ -131,7 +131,7 @@ module Cucumber
               r6 = _nt_cell
               s5 << r6
               if r6
-                if input.index('|', index) == index
+                if has_terminal?('|', false, index)
                   r7 = instantiate_node(SyntaxNode,input, index...(index + 1))
                   @index += 1
                 else
@@ -144,7 +144,7 @@ module Cucumber
                 r5 = instantiate_node(SyntaxNode,input, i5...index, s5)
                 r5.extend(TableRow0)
               else
-                self.index = i5
+                @index = i5
                 r5 = nil
               end
               if r5
@@ -154,7 +154,7 @@ module Cucumber
               end
             end
             if s4.empty?
-              self.index = i4
+              @index = i4
               r4 = nil
             else
               r4 = instantiate_node(SyntaxNode,input, i4...index, s4)
@@ -184,7 +184,7 @@ module Cucumber
                   end
                 end
                 if s11.empty?
-                  self.index = i11
+                  @index = i11
                   r11 = nil
                 else
                   r11 = instantiate_node(SyntaxNode,input, i11...index, s11)
@@ -196,7 +196,7 @@ module Cucumber
                   if r13
                     r10 = r13
                   else
-                    self.index = i10
+                    @index = i10
                     r10 = nil
                   end
                 end
@@ -210,13 +210,13 @@ module Cucumber
           r0.extend(TableRow1)
           r0.extend(TableRow2)
         else
-          self.index = i0
+          @index = i0
           r0 = nil
         end
 
         node_cache[:table_row][start_index] = r0
 
-        return r0
+        r0
       end
 
       module Cell0
@@ -235,7 +235,7 @@ module Cucumber
           i1, s1 = index, []
           i2 = index
           i3 = index
-          if input.index('|', index) == index
+          if has_terminal?('|', false, index)
             r4 = instantiate_node(SyntaxNode,input, index...(index + 1))
             @index += 1
           else
@@ -249,14 +249,14 @@ module Cucumber
             if r5
               r3 = r5
             else
-              self.index = i3
+              @index = i3
               r3 = nil
             end
           end
           if r3
             r2 = nil
           else
-            self.index = i2
+            @index = i2
             r2 = instantiate_node(SyntaxNode,input, index...index)
           end
           s1 << r2
@@ -274,7 +274,7 @@ module Cucumber
             r1 = instantiate_node(SyntaxNode,input, i1...index, s1)
             r1.extend(Cell0)
           else
-            self.index = i1
+            @index = i1
             r1 = nil
           end
           if r1
@@ -287,7 +287,7 @@ module Cucumber
 
         node_cache[:cell][start_index] = r0
 
-        return r0
+        r0
       end
 
       def _nt_space
@@ -298,7 +298,7 @@ module Cucumber
           return cached
         end
 
-        if input.index(Regexp.new('[ \\t]'), index) == index
+        if has_terminal?('\G[ \\t]', true, index)
           r0 = instantiate_node(SyntaxNode,input, index...(index + 1))
           @index += 1
         else
@@ -307,7 +307,7 @@ module Cucumber
 
         node_cache[:space][start_index] = r0
 
-        return r0
+        r0
       end
 
       module Eol0
@@ -322,7 +322,7 @@ module Cucumber
         end
 
         i0 = index
-        if input.index("\n", index) == index
+        if has_terminal?("\n", false, index)
           r1 = instantiate_node(SyntaxNode,input, index...(index + 1))
           @index += 1
         else
@@ -333,7 +333,7 @@ module Cucumber
           r0 = r1
         else
           i2, s2 = index, []
-          if input.index("\r", index) == index
+          if has_terminal?("\r", false, index)
             r3 = instantiate_node(SyntaxNode,input, index...(index + 1))
             @index += 1
           else
@@ -342,7 +342,7 @@ module Cucumber
           end
           s2 << r3
           if r3
-            if input.index("\n", index) == index
+            if has_terminal?("\n", false, index)
               r5 = instantiate_node(SyntaxNode,input, index...(index + 1))
               @index += 1
             else
@@ -360,20 +360,20 @@ module Cucumber
             r2 = instantiate_node(SyntaxNode,input, i2...index, s2)
             r2.extend(Eol0)
           else
-            self.index = i2
+            @index = i2
             r2 = nil
           end
           if r2
             r0 = r2
           else
-            self.index = i0
+            @index = i0
             r0 = nil
           end
         end
 
         node_cache[:eol][start_index] = r0
 
-        return r0
+        r0
       end
 
       def _nt_eof
@@ -395,13 +395,13 @@ module Cucumber
         if r1
           r0 = nil
         else
-          self.index = i0
+          @index = i0
           r0 = instantiate_node(SyntaxNode,input, index...index)
         end
 
         node_cache[:eof][start_index] = r0
 
-        return r0
+        r0
       end
 
     end

data/lib/cucumber/version.rb

@@ -2,7 +2,7 @@ module Cucumber #:nodoc:
   class VERSION #:nodoc:
     MAJOR = 0
     MINOR = 3
-    TINY  = 92
+    TINY  = 93
     PATCH = nil # Set to nil for official release
 
     STRING = [MAJOR, MINOR, TINY, PATCH].compact.join('.')

data/lib/cucumber/webrat/table_locator.rb

@@ -4,7 +4,7 @@ module Webrat
       ".//table"
     end
 
-    # Converts this Table element into a 2-dimensional array of String where each cell
+    # Converts this Table element into an Array of Array of String where each cell
     # represents the inner_html of the <td> and <th> elements. The number of columns is
     # determined by the number of cells in the first row.
     def to_a

data/rails_generators/cucumber/cucumber_generator.rb

@@ -20,7 +20,7 @@ class CucumberGenerator < Rails::Generator::Base
 
       m.directory 'features/support'
 
-      if options[:spork]
+      if spork?
         m.template  'spork_env.rb',     'features/support/env.rb'
       else
         m.template  'env.rb',           'features/support/env.rb'
@@ -40,6 +40,10 @@ class CucumberGenerator < Rails::Generator::Base
   def framework
     options[:framework] || :rspec
   end
+  
+  def spork?
+    options[:spork]
+  end
 
 protected
 

data/rails_generators/cucumber/templates/cucumber

@@ -3,6 +3,7 @@ begin
   load File.expand_path(File.dirname(__FILE__) + "/../vendor/plugins/cucumber/bin/cucumber")
 rescue LoadError => e
   raise unless e.to_s =~ /cucumber/  
-  require "rubygems"
-  load File.join(Gem.bindir, "cucumber")
+  require 'rubygems'
+  require 'cucumber'
+  load Cucumber::BINARY
 end

data/rails_generators/cucumber/templates/cucumber_environment.rb

@@ -15,9 +15,12 @@ config.action_controller.allow_forgery_protection    = false
 # ActionMailer::Base.deliveries array.
 config.action_mailer.delivery_method = :test
 
-config.gem "cucumber",    :lib => false,        :version => ">=<%= cucumber_version %>" unless File.directory?(File.join(Rails.root, 'vendor/plugins/cucumber'))
-config.gem "webrat",      :lib => false,        :version => ">=0.4.4" unless File.directory?(File.join(Rails.root, 'vendor/plugins/webrat'))
+config.gem 'cucumber',    :lib => false,        :version => '>=<%= cucumber_version %>' unless File.directory?(File.join(Rails.root, 'vendor/plugins/cucumber'))
+config.gem 'webrat',      :lib => false,        :version => '>=0.4.4' unless File.directory?(File.join(Rails.root, 'vendor/plugins/webrat'))
 <% if framework == :rspec -%>
-config.gem "rspec",       :lib => false,        :version => ">=1.2.6" unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec'))
-config.gem "rspec-rails", :lib => 'spec/rails', :version => ">=1.2.6" unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec-rails'))
+config.gem 'rspec',       :lib => false,        :version => '>=1.2.6' unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec'))
+config.gem 'rspec-rails', :lib => 'spec/rails', :version => '>=1.2.6' unless File.directory?(File.join(Rails.root, 'vendor/plugins/rspec-rails'))
+<% end %>
+<% if spork? -%>
+config.gem 'spork',       :lib => false,        :version => '>=0.5.7' unless File.directory?(File.join(Rails.root, 'vendor/plugins/spork'))
 <% end %>

data/rails_generators/cucumber/templates/webrat_steps.rb

@@ -19,6 +19,10 @@ When /^I follow "([^\"]*)"$/ do |link|
   click_link(link)
 end
 
+When /^I follow "([^\"]*)" within "([^\"]*)"$/ do |link, parent|
+  click_link_within(parent, link)
+end
+
 When /^I fill in "([^\"]*)" with "([^\"]*)"$/ do |field, value|
   fill_in(field, :with => value) 
 end

data/spec/cucumber/cli/configuration_spec.rb

@@ -17,13 +17,25 @@ module Cli
     end
 
     before(:each) do
+      #given_cucumber_yml_defined_as({'default' => '-q'})
+      File.stub!(:exist?).and_return(false) # Meaning, no cucumber.yml exists
       Kernel.stub!(:exit).and_return(nil)
     end
 
+    def config
+      @config ||= Configuration.new(@out = StringIO.new, @error = StringIO.new)
+    end
+
+    def reset_config
+      @config = nil
+    end
+
+    attr_reader :out, :error
+
+
     it "should require files in support paths first" do
       given_the_following_files("/features/step_definitions/foo.rb","/features/support/bar.rb")
 
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--require /features})
 
       config.files_to_require.should == [
@@ -35,7 +47,6 @@ module Cli
     it "should require env.rb files first" do
       given_the_following_files("/features/support/a_file.rb","/features/support/env.rb")
 
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--require /features})
 
       config.files_to_require.should == [
@@ -47,7 +58,6 @@ module Cli
     it "should not require env.rb files when --dry-run" do
       given_the_following_files("/features/support/a_file.rb","/features/support/env.rb")
 
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--require /features --dry-run})
 
       config.files_to_require.should == [
@@ -59,7 +69,6 @@ module Cli
       given_the_following_files("/vendor/plugins/plugin_a/cucumber/foo.rb",
                                 "/vendor/gems/gem_a/cucumber/bar.rb")
 
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--require /features})
 
       config.files_to_require.should == [
@@ -73,7 +82,6 @@ module Cli
       it "excludes a ruby file from requiring when the name matches exactly" do
         given_the_following_files("/features/support/a_file.rb","/features/support/env.rb")
 
-        config = Configuration.new(StringIO.new)
         config.parse!(%w{--require /features --exclude a_file.rb})
 
         config.files_to_require.should == [
@@ -86,7 +94,6 @@ module Cli
                                   "/features/support/food.rb","/features/blah.rb",
                                   "/features/support/fooz.rb")
 
-        config = Configuration.new(StringIO.new)
         config.parse!(%w{--require /features --exclude foo[df] --exclude blah})
 
         config.files_to_require.should == [
@@ -98,29 +105,23 @@ module Cli
 
     describe '#drb?' do
       it "indicates whether the --drb flag was passed in or not" do
-        config = Configuration.new(StringIO.new)
-
         config.parse!(%w{features})
-        config.drb?.should == false
+        config.should_not be_drb
 
 
         config.parse!(%w{features --drb})
-        config.drb?.should == true
+        config.should be_drb
       end
     end
 
     context '--drb' do
       it "removes the --drb flag from the args" do
-        config = Configuration.new(StringIO.new)
-
         args = %w{features --drb}
         config.parse!(args)
         args.should == %w{features}
       end
 
       it "keeps all other flags intact" do
-        config = Configuration.new(StringIO.new)
-
         args = %w{features --drb --format profile}
         config.parse!(args)
         args.should == %w{features --format profile}
@@ -131,7 +132,6 @@ module Cli
     context '--drb in a profile' do
       it "removes the --drb flag from the args" do
         given_cucumber_yml_defined_as({'server' => '--drb features'})
-        config = Configuration.new(StringIO.new)
 
         args = %w{--profile server}
         config.parse!(args)
@@ -142,11 +142,9 @@ module Cli
         given_cucumber_yml_defined_as({'server' => '--drb features --profile nested',
                                        'nested' => '--verbose'})
 
-        config = Configuration.new(StringIO.new)
-
         args = %w{--profile server --format profile}
         config.parse!(args)
-        args.should == %w{features --verbose --format profile}
+        args.should == %w{--format profile features --verbose}
       end
 
     end
@@ -154,35 +152,40 @@ module Cli
     context '--drb in the default profile and no arguments specified' do
       it "expands the profile's arguments into the args excpet for --drb" do
         given_cucumber_yml_defined_as({'default' => '--drb features --format pretty'})
-        config = Configuration.new(StringIO.new)
         args = []
         config.parse!(args)
         args.should == %w{features --format pretty}
       end
     end
 
-    it "should expand args from YAML file" do
-      given_cucumber_yml_defined_as({'bongo' => '--require from/yml'})
+    it "uses the default profile when no profile is defined" do
+      given_cucumber_yml_defined_as({'default' => '--require some_file'})
 
-      config = Configuration.new
-      config.parse!(%w{--format progress --profile bongo})
-      config.options[:formats].should == [['progress', STDOUT]]
-      config.options[:require].should == ['from/yml']
+      config.parse!(%w{--format progress})
+      config.options[:require].should include('some_file')
     end
 
-    it "should expand args from YAML file's default if there are no args" do
-      given_cucumber_yml_defined_as({'default' => '--require from/yml'})
+    context '--profile' do
 
-      config = Configuration.new
-      config.parse!([])
-      config.options[:require].should == ['from/yml']
-    end
+      it "expands args from profiles in the cucumber.yml file" do
+        given_cucumber_yml_defined_as({'bongo' => '--require from/yml'})
+
+        config.parse!(%w{--format progress --profile bongo})
+        config.options[:formats].should == [['progress', out]]
+        config.options[:require].should == ['from/yml']
+      end
+
+      it "expands args from the default profile when no flags are provided" do
+        given_cucumber_yml_defined_as({'default' => '--require from/yml'})
+
+        config.parse!([])
+        config.options[:require].should == ['from/yml']
+      end
 
-    it "should provide a helpful error message when a specified profile does not exists in YAML file" do
-      given_cucumber_yml_defined_as({'default' => '--require from/yml', 'html_report' =>  '--format html'})
+      it "provides a helpful error message when a specified profile does not exists in cucumber.yml" do
+        given_cucumber_yml_defined_as({'default' => '--require from/yml', 'html_report' =>  '--format html'})
 
-      config = Configuration.new(StringIO.new, error = StringIO.new)
-      expected_message = <<-END_OF_MESSAGE
+        expected_message = <<-END_OF_MESSAGE
 Could not find profile: 'i_do_not_exist'
 
 Defined profiles in cucumber.yml:
@@ -190,141 +193,175 @@ Defined profiles in cucumber.yml:
   * html_report
 END_OF_MESSAGE
 
-      lambda{config.parse!(%w{--profile i_do_not_exist})}.should raise_error(expected_message)
-    end
+        lambda{config.parse!(%w{--profile i_do_not_exist})}.should raise_error(ProfileNotFound, expected_message)
+      end
 
-    it "should allow array as profile" do
-      given_cucumber_yml_defined_as({'foo' => [1,2,3]})
+      it "allows profiles to be defined in arrays" do
+        given_cucumber_yml_defined_as({'foo' => [1,2,3]})
 
-      config = Configuration.new(StringIO.new, error = StringIO.new)
-      config.parse!(%w{--profile foo})
-      config.paths.should == [1,2,3]
-    end
+        config.parse!(%w{--profile foo})
+        config.paths.should == [1,2,3]
+      end
 
-    it "should provide a helpful error message when a specified profile exists but is nil or blank" do
-      [nil, '   '].each do |bad_input|
-        given_cucumber_yml_defined_as({'foo' => bad_input})
+      it "notifies the user that an individual profile is being used" do
+        given_cucumber_yml_defined_as({'foo' => [1,2,3]})
 
-        config = Configuration.new(StringIO.new, error = StringIO.new)
-        expected_error = /The 'foo' profile in cucumber.yml was blank.  Please define the command line arguments for the 'foo' profile in cucumber.yml./
-        lambda{config.parse!(%w{--profile foo})}.should raise_error(expected_error)
+        config.parse!(%w{--profile foo})
+        out.string.should =~ /Using the foo profile...\n/
       end
-    end
 
-    it "should provide a helpful error message when no YAML file exists and a profile is specified" do
-      File.should_receive(:exist?).with('cucumber.yml').and_return(false)
+      it "notifies the user when multiple profiles are being used" do
+        given_cucumber_yml_defined_as({'foo' => [1,2,3], 'bar' => ['v'], 'dog' => ['v']})
 
-      config = Configuration.new(StringIO.new, error = StringIO.new)
-      expected_error = /cucumber.yml was not found.  Please refer to cucumber's documentation on defining profiles in cucumber.yml./
-      lambda{config.parse!(%w{--profile i_do_not_exist})}.should raise_error(expected_error)
-    end
+        config.parse!(%w{--profile foo --profile bar})
+        out.string.should =~ /Using the foo and bar profiles...\n/
 
-    it "should provide a helpful error message when cucumber.yml is blank or malformed" do
-        expected_error_message = /cucumber.yml was found, but was blank or malformed. Please refer to cucumber's documentation on correct profile usage./
+        reset_config
 
-      ['', 'sfsadfs', "--- \n- an\n- array\n", "---dddfd"].each do |bad_input|
-        given_cucumber_yml_defined_as(bad_input)
+        config.parse!(%w{--profile foo --profile bar --profile dog})
+        out.string.should =~ /Using the foo, bar and dog profiles...\n/
+      end
 
-        config = Configuration.new(StringIO.new, error = StringIO.new)
-        lambda{config.parse!([])}.should raise_error(expected_error_message)
+      it "disregards paths in profiles when other paths are passed in (via cmd line)" do
+        given_cucumber_yml_defined_as({'foo' => %w[-v features]})
+
+        config.parse!(%w{--profile foo features/specific.feature --format pretty})
+        config.paths.should == ['features/specific.feature']
       end
-    end
 
-    it "should procide a helpful error message when the YAML can not be parsed" do
-      expected_error_message = /cucumber.yml was found, but could not be parsed. Please refer to cucumber's documentation on correct profile usage./
+      it "disregards default STDOUT formatter defined in profile when another is passed in (via cmd line)" do
+        given_cucumber_yml_defined_as({'foo' => %w[--format pretty]})
+        config.parse!(%w{--format progress --profile foo})
+        config.options[:formats].should == [['progress', out]]#, ['pretty', 'pretty.txt']]
+      end
+
+
+
+      ["--no-profile", "-P"].each do |flag|
+
+        context 'when none is specified with #{flag}' do
 
-      given_cucumber_yml_defined_as("input that causes an exception in YAML loading")
-      YAML.should_receive(:load).and_raise ArgumentError
+          it "disables profiles" do
+            given_cucumber_yml_defined_as({'default' => '-v --require file_specified_in_default_profile.rb'})
 
-      config = Configuration.new(StringIO.new, error = StringIO.new)
-      lambda{config.parse!([])}.should raise_error(expected_error_message)
+            config.parse!("#{flag} --require some_file.rb".split(" "))
+            config.options[:require].should == ['some_file.rb']
+          end
+
+          it "notifies the user that the profiles are being disabled" do
+            given_cucumber_yml_defined_as({'default' => '-v'})
+
+            config.parse!("#{flag} --require some_file.rb".split(" "))
+            out.string.should =~ /Disabling profiles.../
+          end
+
+        end
+
+      end
+
+
+
+      it "issues a helpful error message when a specified profile exists but is nil or blank" do
+        [nil, '   '].each do |bad_input|
+          given_cucumber_yml_defined_as({'foo' => bad_input})
+
+          expected_error = /The 'foo' profile in cucumber.yml was blank.  Please define the command line arguments for the 'foo' profile in cucumber.yml./
+          lambda{config.parse!(%w{--profile foo})}.should raise_error(expected_error)
+        end
+      end
+
+      it "issues a helpful error message when no YAML file exists and a profile is specified" do
+        File.should_receive(:exist?).with('cucumber.yml').and_return(false)
+
+        expected_error = /cucumber.yml was not found.  Please refer to cucumber's documentation on defining profiles in cucumber.yml./
+        lambda{config.parse!(%w{--profile i_do_not_exist})}.should raise_error(expected_error)
+      end
+
+      it "issues a helpful error message when cucumber.yml is blank or malformed" do
+          expected_error_message = /cucumber.yml was found, but was blank or malformed. Please refer to cucumber's documentation on correct profile usage./
+
+        ['', 'sfsadfs', "--- \n- an\n- array\n", "---dddfd"].each do |bad_input|
+          given_cucumber_yml_defined_as(bad_input)
+          lambda{config.parse!([])}.should raise_error(expected_error_message)
+          reset_config
+        end
+      end
+
+      it "issues a helpful error message when cucumber.yml can not be parsed" do
+        expected_error_message = /cucumber.yml was found, but could not be parsed. Please refer to cucumber's documentation on correct profile usage./
+
+        given_cucumber_yml_defined_as("input that causes an exception in YAML loading")
+        YAML.should_receive(:load).and_raise ArgumentError
+
+        lambda{config.parse!([])}.should raise_error(expected_error_message)
+      end
     end
 
+
     it "should accept --dry-run option" do
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--dry-run})
       config.options[:dry_run].should be_true
     end
 
     it "should accept --no-source option" do
-      config = Configuration.new
       config.parse!(%w{--no-source})
 
       config.options[:source].should be_false
     end
 
     it "should accept --no-snippets option" do
-      config = Configuration.new
       config.parse!(%w{--no-snippets})
 
       config.options[:snippets].should be_false
     end
 
     it "should set snippets and source to false with --quiet option" do
-      config = Configuration.new
       config.parse!(%w{--quiet})
 
-      config.options[:snippets].should be_nil
-      config.options[:source].should be_nil
+      config.options[:snippets].should be_false
+      config.options[:source].should be_false
     end
 
     it "should accept --verbose option" do
-      config = Configuration.new
       config.parse!(%w{--verbose})
 
       config.options[:verbose].should be_true
     end
 
     it "should accept --out option" do
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--out jalla.txt})
       config.options[:formats].should == [['pretty', 'jalla.txt']]
     end
 
     it "should accept multiple --out options" do
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--format progress --out file1 --out file2})
       config.options[:formats].should == [['progress', 'file2']]
     end
 
     it "should accept multiple --format options and put the STDOUT one first so progress is seen" do
-      io = StringIO.new
-      config = Configuration.new(io)
       config.parse!(%w{--format pretty --out pretty.txt --format progress})
-      config.options[:formats].should == [['progress', io], ['pretty', 'pretty.txt']]
+      config.options[:formats].should == [['progress', out], ['pretty', 'pretty.txt']]
     end
 
     it "should not accept multiple --format options when both use implicit STDOUT" do
-      io = StringIO.new
-      config = Configuration.new(io)
       lambda do
         config.parse!(%w{--format pretty --format progress})
       end.should raise_error("All but one formatter must use --out, only one can print to STDOUT")
     end
 
     it "should associate --out to previous --format" do
-      config = Configuration.new(StringIO.new)
       config.parse!(%w{--format progress --out file1 --format profile --out file2})
       config.options[:formats].should == [["progress", "file1"], ["profile" ,"file2"]]
     end
 
     it "should accept --color option" do
       Term::ANSIColor.should_receive(:coloring=).with(true)
-      config = Configuration.new(StringIO.new)
       config.parse!(['--color'])
     end
 
     it "should accept --no-color option" do
       Term::ANSIColor.should_receive(:coloring=).with(false)
-      config = Configuration.new(StringIO.new)
-      config.parse!(['--no-color'])
-    end
-
-    it "should parse tags" do
-      config = Configuration.new(nil)
-      includes, excludes = config.parse_tags("one,~two,@three,~@four")
-      includes.should == ['one', 'three']
-      excludes.should == ['two', 'four']
+      config.parse!(%w[--no-color])
     end
 
     describe "--backtrace" do
@@ -349,12 +386,10 @@ END_OF_MESSAGE
     describe "diff output" do
 
       it "is enabled by default" do
-        config = Configuration.new
         config.diff_enabled?.should be_true
       end
 
       it "is disabled when the --no-diff option is supplied" do
-        config = Configuration.new
         config.parse!(%w{--no-diff})
 
         config.diff_enabled?.should be_false
@@ -363,7 +398,6 @@ END_OF_MESSAGE
     end
 
     it "should accept multiple --name options" do
-      config = Configuration.new
       config.parse!(['--name', "User logs in", '--name', "User signs up"])
 
       config.options[:name_regexps].should include(/User logs in/)
@@ -371,7 +405,6 @@ END_OF_MESSAGE
     end
 
     it "should accept multiple -n options" do
-      config = Configuration.new
       config.parse!(['-n', "User logs in", '-n', "User signs up"])
 
       config.options[:name_regexps].should include(/User logs in/)
@@ -383,22 +416,19 @@ END_OF_MESSAGE
       Dir.should_receive(:[]).with("feature_directory/**/*.feature").
         any_number_of_times.and_return(["cucumber.feature"])
 
-      config = Configuration.new(StringIO)
       config.parse!(%w{feature_directory/})
 
       config.feature_files.should == ["cucumber.feature"]
     end
 
     it "should allow specifying environment variables on the command line" do
-      config = Configuration.new
       config.parse!(["foo=bar"])
       ENV["foo"].should == "bar"
       config.feature_files.should == []
     end
-    
+
     it "should allow specifying environment variables in profiles" do
       given_cucumber_yml_defined_as({'selenium' => 'RAILS_ENV=selenium'})
-      config = Configuration.new
       config.parse!(["--profile", "selenium"])
       ENV["RAILS_ENV"].should == "selenium"
       config.feature_files.should == []