Skip to content
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

Roadmap for Mockito.MockedStatic<T> #2433

Closed
sfc-gh-vparaschiv opened this issue Sep 23, 2021 · 5 comments · Fixed by #2437
Closed

Roadmap for Mockito.MockedStatic<T> #2433

sfc-gh-vparaschiv opened this issue Sep 23, 2021 · 5 comments · Fixed by #2437

Comments

@sfc-gh-vparaschiv
Copy link

sfc-gh-vparaschiv commented Sep 23, 2021

Hello.

What is the current roadmap for Mockito.MockedStatic<T>?

Is it feature complete, or there are new features planned for it? Any other changes expected, before graduating?

How stable is the current implementation?

When is it expected for it to graduate out of incubation? Will it ever?

What work should be done and in general what else needs to happen to reach that? Maybe the community can help with that work.

Thanks a lot for your open-source contributions!
--Vasile


P.S. Don't know if these questions should be answered by @raphw, who seems to be the author of Mockito.MockedStatic<T> based on github history, or somebody else.

@raphw
Copy link
Member

raphw commented Sep 24, 2021

They are stable but we did not know that they covered all cases appropriately when we released the mock maker. I do not personally think that we need further changes and I consider them as safe to use. We just discussed this internally and I think it is likely that we simply remove the annotations from any API that we consider stable.

@sfc-gh-vparaschiv
Copy link
Author

They are stable but we did not know that they covered all cases appropriately when we released the mock maker. I do not personally think that we need further changes and I consider them as safe to use. We just discussed this internally and I think it is likely that we simply remove the annotations from any API that we consider stable.

@raphw, thanks a bunch for your prompt response.

Ball-pack wise, would you expect that to happen in days, weeks, or months?

o( _ _ )o

@raphw
Copy link
Member

raphw commented Sep 24, 2021

I'd say weeks, but the decision seems pretty solid so I would not worry about the annotation still being present.

@TimvdLippe
Copy link
Contributor

Yeah I will prepare a PR this week to promote all incubating APIs to stable, included in Mockito 4.0.

@sfc-gh-vparaschiv
Copy link
Author

Thanks a lot @raphw, @TimvdLippe, @mockitoguy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants