6 Commits (0c7f4d8307d363a7af7b751a78a29b9844acf3dc)

Author	SHA1	Message	Date
Phillip Webb	3f7bf7d34f	Modify SpringApplication Environment rather than setting it ... Update `SpringBootContextLoader` so that when possible the `SpringApplication` remains in control of creating the `Environment` instance. Prior to this commit, we would always create the `Environment` in the `SpringBootContextLoader` and then call `setEnvironment` on the `SpringApplication`. This meant that the `ApplicationEnvironment` classes were not used and that `isCustomEnvironment` was set to `true` so no conversion was applied. With the updated code, an `ApplicationListener` is used to mutate the `Environment` instance and add the required test property sources. Fixes gh-29169	3 years ago
Phillip Webb	eb6b48fff0	Use side-effect free environment with tests rather than converting ... Refine the logic introduced in 64270eca to use a side-effect free Environment implementation rather than converting the Environment early. Early conversion can cause condition evaluation issues if `src/test/resources/application.properties` files are bound to the `SpringApplication`. Specifically the `spring.main.web-application-type` property can change the `Environment` type which must happen before conditions are evaluated. Fixes gh-29169	3 years ago
Madhura Bhave	64270eca51	Convert environment used by SpringBootTestContextLoader ... This commit aligns `SpringBootTest`s to also use `ApplicationEnvironment` instead of `StandardEnvironment`. This prevents the side-effect of active profiles from `@ActiveProfiles` from being added to the environment when doGetActiveProfiles is called. In this case, calling `addActiveProfiles()` in the environment post processor would result in `@ActiveProfiles` being added to the environment first, resulting in the wrong order. The additional call to `setActiveProfiles()` is also not necessary when using ApplicationEnvironment because that call was put in place to prevent the side-effect which `ApplicationEnvironment` does not have. Fixes gh-28530	3 years ago
Phillip Webb	3352024b1c	Provide ConfigFileApplicationListener replacement ... Deprecate `ConfigFileApplicationListener` and provide a replacement mechanism that supports arbitrary config data imports. This commit updates the following areas: - Extract `EnvironmentPostProcessor` invocation logic from the `ConfigFileApplicationListener` to new dedicated listener. Also providing support for `Log` injection. - Extract `RandomPropertySource` adding logic from the `ConfigFileApplicationListener` to a dedicated class. - Migrate to the recently introduced `DefaultPropertiesPropertySource` class when moving the defaultProperties `PropertySource` - Replace processing logic with a phased approach to ensure that profile enablement happens in a distinct phase and that profiles can no longer be activated on an ad-hoc basis. - Provide a more predictable and logical import order for processing `application.properties` and `application.yml` files. - Add support for a `spring.config.import` property which can be used to import additional config data. Also provide a pluggable API allowing third-parties to resolve and load locations themselves. - Add `spring.config.activate.on-profile` support which replaces the existing `spring.profiles` property. - Add `spring.config.activate.on-cloud-platform` support which allows a config data document to be active only on a given cloud platform. - Support a `spring.config.use-legacy-processing` property allowing the previous processing logic to be used. Closes gh-22497 Co-authored-by: Madhura Bhave <mbhave@vmware.com>	4 years ago
Phillip Webb	c3816bfe7b	Polish output capture names	5 years ago
Madhura Bhave	d9466f5659	Move smoke tests under spring-boot-test ... See gh-17393	5 years ago