adobe / adobe/aepsdk-core-android
Increase code coverage for listed classes
- Dominant language
- Kotlin
- Stars
- 17
- Forks
- 30
- Avg merge
- 8h 37m
- Merged PRs (30d)
- 5
Description
To set code coverage to 85% and make future PRs have good code coverage, we have to exclude the following classes for now that are not covered properly in tests. We should remove those classes from the ignore list and add more tests for them later.
```
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/AppResourceStore.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/eventhub/history"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/eventhub/ExtensionContainer.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/util/EventDataMerger.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/util/FileUtils.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/util/MapUtils.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/util/VisitorIDSerializer.java"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/configuration/ConfigurationRulesManager.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/internal/configuration/ConfigurationStateManager.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/launch/rulesengine/json/HistoricalCondition.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/launch/rulesengine/json/JSONConsequence.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/launch/rulesengine/json/JSONDefinition.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/launch/rulesengine/json/JSONRule.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/launch/rulesengine/json/JSONRuleRoot.kt"
- "code/core/src/main/java/com/adobe/marketing/mobile/ExtensionHelper.java"
```
Contributor guide
Research direction
Start by locating the coverage ignore configuration containing the listed Kotlin and Java classes, then review their existing tests. Remove the listed exclusions, add tests for the uncovered behavior, and run the project's coverage checks to verify coverage reaches 85%.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java, kotlin
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100