apache / apache/maven-mapping

DEFAULT_FILE_NAME_MAPPING constant is never tested

Open Beginner friendly
#63 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
2
Forks
7
Avg merge
11h 11m
Merged PRs (30d)
4

Description

## Description

The `DEFAULT_FILE_NAME_MAPPING` constant is defined at `MappingUtils.java:72` but is never referenced in any test. Only `DEFAULT_FILE_NAME_MAPPING_CLASSIFIER` is tested (in `MappingUtilsTest.java`).

## Risk

Regressions in the default file name mapping expression would go completely undetected by the test suite.

## Suggestion

Add a test case that exercises `DEFAULT_FILE_NAME_MAPPING` similar to how `mappingWithClassifier` tests `DEFAULT_FILE_NAME_MAPPING_CLASSIFIER`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at DEFAULT_FILE_NAME_MAPPING in MappingUtils.java:72 and review the mappingWithClassifier test in MappingUtilsTest.java. Add a comparable test that exercises the default file name mapping expression, then run the MappingUtilsTest suite. Done means the constant is directly covered by a passing test.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
88/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.