[x] Required existing tests pass (ie full_test.sh, others as appropriate)
[x] Required unit tests added and pass for new/modified functionality
[ ] NO REQUIRED integration tests (Integration tests not checked but should pass as changes are confined to unit-level functionality)
[ ] NO REQUIRED documentation (No additional documentation needed)
[x] Required release notes added for new/modified functionality
[ ] NO REQUIRED updates to other repos (No updates to other repos were needed)
Related Issues
fixes NRLMMD-GEOIPS/geoips#469
Summary
Fixed an issue where LogWithEmphasis could not handle an empty string. It now correctly processes a set of messages that includes an empty string, and appropriately errors out if only an empty string is provided.
Added a unit test to ensure LogWithEmphasis correctly handles empty strings within message sets.
Note
I accidentally committed the dockerfile from #452 before it was merged. I can fix that when we merge with main.
Related Issues
fixes NRLMMD-GEOIPS/geoips#469
Summary
LogWithEmphasis
could not handle an empty string. It now correctly processes a set of messages that includes an empty string, and appropriately errors out if only an empty string is provided.LogWithEmphasis
correctly handles empty strings within message sets.Note