adobe / adobe/aepsdk-core-android

Increase code coverage for listed classes

Open
#398 0 comments 0 reactions 0 assignees View on GitHub
defer
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.