-
Notifications
You must be signed in to change notification settings - Fork 31
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
Bind mocks by constructor #104
Comments
+1 |
Sorry for the very very late reply. |
Oh noes, we've (organisation with 100 devs) been waiting for constructor support, as it lends itself to more idiomatic Kotlin. Please reconsider! 🤞 |
TBH, I don't use Kotlin very much anymore these days, and I maintain this for free, and on my free time. If you or your company would like to fork this library and take over its maintenance in the future, I'd be happy to archive this repo and make it point to your fork. I see that the change is not huge, so I might consider integrating it, but it would be nice if you could step up to fix potential bugs, and help me at least concerning this part, in the future, so that it doesn't become a hassle for me. |
Hi,
currently this project allows to scan fields with
lateinit var
for@MockkBean
and@SpykBean
annotations, would you be interested in PR that extends scanning to constructor parameters?I did a some prototyping and besides changes in
DefinitionsParser
this feature requires Junit 5ParameterResolver to be implemented as well - not sure if you would want it in the same artifact or as a separate one.So, what do you say?
The text was updated successfully, but these errors were encountered: