keymanapp / keymanapp/keyman

test(developer): add tests for low-coverage modules in kmc

Open
#13,563 0 comments 0 reactions 0 assignees View on GitHub
developer/ good first issue test
Dominant language
Pascal
Stars
534
Forks
143
Avg merge
2d 10h
Merged PRs (30d)
113

Description

The following modules have low coverage, as of 18.0.219-beta. The process for adding coverage is fairly simple: examine the coverage log, pick some uncovered lines, and craft a test accordingly -- look at existing tests for patterns to follow (e.g. message-related tests usually take a fixture which generates the message).

```
[23:04:03] : [Step 2/6] [developer/src/common/web/utils] Coverage thresholds are currently 45%, which is lower than ideal.
[23:04:53] : [Step 2/6] [developer/src/kmc-analyze] Coverage thresholds are currently 70%, which is lower than ideal.
[23:05:03] : [Step 2/6] [developer/src/kmc-copy] Coverage thresholds are currently 75%, which is lower than ideal.
[23:05:37] : [Step 2/6] [developer/src/kmc-kmn] Coverage thresholds are currently 80%, which is lower than ideal.
[23:05:50] : [Step 2/6] [developer/src/kmc-ldml] Coverage thresholds are currently 75%, which is lower than ideal.
[23:06:12] : [Step 2/6] [developer/src/kmc-model-info] Coverage thresholds are currently 55%, which is lower than ideal.
[23:06:42] : [Step 2/6] [developer/src/kmc] Coverage thresholds are currently 50%, which is lower than ideal.
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.