RUM-1517 Add test pyramid scaffolding#1674
Conversation
ca1db6b to
4894f88
Compare
Codecov Report
@@ Coverage Diff @@
## develop #1674 +/- ##
===========================================
+ Coverage 83.58% 83.73% +0.15%
===========================================
Files 459 459
Lines 15778 15775 -3
Branches 2354 2353 -1
===========================================
+ Hits 13187 13208 +21
+ Misses 1956 1938 -18
+ Partials 635 629 -6 |
| test-pyramid:single-fit-logs: | ||
| tags: [ "runner:main" ] | ||
| image: $CI_IMAGE_DOCKER | ||
| stage: test |
There was a problem hiding this comment.
should it be test-pyramid here and below? Or it should stay test until this stage is completely replaced by test-pyramid?
There was a problem hiding this comment.
Is empty manifest needed? I guess since we are using namespace in build.gradle.kts, it is not useful. Same applies to other manifest files added in this PR.
There was a problem hiding this comment.
It is needed or the compiler complains
| api(project(":dd-sdk-android-core")) | ||
| api(project(":features:dd-sdk-android-logs")) |
There was a problem hiding this comment.
shouldn't it be implementation here? api is for re-export, which is not needed.
| api(project(":dd-sdk-android-core")) | ||
| api(project(":features:dd-sdk-android-rum")) |
There was a problem hiding this comment.
| api(project(":dd-sdk-android-core")) | |
| api(project(":features:dd-sdk-android-rum")) | |
| implementation(project(":dd-sdk-android-core")) | |
| implementation(project(":features:dd-sdk-android-rum")) |
| api(project(":dd-sdk-android-core")) | ||
| api(project(":features:dd-sdk-android-trace")) |
There was a problem hiding this comment.
| api(project(":dd-sdk-android-core")) | |
| api(project(":features:dd-sdk-android-trace")) | |
| implementation(project(":dd-sdk-android-core")) | |
| implementation(project(":features:dd-sdk-android-trace")) |
4 participants
What does this PR do?
Creates the scaffolding for our Single Feature Integration Test level from our Test Pyramid strategy.