- Notifications
You must be signed in to change notification settings - Fork45
Added ForceStaticMockListener to run "configure mockito-inline" action.#438
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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.
Have several minor points to discussion, sent them to Nikita.
Approve changes anyway as we should merge this changes tomain as soon as possible.
7911545 to916623cCompareThere 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.
LGTM
Refactored url listener in notifications.
916623c to0ee36a1Compare
Uh oh!
There was an error while loading.Please reload this page.
Description
Added ForceStaticMockListener to run "configure mockito-inline" action.
Refactored url listener in notifications to support callbacks (to encapsulate GenerateTestsDialogWindow).
Fixes#447
Type of Change
Please delete options that are not relevant.
How Has This Been Tested?
Automated Testing
Not applicable.
Manual Scenario
Warning with link to an action with "configure mockito-inline" must be available.
Mockito-inline must be installed.
Tests with static mocks should be generated.
Run tests, no errors must be shown.
Checklist (remove irrelevant options):