Skip to content

Commit b61f4e9

Browse files
authored
Fix: Coverage not reported correctly when running connected tests using testOrchestrator (#118)
1 parent dc9f71c commit b61f4e9

File tree

3 files changed

+36
-11
lines changed

3 files changed

+36
-11
lines changed

plugin/src/main/kotlin/org/neotech/plugin/rootcoverage/JaCoCoConfiguration.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,16 +66,16 @@ internal fun Project.getExecutionDataFileTree(includeUnitTestResults: Boolean, i
6666
// buildFolderPatterns.add("outputs/code_coverage/*/connected/*coverage.ec")
6767

6868
// Android Build Tools Plugin 7.1+
69-
buildFolderPatterns.add("outputs/code_coverage/*/connected/*/coverage.ec")
69+
buildFolderPatterns.add("outputs/code_coverage/*/connected/*/*.ec")
7070
}
7171
if(includeGradleManagedDevicesResults) {
7272
// Gradle Managed Devices 7.4
7373
// buildFolderPatterns.add("outputs/managed_device_code_coverage/*/coverage.ec")
7474

7575
// Gradle Managed Devices 8.3+
76-
buildFolderPatterns.add("outputs/managed_device_code_coverage/*/*/coverage.ec")
76+
buildFolderPatterns.add("outputs/managed_device_code_coverage/*/*/*.ec")
7777
// In case of flavors coverage is nested an additional 2 folder deeper
78-
buildFolderPatterns.add("outputs/managed_device_code_coverage/*/flavors/*/*/coverage.ec")
78+
buildFolderPatterns.add("outputs/managed_device_code_coverage/*/flavors/*/*/*.ec")
7979
}
8080
return if(buildFolderPatterns.isEmpty()) {
8181
null

plugin/src/test/test-fixtures/multi-module/configurations/connected-device-clear-package-data.yaml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,5 @@
11
# This test was added in reaction to:
22
# https://github.com/NeoTech-Software/Android-Root-Coverage-Plugin/issues/83
3-
#
4-
# Currently this test case fails, but probably not because of this plugin as it seems to be
5-
# an Android tooling issue:
6-
# - https://issuetracker.google.com/issues/126258801
7-
# - https://issuetracker.google.com/issues/123987001
8-
#
9-
# For now this test configuration is ignored (until Google fixes this)
10-
ignore: true
113
projectConfiguration:
124
addGradleManagedDevice: false
135
clearPackageData: true
Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
projectConfiguration:
2+
addGradleManagedDevice: true
3+
clearPackageData: true
4+
testOrchestrator: true
5+
pluginConfiguration:
6+
properties:
7+
- name: generateHtml
8+
value: true
9+
- name: generateXml
10+
value: false
11+
- name: generateCsv
12+
value: true
13+
14+
- name: buildVariant
15+
value: debug
16+
- name: buildVariantOverrides
17+
value:
18+
":library_android_flavors": "demoDebug"
19+
20+
- name: executeTests
21+
value: true
22+
23+
- name: includeUnitTestResults
24+
value: true
25+
- name: includeAndroidTestResults
26+
value: true
27+
- name: includeNoLocationClasses
28+
value: true
29+
30+
- name: runOnGradleManagedDevices
31+
value: true
32+
- name: gradleManagedDeviceName
33+
value: nexusoneapi30

0 commit comments

Comments
 (0)