aslakhellesoy-cucumber 0.3.102.1 → 0.3.102.2

data/History.txt

@@ -10,6 +10,9 @@ Using this release on a Rails project requires a rerun of script/generate cucumb
 * Added back a way to globally turn off transactions.
 * Renamed @allow_rescue tag to @allow-rescue.
 
+=== Bugfixes
+* Gracefully handle cases when optional regexp groups are not matched. Ex: /should( not)? be flashed '([^']*?)'$/ (Aslak Hellesøy)
+
 == 2009-09-22
 
 This release has some changes in the Rails support, so make sure you run "script/generate cucumber" after you upgrade.

data/examples/java/README.textile

@@ -11,8 +11,8 @@ h2. Running the scenarios
 
 Open a shell in this directory (java) and execute the following command:
 
-<pre><code>
-ant
-</code></pre>
+<pre>ant</pre>
 
 There is a deliberate error. See if you can fix it!
+
+Also see "Cuke4Duke":http://wiki.github.com/aslakhellesoy/cuke4duke for more powerful JVM and Java support for Cucumber.

data/lib/cucumber/rb_support/regexp_argument_matcher.rb

@@ -9,7 +9,8 @@ module Cucumber
           n = 0
           match.captures.map do |val|
             n += 1
-            StepArgument.new(val, match.offset(n)[0])
+            start = match.offset(n)[0]
+            StepArgument.new(val, start)
           end
         else
           nil

data/lib/cucumber/step_match.rb

@@ -55,6 +55,7 @@ module Cucumber
       s = string.dup
       offset = 0
       step_arguments.each do |step_argument|
+        next if step_argument.pos.nil?
         replacement = if block_given?
           proc.call(step_argument.val)
         elsif Proc === format

data/lib/cucumber/version.rb

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

data/spec/cucumber/rb_support/regexp_argument_matcher_spec.rb

@@ -8,6 +8,11 @@ module Cucumber
         arguments = RegexpArgumentMatcher.arguments_from(/I (\w+) (\w+)/, "I like fish")
         arguments.map{|argument| [argument.val, argument.pos]}.should == [["like", 2], ["fish", 7]]
       end
+
+      it "should create 2 arguments when first group is optional" do
+        arguments = RegexpArgumentMatcher.arguments_from(/should( not)? be flashed '([^']*?)'$/, "I should be flashed 'Login failed.'")
+        arguments.map{|argument| [argument.val, argument.pos]}.should == [[nil, nil], ["Login failed.", 21]]
+      end
     end
   end
 end

data/spec/cucumber/step_match_spec.rb

@@ -31,5 +31,10 @@ module Cucumber
       m = stepdef(/I (\w+) (\d+) (\w+) this (\w+)/).step_match("I ate 1 egg this morning", nil)
       m.format_args(lambda{|m| "<span>#{m}</span>"}).should == "I <span>ate</span> <span>1</span> <span>egg</span> this <span>morning</span>"
     end
+
+    it "should format groups even when first group is optional and not matched" do
+      m = stepdef(/should( not)? be flashed '([^']*?)'$/).step_match("I should be flashed 'Login failed.'", nil)
+      m.format_args("<span>%s</span>").should == "I should be flashed '<span>Login failed.</span>'"
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: aslakhellesoy-cucumber
 version: !ruby/object:Gem::Version 
-  version: 0.3.102.1
+  version: 0.3.102.2
 platform: ruby
 authors: 
 - "Aslak Helles\xC3\xB8y"
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-09-23 00:00:00 -07:00
+date: 2009-09-24 00:00:00 -07:00
 default_executable: cucumber
 dependencies: 
 - !ruby/object:Gem::Dependency