-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error parsing background #146
Comments
Suspect it's a bug. If you insert a blank line it will work.
This is probably a left over from an earlier version of Yadda when Scenarios could have descriptions. Please leave open and I'll investigate a bit further |
The fix is easy, but want to get some feedback on whether anyone finds background descriptions useful. I've added the following console.log to the latest version of yadda.
If no-ones raised any concerns in the next month I'll bin them. |
You mean bin them as in remove the background feature entirely? I don't mind the extra line of whitespace, but to do without background would be a pain. |
No, the background feature is staying. Earlier on in Yadda's lifetime I misunderstood a feature request and added descriptions to scenarios.
The consequence was that you needed to add a blank line before the steps so Yadda could tell the difference between descriptions and step text. It caused no end of problems so I removed it. Looks like I forgot to remove it from Backgrounds |
IMHO, background description isn't necessary because the background steps are more or less the descriptions. |
With https://gist.github.com/3e9f6ff73d5caf31acdb, the parser fails with
the background does have a step though. Commenting out line 4+5 makes the error go away.
The text was updated successfully, but these errors were encountered: