Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit e59f9c8

Browse files
committedJan 7, 2025·
Assertions arguments corrected.
1 parent 08d4aaf commit e59f9c8

File tree

2 files changed

+9
-13
lines changed

2 files changed

+9
-13
lines changed
 

‎src/test/java/it/pagopa/swclient/mil/auth/service/RefreshTokensServiceTest.java

+8-12
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,12 @@
2929

3030
import io.quarkus.test.InjectMock;
3131
import io.quarkus.test.junit.QuarkusTest;
32+
import io.smallrye.mutiny.Uni;
3233
import io.smallrye.mutiny.helpers.test.UniAssertSubscriber;
3334
import it.pagopa.swclient.mil.auth.AuthErrorCode;
3435
import it.pagopa.swclient.mil.auth.bean.ClaimName;
3536
import it.pagopa.swclient.mil.auth.bean.GetAccessTokenRequest;
37+
import it.pagopa.swclient.mil.auth.bean.GetAccessTokenResponse;
3638
import it.pagopa.swclient.mil.auth.bean.GrantType;
3739
import it.pagopa.swclient.mil.auth.bean.Scope;
3840
import it.pagopa.swclient.mil.auth.dao.ClientEntity;
@@ -145,7 +147,7 @@ void given_refreshToken_when_allGoesOk_then_getTokens() throws ParseException {
145147
.with(
146148
response -> {
147149
assertEquals(
148-
"eyJraWQiOiJrZXlfbmFtZS9rZXlfdmVyc2lvbiIsImFsZyI6IlJTMjU2In0.eyJzdWIiOiJjbGllbnRfaWQiLCJjbGllbnRJZCI6ImNsaWVudF9pZCIsImNoYW5uZWwiOiJjaGFubmVsIiwiaXNzIjoiaHR0cHM6Ly9taWwtYXV0aCIsImdyb3VwcyI6InJvbGUiLCJ0ZXJtaW5hbElkIjoidGVybWluYWxfaWQiLCJhdWQiOiJodHRwczovL21pbCIsIm1lcmNoYW50SWQiOiJtZXJjaGFudF9pZCIsInNjb3BlIjoic2NvcGUiLCJmaXNjYWxDb2RlIjoiZW5jX2Zpc2NhbF9jb2RlIiwiZXhwIjoxNzE3NjUyLCJhY3F1aXJlcklkIjoiYWNxdWlyZXJfaWQiLCJpYXQiOjE3MTc1OTJ9.AA",
150+
signedJwt,
149151
response.getAccessToken());
150152
},
151153
f -> fail(f));
@@ -211,7 +213,7 @@ void given_refreshCookie_when_allGoesOk_then_getTokens() throws ParseException {
211213
.with(
212214
response -> {
213215
assertEquals(
214-
"eyJraWQiOiJrZXlfbmFtZS9rZXlfdmVyc2lvbiIsImFsZyI6IlJTMjU2In0.eyJzdWIiOiJjbGllbnRfaWQiLCJjbGllbnRJZCI6ImNsaWVudF9pZCIsImNoYW5uZWwiOiJjaGFubmVsIiwiaXNzIjoiaHR0cHM6Ly9taWwtYXV0aCIsImdyb3VwcyI6InJvbGUiLCJ0ZXJtaW5hbElkIjoidGVybWluYWxfaWQiLCJhdWQiOiJodHRwczovL21pbCIsIm1lcmNoYW50SWQiOiJtZXJjaGFudF9pZCIsInNjb3BlIjoic2NvcGUiLCJmaXNjYWxDb2RlIjoiZW5jX2Zpc2NhbF9jb2RlIiwiZXhwIjoxNzE3NjUyLCJhY3F1aXJlcklkIjoiYWNxdWlyZXJfaWQiLCJpYXQiOjE3MTc1OTJ9.AA",
216+
signedJwt,
215217
response.getAccessToken());
216218
},
217219
f -> fail(f));
@@ -277,7 +279,7 @@ void given_refreshCookieAndRefreshToken_when_allGoesOk_then_getTokens() throws P
277279
.with(
278280
response -> {
279281
assertEquals(
280-
"eyJraWQiOiJrZXlfbmFtZS9rZXlfdmVyc2lvbiIsImFsZyI6IlJTMjU2In0.eyJzdWIiOiJjbGllbnRfaWQiLCJjbGllbnRJZCI6ImNsaWVudF9pZCIsImNoYW5uZWwiOiJjaGFubmVsIiwiaXNzIjoiaHR0cHM6Ly9taWwtYXV0aCIsImdyb3VwcyI6InJvbGUiLCJ0ZXJtaW5hbElkIjoidGVybWluYWxfaWQiLCJhdWQiOiJodHRwczovL21pbCIsIm1lcmNoYW50SWQiOiJtZXJjaGFudF9pZCIsInNjb3BlIjoic2NvcGUiLCJmaXNjYWxDb2RlIjoiZW5jX2Zpc2NhbF9jb2RlIiwiZXhwIjoxNzE3NjUyLCJhY3F1aXJlcklkIjoiYWNxdWlyZXJfaWQiLCJpYXQiOjE3MTc1OTJ9.AA",
282+
signedJwt,
281283
response.getAccessToken());
282284
},
283285
f -> fail(f));
@@ -346,15 +348,9 @@ void given_refreshCookieAndRefreshTokenBothNull_when_tokesRefreshIsRequested_the
346348
.setMerchantId("merchant_id")
347349
.setTerminalId("terminal_id");
348350

349-
assertThrows(
350-
NullPointerException.class,
351-
() -> refreshTokensService.process(request)
352-
.subscribe()
353-
.with(
354-
i -> {
355-
},
356-
f -> {
357-
}));
351+
Uni<GetAccessTokenResponse> resp = refreshTokensService.process(request);
352+
353+
assertThrows(NullPointerException.class, resp::await);
358354
}
359355

360356
/**

‎src/test/java/it/pagopa/swclient/mil/auth/service/TokenByPasswordServiceTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ void given_userCredentials_when_allGoesOk_then_getAccessToken() throws ParseExce
140140
.with(
141141
response -> {
142142
assertEquals(
143-
"eyJraWQiOiJrZXlfbmFtZS9rZXlfdmVyc2lvbiIsImFsZyI6IlJTMjU2In0.eyJzdWIiOiJjbGllbnRfaWQiLCJjbGllbnRJZCI6ImNsaWVudF9pZCIsImNoYW5uZWwiOiJjaGFubmVsIiwiaXNzIjoiaHR0cHM6Ly9taWwtYXV0aCIsImdyb3VwcyI6InJvbGUiLCJ0ZXJtaW5hbElkIjoidGVybWluYWxfaWQiLCJhdWQiOiJodHRwczovL21pbCIsIm1lcmNoYW50SWQiOiJtZXJjaGFudF9pZCIsInNjb3BlIjoic2NvcGUiLCJmaXNjYWxDb2RlIjoiZW5jX2Zpc2NhbF9jb2RlIiwiZXhwIjoxNzE3NjUyLCJhY3F1aXJlcklkIjoiYWNxdWlyZXJfaWQiLCJpYXQiOjE3MTc1OTJ9.AA",
143+
signedJwt,
144144
response.getAccessToken());
145145
},
146146
f -> fail(f));

0 commit comments

Comments
 (0)
Please sign in to comment.