cucumber 0.3.95 → 0.3.96

data/History.txt

@@ -1,3 +1,22 @@
+== 0.3.96 2009-08-15
+
+This release doesn't have any directly visible new features or bug fixes. But there are big
+internal changes. This release has a new API for plugging in other programming languages.
+You can read more about that here: http://groups.google.com/group/cukes/browse_thread/thread/b9db8bf1f3ec9708
+
+This might break other tools that are using Cucumber's internal APIs. For example Spork broke and had to
+be patched. Please upgrade to Spork 0.5.9 if you are using Spork.
+
+=== New Features
+* Ability to preload natural language in Spork's prefork. Rerun script/generate cucumber --spork to see how. (Aslak Hellesøy)
+* Ability to control which DRb port is used via the --port flag or by setting CUCUMBER_DRB environment variable. (Chris Flipse)
+* Upgrade Rails generator to use webrat 0.5.0. (Aslak Hellesøy)
+* Upgrade Sinatra example to work with rack-test 0.4.1 and webrat 0.5.0. (Aslak Hellesøy)
+
+=== Changed Features
+* --strict will cause an exit code 1 for missing and pending (used to be for missing only). (Mads Buus)
+* junit formatter doesn't report pending steps unless --strict is used. (Mads Buus)
+
 == 0.3.95 2009-08-13
 
 This release improves Webrat support for table-like HTML markup. Now you can easily turn the HTML
@@ -15,6 +34,8 @@ This release also fixes several bugs related to --drb (Spork) and profiles (cucu
 * Resolved infinite loop problem when --drb was defined in a profile. (#408 Ben Mabey)
 
 === New Features
+* Cucumber::World#table has been overloaded to work with 2D Array in addition to a table String to be parsed. 
+* New When /^I fill in the following:$/ step definition for Webrat. Useful for filling out a form with a Table. (Aslak Hellesøy)
 * The object returned by element_at (Webrat::Element) has a #to_table that works for table, dl, ol and ul. (Aslak Hellesøy)
 * An explanation of why failures are ok is printed when --wip is used. (Aslak Hellesøy)
 * Added cucumber alias for cucumber:ok in Rails Rake tasks. (Aslak Hellesøy)

data/Manifest.txt

@@ -321,7 +321,7 @@ lib/cucumber/cli/language_help_formatter.rb
 lib/cucumber/cli/main.rb
 lib/cucumber/cli/options.rb
 lib/cucumber/cli/profile_loader.rb
-lib/cucumber/cli/rb_step_def_loader.rb
+lib/cucumber/constantize.rb
 lib/cucumber/core_ext/exception.rb
 lib/cucumber/core_ext/instance_exec.rb
 lib/cucumber/core_ext/proc.rb
@@ -346,12 +346,15 @@ lib/cucumber/formatter/tag_cloud.rb
 lib/cucumber/formatter/unicode.rb
 lib/cucumber/formatter/usage.rb
 lib/cucumber/formatters/unicode.rb
+lib/cucumber/language_support/hook_methods.rb
+lib/cucumber/language_support/language_methods.rb
+lib/cucumber/language_support/step_definition_methods.rb
 lib/cucumber/languages.yml
 lib/cucumber/parser.rb
 lib/cucumber/parser/feature.rb
 lib/cucumber/parser/feature.tt
 lib/cucumber/parser/i18n.tt
-lib/cucumber/parser/i18n/language.rb
+lib/cucumber/parser/natural_language.rb
 lib/cucumber/parser/table.rb
 lib/cucumber/parser/table.tt
 lib/cucumber/parser/treetop_ext.rb
@@ -359,8 +362,11 @@ lib/cucumber/platform.rb
 lib/cucumber/rails/rspec.rb
 lib/cucumber/rails/world.rb
 lib/cucumber/rake/task.rb
+lib/cucumber/rb_support/rb_dsl.rb
+lib/cucumber/rb_support/rb_hook.rb
+lib/cucumber/rb_support/rb_language.rb
+lib/cucumber/rb_support/rb_step_definition.rb
 lib/cucumber/rspec_neuter.rb
-lib/cucumber/step_definition.rb
 lib/cucumber/step_match.rb
 lib/cucumber/step_mother.rb
 lib/cucumber/version.rb

data/examples/sinatra/features/support/env.rb

@@ -1,11 +1,9 @@
 # See http://wiki.github.com/aslakhellesoy/cucumber/sinatra
 # for more details about Sinatra with Cucumber
 
-gem 'rack-test', '=0.3.0'
-gem 'aslakhellesoy-webrat', '=0.4.4.1'
+gem 'rack-test', '=0.4.1'
 gem 'sinatra', '=0.9.4'
 
-# Sinatra
 app_file = File.join(File.dirname(__FILE__), *%w[.. .. app.rb])
 require app_file
 # Force the application name because polyglot breaks the auto-detection logic.

data/features/cucumber_cli.feature

@@ -152,6 +152,7 @@ Feature: Cucumber command line
   Scenario: Run Norwegian
     Given I am in i18n/no
     When I run cucumber -q --language no features
+    Then STDERR should be empty
     Then it should pass with
       """
       # language: no

data/features/drb_server_integration.feature

@@ -23,21 +23,33 @@ Feature: DRb Server Integration
       """
     And a file named "features/sample.feature" with:
       """
+      # language: en
       Feature: Sample
         Scenario: this is a test
           Given I am just testing stuff
       """
+    And a file named "features/essai.feature" with:
+      """
+      # language: fr
+      Fonction: Essai
+        Scenario: ceci est un test
+          Soit je teste
+      """
     And a file named "features/step_definitions/all_your_steps_are_belong_to_us.rb" with:
     """
     Given /^I am just testing stuff$/ do
       # no-op
     end
+
+    Soit /^je teste$/ do
+      # no-op
+    end
     """
 
   Scenario: Feature Passing with --drb flag
     Given I am running spork in the background
 
-    When I run cucumber features/sample.feature --drb
+    When I run cucumber features --drb
     Then it should pass
     And STDERR should be empty
     And the output should contain
@@ -62,7 +74,7 @@ Feature: DRb Server Integration
     """
     And I am running spork in the background
 
-    When I run cucumber features/sample.feature --drb
+    When I run cucumber features --drb
     Then it should fail
     And the output should contain
       """
@@ -82,7 +94,7 @@ Feature: DRb Server Integration
 
     Given I am not running a DRb server in the background
 
-    When I run cucumber features/sample.feature --drb
+    When I run cucumber features --drb
     Then it should pass
     And STDERR should match
       """
@@ -118,3 +130,44 @@ Feature: DRb Server Integration
       I'm loading all the heavy stuff...
       I'm loading the stuff just for this run...
       """
+
+  Scenario: Feature Run with --drb specifying a non-standard port
+
+    Given I am running spork in the background on port 9000
+
+    When I run cucumber features --drb --port 9000
+    Then it should pass
+    And STDERR should be empty
+    And the output should contain
+      """
+      1 step (1 passed)
+      """
+    And the output should contain
+      """
+      I'm loading the stuff just for this run...
+      """
+    And the output should not contain
+      """
+      I'm loading all the heavy stuff...
+      """
+
+  Scenario: Feature Run with $CUCUMBER_DRB environment variable
+
+    Given I have environment variable CUCUMBER_DRB set to "9000"
+    And I am running spork in the background on port 9000
+
+    When I run cucumber features/ --drb
+    Then it should pass
+    And STDERR should be empty
+    And the output should contain
+      """
+      1 step (1 passed)
+      """
+    And the output should contain
+      """
+      I'm loading the stuff just for this run...
+      """
+    And the output should not contain
+      """
+      I'm loading all the heavy stuff...
+      """

data/features/junit_formatter.feature

@@ -27,15 +27,13 @@ Feature: JUnit output formatter
 
       Message:
        (RuntimeError)
-      ./features/step_definitions/steps.rb:6:in `/a failing scenario/'
       features/one_passing_one_failing.feature:7:in `Given a failing scenario'  </failure>
       </testcase>
       </testsuite>
 
       """
   
-  @mri186
-  Scenario: pending step
+  Scenario: pending steps are simply skipped
     When I run cucumber --format junit --out tmp/ features/pending.feature
     Then it should pass with
       """
@@ -44,17 +42,30 @@ Feature: JUnit output formatter
     And "examples/junit/tmp/TEST-pending.xml" with junit duration "0.009" should contain
       """
       <?xml version="1.0" encoding="UTF-8"?>
-      <testsuite errors="0" failures="1" name="Pending step" tests="1" time="0.009">
-      <testcase classname="Pending step.Pending" name="Pending" time="0.009">
+      <testsuite errors="0" failures="0" name="Pending step" tests="0" time="0.009">
+      </testsuite>
+      
+      """
+
+  Scenario: pending step with strict option should fail
+    When I run cucumber --format junit --out tmp/ features/pending.feature --strict
+    Then it should fail with
+      """
+
+      """
+    And "examples/junit/tmp/TEST-pending.xml" with junit duration "0.000160" should contain
+      """
+      <?xml version="1.0" encoding="UTF-8"?>
+      <testsuite errors="0" failures="1" name="Pending step" tests="1" time="0.000160">
+      <testcase classname="Pending step.Pending" name="Pending" time="0.000160">
         <failure message="pending Pending" type="pending">
       Scenario: Pending
 
       TODO (Cucumber::Pending)
-      ./features/step_definitions/steps.rb:10:in `/a pending step/'
       features/pending.feature:4:in `Given a pending step'  </failure>
       </testcase>
       </testsuite>
-      
+
       """
     
   Scenario: run all features
@@ -68,11 +79,11 @@ Feature: JUnit output formatter
   
   Scenario: show correct error message if no --out is passed
     When I run cucumber --format junit features
-	  Then STDERR should not match 
-		  """
+    Then STDERR should not match 
+      """
 can't convert .* into String \(TypeError\)
-		  """
+      """
     And STDERR should match
-		  """
+      """
 You \*must\* specify \-\-out DIR for the junit formatter
-	    """
+      """

data/features/step_definitions/cucumber_steps.rb

@@ -35,12 +35,20 @@ Given /^I am running spork in the background$/ do
   run_spork_in_background
 end
 
+Given /^I am running spork in the background on port (\d+)$/ do |port|
+  run_spork_in_background(port.to_i)
+end
+
 Given /^I am not running (?:.*) in the background$/ do
   # no-op
 end
 
+Given /^I have environment variable (\w+) set to "([^\"]*)"$/ do |variable, value|
+  set_env_var(variable, value)
+end
+
 When /^I run cucumber (.*)$/ do |cucumber_opts|
-  run "#{Cucumber::RUBY_BINARY} #{Cucumber::BINARY} --no-color #{cucumber_opts}"
+  run "#{Cucumber::RUBY_BINARY} -I #{Spork::LIBDIR} #{Cucumber::BINARY} --no-color #{cucumber_opts}"
 end
 
 When /^I run rake (.*)$/ do |rake_opts|
@@ -74,13 +82,15 @@ Then /^the output should be$/ do |text|
   last_stdout.should == text
 end
 
+
 Then /^"([^\"]*)" should contain$/ do |file, text|
   strip_duration(IO.read(file)).should == text
 end
 
 Then /^"([^\"]*)" with junit duration "([^\"]*)" should contain$/ do |actual_file, duration_replacement, text|
   actual = IO.read(actual_file)
-  actual = replace_junit_duration(actual, duration_replacement)
+  actual = replace_junit_duration(actual, duration_replacement) 
+  actual = strip_ruby186_extra_trace(actual)
   actual.should == text
 end
 
@@ -141,3 +151,4 @@ Then /^print output$/ do
   puts last_stdout
 end
 
+

data/features/support/env.rb

@@ -6,7 +6,7 @@ require 'forwardable'
 begin
   require 'spork'
 rescue Gem::LoadError => ex
-  gem 'spork', '>= 0.5.7' # Ensure correct spork version number to avoid false-negatives.
+  gem 'spork', '>= 0.5.9' # Ensure correct spork version number to avoid false-negatives.
 end
 
 class CucumberWorld
@@ -54,6 +54,10 @@ class CucumberWorld
     s.gsub(/\d+\.\d\d+/m, replacement)
   end
 
+  def strip_ruby186_extra_trace(s)  
+    s.gsub(/^.*\.\/features\/step_definitions(.*)\n/, "")
+  end
+
   def create_file(file_name, file_content)
     file_content.gsub!("CUCUMBER_LIB", "'#{cucumber_lib_dir}'") # Some files, such as Rakefiles need to use the lib dir
     in_current_dir do
@@ -61,6 +65,12 @@ class CucumberWorld
     end
   end
 
+  def set_env_var(variable, value)
+    @original_env_vars ||= {}
+    @original_env_vars[variable] = ENV[variable] 
+    ENV[variable]  = value
+  end
+
   def background_jobs
     @background_jobs ||= []
   end
@@ -84,7 +94,7 @@ class CucumberWorld
     @last_stderr = IO.read(stderr_file.path)
   end
 
-  def run_spork_in_background
+  def run_spork_in_background(port = nil)
     pid = fork
     in_current_dir do
       if pid
@@ -92,7 +102,8 @@ class CucumberWorld
       else
         # STDOUT.close
         # STDERR.close
-        cmd = "#{Cucumber::RUBY_BINARY} -I #{Cucumber::LIBDIR} #{Spork::BINARY} cuc"
+        port_arg = port ? "-p #{port}" : ''
+        cmd = "#{Cucumber::RUBY_BINARY} -I #{Cucumber::LIBDIR} #{Spork::BINARY} cuc #{port_arg}"
         exec cmd
       end
     end
@@ -107,6 +118,10 @@ class CucumberWorld
     end
   end
 
+  def restore_original_env_vars
+    @original_env_vars.each { |variable, value| ENV[variable] = value } if @original_env_vars
+  end
+
 end
 
 World do
@@ -120,4 +135,5 @@ end
 
 After do
   terminate_background_jobs
+  restore_original_env_vars
 end

data/lib/cucumber/ast/feature_element.rb

@@ -21,7 +21,7 @@ module Cucumber
     end
 
     def name_line_lengths
-      if @name.empty?
+      if @name.strip.empty?
         [@keyword.jlength]
       else
         @name.split("\n").enum_for(:each_with_index).map do |line, line_number|

data/lib/cucumber/ast/step.rb

@@ -1,5 +1,5 @@
-require 'cucumber/step_definition'
 require 'cucumber/core_ext/string'
+require 'cucumber/step_match'
 
 module Cucumber
   module Ast

data/lib/cucumber/ast/step_invocation.rb

@@ -1,3 +1,5 @@
+require 'cucumber/step_match'
+
 module Cucumber
   module Ast
     class StepInvocation
@@ -33,8 +35,7 @@ module Cucumber
         unless @skip_invoke || options[:dry_run] || @exception || @step_collection.exception
           @skip_invoke = true
           begin
-            step_mother.current_world.__cucumber_current_step = self if step_mother.current_world # Nil in Pure Java
-            @step_match.invoke(step_mother.current_world, @multiline_arg)
+            @step_match.invoke(@multiline_arg)
             step_mother.after_step
             status!(:passed)
           rescue Pending => e

data/lib/cucumber/cli/configuration.rb

@@ -1,4 +1,5 @@
 require 'cucumber/cli/options'
+require 'cucumber/constantize'
 
 module Cucumber
   module Cli
@@ -7,6 +8,8 @@ module Cucumber
     class ProfileNotFound < StandardError; end
 
     class Configuration
+      include Constantize
+      
       attr_reader :options
 
       def initialize(out_stream = STDOUT, error_stream = STDERR)
@@ -50,6 +53,10 @@ module Cucumber
         @options[:drb]
       end
 
+      def drb_port
+        @options[:drb_port].to_i if @options[:drb_port]
+      end
+
       def build_formatter_broadcaster(step_mother)
         return Formatter::Pretty.new(step_mother, nil, @options) if @options[:autoformat]
         formatters = @options[:formats].map do |format_and_out|
@@ -98,6 +105,8 @@ module Cucumber
         env_files = sorted_files.select {|f| f =~ %r{/support/env\..*} }
         files = env_files + sorted_files.reject {|f| f =~ %r{/support/env\..*} }
         remove_excluded_files_from(files)
+        files.reject! {|f| !File.file?(f)}
+        files.reject! {|f| File.extname(f) == '.feature' }
         files.reject! {|f| f =~ %r{/support/env\..*} } if @options[:dry_run]
         files
       end
@@ -112,22 +121,6 @@ module Cucumber
         potential_feature_files
       end
 
-      def constantize(camel_cased_word)
-        begin
-          names = camel_cased_word.split('::')
-          names.shift if names.empty? || names.first.empty?
-
-          constant = Object
-          names.each do |name|
-            constant = constant.const_defined?(name) ? constant.const_get(name) : constant.const_missing(name)
-          end
-          constant
-        rescue NameError
-          require underscore(camel_cased_word)
-          retry
-        end
-      end
-
     private
 
       def paths
@@ -159,15 +152,6 @@ module Cucumber
       def require_dirs
         feature_dirs + Dir['vendor/{gems,plugins}/*/cucumber']
       end
-
-      # Snagged from active_support
-      def underscore(camel_cased_word)
-        camel_cased_word.to_s.gsub(/::/, '/').
-          gsub(/([A-Z]+)([A-Z][a-z])/,'\1_\2').
-          gsub(/([a-z\d])([A-Z])/,'\1_\2').
-          tr("-", "_").
-          downcase
-      end
       
     end