Skip to content

Commit 742df2c

Browse files
committed
Polish BCrypt upgrade
Issue gh-7042
1 parent d3d6a87 commit 742df2c

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

crypto/src/test/java/org/springframework/security/crypto/password/DelegatingPasswordEncoderTests.java

+12-2
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828

2929
import static org.assertj.core.api.Assertions.assertThat;
3030
import static org.assertj.core.api.Assertions.assertThatThrownBy;
31+
import static org.mockito.ArgumentMatchers.any;
3132
import static org.mockito.Mockito.verify;
3233
import static org.mockito.Mockito.verifyZeroInteractions;
3334
import static org.mockito.Mockito.when;
@@ -215,8 +216,17 @@ public void upgradeEncodingWhenIdInvalidFormatThenTrue() {
215216
}
216217

217218
@Test
218-
public void upgradeEncodingWhenSameIdThenEncoderDecides() {
219-
this.passwordEncoder.upgradeEncoding(this.bcryptEncodedPassword);
219+
public void upgradeEncodingWhenSameIdAndEncoderFalseThenEncoderDecidesFalse() {
220+
assertThat(this.passwordEncoder.upgradeEncoding(this.bcryptEncodedPassword)).isFalse();
221+
222+
verify(bcrypt).upgradeEncoding(this.encodedPassword);
223+
}
224+
225+
@Test
226+
public void upgradeEncodingWhenSameIdAndEncoderTrueThenEncoderDecidesTrue() {
227+
when(this.bcrypt.upgradeEncoding(any())).thenReturn(true);
228+
229+
assertThat(this.passwordEncoder.upgradeEncoding(this.bcryptEncodedPassword)).isTrue();
220230

221231
verify(bcrypt).upgradeEncoding(this.encodedPassword);
222232
}

0 commit comments

Comments
 (0)