-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[Core] Use UndefinedStepsTracker from provided RuntimeGlue #1019
Conversation
optionalGlue.stepDefinitionMatch("", new Step(null, "Given ", "A", 1, null, null), ENGLISH); | ||
|
||
assertEquals(0x1, runtime.exitStatus()); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the current code base no tests are executed in this test so this won't pass.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase onto the latest master. There have been several significant changes.
Nevermind. I just realized this was a PR from 2016! A belated thanks for your contribution. |
Glue only served to pass the UndefinedStepsTracker between Runtime and Runtime. Decoupling this removes some pointless complexity. The constructor `RuntimeGlue(UndefinedStepsTracker tracker, LocalizedXStreams localizedXStreams)` has been deprecated to avoid breaking existing implementations. The Runtime will not use the provided UndefinedStepsTracker but this is consistent with the behaviour prior to #1019. Third parties interested in undefined steps can use the subscribe their own UndefinedStepsTracker to the event bus.
Glue only served to pass the UndefinedStepsTracker between Runtime and Runtime. Decoupling this removes some pointless complexity. The constructor `RuntimeGlue(UndefinedStepsTracker, LocalizedXStreams)` has been deprecated to avoid breaking existing implementations. The Runtime will not use the provided UndefinedStepsTracker but this is consistent with the behaviour prior to #1019. Third parties interested in undefined steps can use the subscribe their own UndefinedStepsTracker to the event bus.
Glue only served to pass the UndefinedStepsTracker between Runtime and Runtime. Decoupling this removes some pointless complexity. The constructor `RuntimeGlue(UndefinedStepsTracker, LocalizedXStreams)` has been deprecated to avoid breaking existing implementations. The Runtime will not use the provided UndefinedStepsTracker but this is consistent with the behaviour prior to #1019. Third parties interested in undefined steps can use the subscribe their own UndefinedStepsTracker to the event bus.
The exit code is now not connected at all to the |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Passed RuntimeGlue and it's tracker is not connected with Runtime and it's tracker.