Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

url should match with placeholder #99

Closed
jaymecd opened this issue Aug 19, 2013 · 2 comments
Closed

url should match with placeholder #99

jaymecd opened this issue Aug 19, 2013 · 2 comments

Comments

@jaymecd
Copy link

jaymecd commented Aug 19, 2013

Hello,
Today I've notice unexpected behaviour statement when using value from TableNode:
@Then /^the (?i)url(?-i) should match (?P<pattern>"([^"]|\\")*")$/
It generates broken formatted output:
Then the url should match "<placeholder{-skipped}{+skipped_param}>{-skipped_param}{+skipped}"
instead of expected:
Then the url should match "<placeholder>"

After small research, solution is pretty easy, update regex to exclude nested block of pattern inside quotes as its done on all other text/match patterns:
@Then /^the (?i)url(?-i) should match (?P<pattern>"(?:[^"]|\\")*")$/

Regards,
Nick

stof added a commit that referenced this issue Apr 27, 2014
The inner group needs to be non-capturing. Fixes #99
@stof
Copy link
Member

stof commented Apr 27, 2014

Fixed in v1.3.1

@stof stof closed this as completed Apr 27, 2014
@jaymecd
Copy link
Author

jaymecd commented Apr 27, 2014

Nice, thank you

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants