LeaseAwareVaultPropertySource assertion mixup between variables and text

[smurf667]

Looking at release 3.2.0, it seems to me that here text and variable have been mixed up:

spring-vault/spring-vault-core/src/main/java/org/springframework/vault/core/env/LeaseAwareVaultPropertySource.java

Lines 141 to 142 in 58842dc

	Assert.notNull(secretLeaseContainer, "Path name must contain at least one character");
	Assert.notNull(requestedSecret, "SecretLeaseContainer must not be null");

Should be a simple thing of switch either the text or the variables around.

[mp911de]

Good catch. Do you want to submit a pull request to fix the issue or should we do that? Generally speaking, we don't test assertions an hence the change will be a tiny one.

[smurf667]

Hi @mp911de - I think might have to jump through some hoops (CLA/DCO?) and would be glad if someone empowered enough could just do this minor change, maybe as an aside on an otherwise already open branch? If you ask me to, I'll do it, but it's really just these two lines...

[mp911de]

Alright, that's fixed now.