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

Run Gradle Android tests conditionally when sdk path is defined #3014

Merged
merged 1 commit into from
Aug 25, 2020

Conversation

arturbosch
Copy link
Member

@arturbosch arturbosch added this to the 1.12.0 milestone Aug 25, 2020
@codecov
Copy link

codecov bot commented Aug 25, 2020

Codecov Report

Merging #3014 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3014   +/-   ##
=========================================
  Coverage     79.53%   79.53%           
  Complexity     2532     2532           
=========================================
  Files           431      431           
  Lines          7637     7637           
  Branches       1447     1447           
=========================================
  Hits           6074     6074           
  Misses          795      795           
  Partials        768      768           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e03bd9...f7a389c. Read the comment docs.

@arturbosch arturbosch added the housekeeping Marker for housekeeping tasks and refactorings label Aug 25, 2020
describe("detekt plugin - Android") {
describe(
"detekt plugin - Android",
skip = if (System.getenv("ANDROID_SDK_ROOT") != null) Skip.No else Skip.Yes("No android sdk.")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to say that "normal" environment is ANDROID_HOME but I just found that they deprecated that environment. A thing that I learned today. And a thing that I need to change in all my environments too...

Copy link
Member Author

@arturbosch arturbosch Aug 25, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the one got reported when the build crashed for me locally xD.
One day I will code some Android too 8-)

@arturbosch arturbosch merged commit cd91fb8 into master Aug 25, 2020
@arturbosch arturbosch deleted the conditional-android-tests branch August 25, 2020 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Marker for housekeeping tasks and refactorings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants