avaje / avaje-config

Application configuration / properties loading for JVM applications
https://avaje.io/config
Apache License 2.0
55 stars 8 forks source link

Change so that properties via command line arguments is ALWAYS read (it wasn't when test resources detected) #37

Closed rbygrave closed 2 years ago

rbygrave commented 2 years ago

When test resources where detected (application-test.yaml, application-test.properties) then the command line arguments were not being read / checked.

This prevents the case where we run something in src/test (usually a test) and provide properties configuration via command line arguments. This is a pretty rare scenario, in fact I don't think anyone has hit this scenario yet but I think the existing behaviour is confusing.

Changing to: Always going to read command line arg properties.