Skip to content

Experiment to resolve Gradle issue #14132 with code cov.#180

Open
richard-bogdan-bitrise wants to merge 14 commits intomainfrom
code_cov_experiment
Open

Experiment to resolve Gradle issue #14132 with code cov.#180
richard-bogdan-bitrise wants to merge 14 commits intomainfrom
code_cov_experiment

Conversation

@richard-bogdan-bitrise
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread trace-sdk/build.gradle
// Replace the original dependsOn with the filtered
connectedAndroidTestTask.setDependsOn(deps)

// Finally make sure that each report task depends on the connected tests.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this looks like it's failing the build as it's trying to run the connected tests as well i'm not sure this is accurate - we run the unit tests in the build-trace_sdk workflow and this now fails.

Comment thread bitrise.yml
- module: trace-sdk
- arguments: "--stacktrace"
title: android_build
- gradle-runner@1:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not sure removing the unit tests here is the best idea - possibly just revert the previous workflow step and run the unit tests without coverage? I presume the unit tests then get run later with the coverage at the same time the instrumentation tests run?

Base automatically changed from code_cov to main April 9, 2021 16:03
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 this pull request may close these issues.

2 participants