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.

Sign up for GitHub

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

Jump to bottom

Provide a new API for verifying static mock and deprecate old for Mockito #832

Closed
thekingn0thing opened this issue Aug 12, 2017 · 0 comments
Closed

Provide a new API for verifying static mock and deprecate old for Mockito #832

thekingn0thing opened this issue Aug 12, 2017 · 0 comments
Labels
component:mockito enhancement
Milestone
PowerMock 1.7.x

Comments

@thekingn0thing
Copy link
Member

The current API for verifying static is inconsistent with Mockito Verify API and cannot be implemented with using only Mockito public API (mockito/mockito#1110).

verifyStatic();
ClassWithStatic.callStatic();

verifyStatic();
SecondClassWithStatic.callStatic();

As result, it should be replaced with a new one

verifyStatic(ClassWithStatic.class);
ClassWithStatic.callStatic();

The current API have to marked as deprecated in versions 1.7.x and a new one should be provided.
@thekingn0thing thekingn0thing added this to the PowerMock 1.7.x milestone Aug 12, 2017
thekingn0thing pushed a commit that referenced this issue Aug 12, 2017
Fixes #832 Provide a new API for verifying static mock and deprecate …
069e795 
…old for Mockito

Signed-off-by: Arthur Zagretdinov <arthur.zagretdinov@outlook.com>
@thekingn0thing thekingn0thing mentioned this issue Aug 12, 2017
Merged
thekingn0thing pushed a commit that referenced this issue Aug 12, 2017
@thekingn0thing
Fixes #832 Provide a new API for verifying static mock and deprecate …
62c8fbb 
…old for Mockito

Signed-off-by: Arthur Zagretdinov <arthur.zagretdinov@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component:mockito enhancement
Projects
None yet
Milestone
PowerMock 1.7.x
Development

No branches or pull requests
1 participant
@thekingn0thing