\n has been ignored

jnicklas · Jun 8, 2016 · 79b9115 · 79b9115
1 parent 02f743d
commit 79b9115
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/turnip/placeholder.rb b/lib/turnip/placeholder.rb
@@ -77,7 +77,7 @@ def find_match(value)
       #  %{'Jhon Doe'}
       #
       # In any case, it passed to the step block in the state without quotes
-      return @default, value.sub(/^(["'])(.*)\1$/, '\2')
+      return @default, value.sub(/^(["'])([^\1]*)\1$/, '\2')
     end
 
     def placeholder_matches

diff --git a/spec/placeholder_spec.rb b/spec/placeholder_spec.rb
@@ -105,6 +105,9 @@
       expect(placeholder.apply('John Doe')).to eq('John Doe')
       expect(placeholder.apply('"John Doe"')).to eq('John Doe')
       expect(placeholder.apply('\'John Doe\'')).to eq('John Doe')
+
+      expect(placeholder.apply('John \n Doe')).to eq('John \n Doe')
+      expect(placeholder.apply('"John \n Doe"')).to eq('John \n Doe')
     end
   end