KotlinConstructorParametersBinderTests:
- Minor enhancements to unit tests
- Add new constructor for MockConfigurationPropertySource
with map of configs
- Inside of KotlinConstructorParametersBinderTests
, create instances of MockConfigurationPropertySource
with a map of configs
- Use Kotlin property access on objects when possible
- Assert against a whole expected object when validating a data class rather than individual properties of the object (isEquals()
function is built into the data class)
- Added test case to Kotlin singleton object
Testing:
- Ran ./gradlew :spring-boot-project:spring-boot:test
on project with all tests succeeding
Comment From: pivotal-issuemaster
@katielevy1 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.
Comment From: snicoll
@katielevy1 thanks for the PR. Could you please sign the CLA?
Comment From: snicoll
@katielevy1 how is it going? Sorry but we can't merge this PR until you sign the CLA.
Comment From: katielevy1
@snicoll apologies for the delay! I'm working with my company's legal team to get approval to sign the CLA, it is taking longer than I expected.
Comment From: snicoll
@katielevy1 Hi there. Any news?
Comment From: pivotal-issuemaster
@katielevy1 Thank you for signing the Contributor License Agreement!
Comment From: katielevy1
@snicoll finally got the thumbs up from legal, I just signed the contributor agreement
Comment From: snicoll
@katielevy1 thank you for making your first contribution to Spring Boot. I've polished things a bit in bac1016.