Skip to content

Commit 33b62bd

Browse files
committed
Rebase and Fix Formatting
Issue gh-16604 Signed-off-by: Josh Cummings <[email protected]>
1 parent f12ebdf commit 33b62bd

File tree

3 files changed

+13
-9
lines changed

3 files changed

+13
-9
lines changed

config/src/test/java/org/springframework/security/SpringSecurityCoreVersionSerializableTests.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -239,9 +239,9 @@
239239
import org.springframework.security.web.webauthn.api.PublicKeyCredentialUserEntity;
240240
import org.springframework.security.web.webauthn.api.TestAuthenticationAssertionResponses;
241241
import org.springframework.security.web.webauthn.api.TestBytes;
242-
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentials;
243242
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentialRequestOptions;
244243
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentialUserEntities;
244+
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentials;
245245
import org.springframework.security.web.webauthn.api.UserVerificationRequirement;
246246
import org.springframework.security.web.webauthn.authentication.WebAuthnAuthentication;
247247
import org.springframework.security.web.webauthn.authentication.WebAuthnAuthenticationRequestToken;

web/src/test/java/org/springframework/security/web/webauthn/management/JdbcUserCredentialRepositoryTests.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ void saveCredentialRecordWhenSaveThenReturnsSaved() {
136136

137137
@Test
138138
void saveCredentialRecordWhenRecordExistsThenReturnsUpdated() {
139-
CredentialRecord userCredential = TestCredentialRecord.fullUserCredential().build();
139+
CredentialRecord userCredential = TestCredentialRecords.fullUserCredential().build();
140140
this.jdbcUserCredentialRepository.save(userCredential);
141141
// @formatter:off
142142
CredentialRecord updatedRecord = ImmutableCredentialRecord.fromCredentialRecord(userCredential)

web/src/test/java/org/springframework/security/web/webauthn/management/Webauthn4jRelyingPartyOperationsTests.java

+11-7
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,9 @@
5959
import org.springframework.security.web.webauthn.api.PublicKeyCredentialUserEntity;
6060
import org.springframework.security.web.webauthn.api.TestAuthenticatorAttestationResponses;
6161
import org.springframework.security.web.webauthn.api.TestCredentialRecords;
62-
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentials;
6362
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentialCreationOptions;
6463
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentialUserEntities;
64+
import org.springframework.security.web.webauthn.api.TestPublicKeyCredentials;
6565
import org.springframework.security.web.webauthn.api.UserVerificationRequirement;
6666

6767
import static org.assertj.core.api.Assertions.assertThat;
@@ -305,7 +305,8 @@ void registerCredentialWhenCTypeIsNotWebAuthn() {
305305
AuthenticatorAttestationResponseBuilder responseBldr = TestAuthenticatorAttestationResponses
306306
.createAuthenticatorAttestationResponse()
307307
.clientDataJSON(new Bytes(invalidTypeClientDataJSON.getBytes(StandardCharsets.UTF_8)));
308-
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build()).build();
308+
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build())
309+
.build();
309310
ImmutableRelyingPartyRegistrationRequest registrationRequest = new ImmutableRelyingPartyRegistrationRequest(
310311
options, new RelyingPartyPublicKey(publicKey, this.label));
311312
assertThatRuntimeException().isThrownBy(() -> this.rpOperations.registerCredential(registrationRequest))
@@ -327,7 +328,8 @@ void registerCredentialWhenCChallengeNotEqualBase64UrlEncodingOptionsChallenge()
327328
.build();
328329
AuthenticatorAttestationResponseBuilder responseBldr = TestAuthenticatorAttestationResponses
329330
.createAuthenticatorAttestationResponse();
330-
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build()).build();
331+
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build())
332+
.build();
331333
ImmutableRelyingPartyRegistrationRequest registrationRequest = new ImmutableRelyingPartyRegistrationRequest(
332334
options, new RelyingPartyPublicKey(publicKey, this.label));
333335

@@ -350,7 +352,8 @@ void registerCredentialWhenCOriginNotExpected() {
350352
.build();
351353
AuthenticatorAttestationResponseBuilder responseBldr = TestAuthenticatorAttestationResponses
352354
.createAuthenticatorAttestationResponse();
353-
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build()).build();
355+
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build())
356+
.build();
354357
ImmutableRelyingPartyRegistrationRequest registrationRequest = new ImmutableRelyingPartyRegistrationRequest(
355358
options, new RelyingPartyPublicKey(publicKey, this.label));
356359

@@ -375,7 +378,8 @@ void registerCredentialWhenClientDataJSONDoesNotMatchHash() {
375378
.build();
376379
AuthenticatorAttestationResponseBuilder responseBldr = TestAuthenticatorAttestationResponses
377380
.createAuthenticatorAttestationResponse();
378-
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build()).build();
381+
PublicKeyCredential publicKey = TestPublicKeyCredentials.createPublicKeyCredential(responseBldr.build())
382+
.build();
379383
ImmutableRelyingPartyRegistrationRequest registrationRequest = new ImmutableRelyingPartyRegistrationRequest(
380384
options, new RelyingPartyPublicKey(publicKey, this.label));
381385

@@ -570,8 +574,8 @@ void createCredentialRequestOptionsWhenAuthenticated() {
570574
UserDetails user = PasswordEncodedUser.user();
571575
UsernamePasswordAuthenticationToken auth = new UsernamePasswordAuthenticationToken(user, null,
572576
user.getAuthorities());
573-
PublicKeyCredentialUserEntity userEntity = TestPublicKeyCredentialUserEntity.userEntity().build();
574-
CredentialRecord credentialRecord = TestCredentialRecord.userCredential().build();
577+
PublicKeyCredentialUserEntity userEntity = TestPublicKeyCredentialUserEntities.userEntity().build();
578+
CredentialRecord credentialRecord = TestCredentialRecords.userCredential().build();
575579
given(this.userEntities.findByUsername(user.getUsername())).willReturn(userEntity);
576580
given(this.userCredentials.findByUserId(userEntity.getId())).willReturn(Arrays.asList(credentialRecord));
577581
PublicKeyCredentialRequestOptionsRequest createRequest = new ImmutablePublicKeyCredentialRequestOptionsRequest(

0 commit comments

Comments
 (0)