-
Notifications
You must be signed in to change notification settings - Fork 27.3k
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
Add unit tests for AndroidWorkflow.runLicenseManager() #14260
Conversation
Unit tests for #14249. |
bc142a5
to
93b1556
Compare
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.
LGTM
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.
LGTM
when(sdk.sdkManagerPath).thenReturn('/foo/bar/sdkmanager'); | ||
when(processManager.canRun('/foo/bar/sdkmanager')).thenReturn(false); | ||
|
||
expect(() async => await AndroidWorkflow.runLicenseManager(), throwsToolExit()); |
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.
I think this can just be:
expect(AndroidWorkflow.runLicenseManager(), throwsToolExit());
Same in the tests above.
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.
Good call! Done for the two that throw.
bfbacd3
to
d7c1bec
Compare
Adds unit tests for AndroidWorkflow.runLicenseManager().
d7c1bec
to
7bce252
Compare
Adds unit tests for AndroidWorkflow.runLicenseManager().
Adds unit tests for AndroidWorkflow.runLicenseManager().