Commit 9f8eda18 authored by Phillip Webb's avatar Phillip Webb

Merge pull request #4362 from marcingrzejszczak/issues/config-props-before-default

* pr/4362:
  Add config properties before default properties
parents 18db4f6c 19f141dc
......@@ -569,9 +569,20 @@ public class ConfigFileApplicationListener implements EnvironmentPostProcessor,
for (PropertySource<?> item : sources) {
reorderedSources.add(item);
}
// Maybe we should add before the DEFAULT_PROPERTIES if it exists?
this.environment.getPropertySources()
.addLast(new ConfigurationPropertySources(reorderedSources));
addConfigurationProperties(
new ConfigurationPropertySources(reorderedSources));
}
private void addConfigurationProperties(
ConfigurationPropertySources configurationSources) {
MutablePropertySources existingSources = this.environment
.getPropertySources();
if (existingSources.contains(DEFAULT_PROPERTIES)) {
existingSources.addBefore(DEFAULT_PROPERTIES, configurationSources);
}
else {
existingSources.addLast(configurationSources);
}
}
}
......
......@@ -765,6 +765,18 @@ public class ConfigFileApplicationListenerTests {
assertThat(property, equalTo("false"));
}
@Test
public void addBeforeDefaultProperties() throws Exception {
MapPropertySource defaultSource = new MapPropertySource("defaultProperties",
Collections.<String, Object>singletonMap("the.property",
"fromdefaultproperties"));
this.environment.getPropertySources().addFirst(defaultSource);
this.initializer.setSearchNames("testproperties");
this.initializer.postProcessEnvironment(this.environment, this.application);
String property = this.environment.getProperty("the.property");
assertThat(property, equalTo("frompropertiesfile"));
}
private static Matcher<? super ConfigurableEnvironment> containsPropertySource(
final String sourceName) {
return new TypeSafeDiagnosingMatcher<ConfigurableEnvironment>() {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment