-
Notifications
You must be signed in to change notification settings - Fork 395
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
Mock helper WIP #61
base: master
Are you sure you want to change the base?
Mock helper WIP #61
Conversation
7839bfb
to
9fb32e8
Compare
This is really helpful. Any progress on this PR ? |
Hey @EQuincerot need any help to complete this feature? This could be really helpful. |
At first quick glance this looks really great! |
It's an interesting idea but I don't really understand the use-case that is motivating this PR. Can you provide a sample test case and show how StepMock would make this type of test easier to implement? |
@nre-ableton Sorry for my potentially "premature" comment, I have admittedly just started looking at jenkins pipeline (unit) testing and stumbled over this (interesting looking) PR... I actually hope/plan to really start using Jenkins Pipeline Unit this month, so maybe with more (or in fact some) experience of really using it, I hope to be able to comment here... |
@reinholdfuereder No problem! My comment was also addressed to anyone else interested in this feature, including the PR author (ping @EQuincerot). |
Preview of mock helper we may have