eeverman / eeverman/andhow

Add default value testing to bulk tests

Open
#728 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
25
Forks
34
PR merge metrics
No merged PRs in 30d

Description

This is related to #695 and #676.

The current bulk tests only test the getExplicitValue() method, not defaults. This makes the tests a bit counterintuitive (shouldn't that property never return null??) and also misses an entire aspect of the testing.

There are two flavors of loaders: Trimming and NonTrimming, which leads to two possible explicit values for each property. Adding another response value to test would seem to increase the number of values to test to 4 (2 x 2), however, it is simpler than that. Since the default value is only relevant when the explicit value is null, a single default value can be specified to apply to any null explicit value. Additionally, that it can only be required of the builder when one of the explicit values is null.

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.