-
Notifications
You must be signed in to change notification settings - Fork 220
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
Reusable step functions #534
Conversation
Codecov Report
@@ Coverage Diff @@
## master #534 +/- ##
==========================================
- Coverage 92.65% 92.43% -0.22%
==========================================
Files 48 49 +1
Lines 1660 1679 +19
Branches 207 208 +1
==========================================
+ Hits 1538 1552 +14
- Misses 84 90 +6
+ Partials 38 37 -1
Continue to review full report at Codecov.
|
Items in fixturemanager._arg2fixturedefs are LIFO, not FIFO
And fix tests, remove useless ones.
I believe this introduced a regression when a specific step definition shadows one with a parser - see #542. |
…ions Reusable step functions
…ions Reusable step functions
This PR fixes some bugs happening when a step function is decorated multiple times with
@given
(or when, then):converters
andtarget_fixture
every at every application.Fixes #525