-
Notifications
You must be signed in to change notification settings - Fork 25
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Testing APD web client with a dummy APD server + bug fix
- We create a web server TestApdServerVerticle before testing at port 7331 - Server contains several error conditions and success conditions - Testing for success/failures in GET /userclasses and POST /verify ApdWebClient ------------ - Add constructor so that the port the web client reaches the APD on is configurable - default is 443 - **Bug fix** - handle case if an APD sends an empty response using Optional and throwing a DecodeException if null TestApdServerVerticle --------------------- - Deploys a test server on HTTP at port 7331 - Has error, success cases ApdWebClientTest ---------------- - Deploying ApdWebClient to connect with APDs on port 7331 and not use SSL - Added tests
- Loading branch information
1 parent
99a0a2e
commit d049212
Showing
3 changed files
with
426 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
160 changes: 160 additions & 0 deletions
160
src/test/java/iudx/aaa/server/apd/ApdWebClientTest.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,160 @@ | ||
| package iudx.aaa.server.apd; | ||
|
|
||
| import static iudx.aaa.server.apd.Constants.APD_REQ_PROVIDER; | ||
| import static iudx.aaa.server.apd.Constants.APD_REQ_RESOURCE; | ||
| import static iudx.aaa.server.apd.Constants.APD_REQ_USER; | ||
| import static iudx.aaa.server.apd.Constants.APD_REQ_USERCLASS; | ||
| import static iudx.aaa.server.apd.Constants.APD_RESP_DETAIL; | ||
| import static iudx.aaa.server.apd.Constants.APD_RESP_SESSIONID; | ||
| import static iudx.aaa.server.apd.Constants.APD_RESP_TYPE; | ||
| import static iudx.aaa.server.apd.Constants.APD_URN_ALLOW; | ||
| import static iudx.aaa.server.apd.Constants.APD_URN_DENY; | ||
| import static iudx.aaa.server.apd.Constants.APD_URN_DENY_NEEDS_INT; | ||
| import static iudx.aaa.server.apd.Constants.ERR_DETAIL_APD_NOT_RESPOND; | ||
| import static org.junit.jupiter.api.Assertions.assertEquals; | ||
| import static org.junit.jupiter.api.Assertions.assertTrue; | ||
|
|
||
| import io.vertx.core.Future; | ||
| import io.vertx.core.Vertx; | ||
| import io.vertx.core.json.JsonObject; | ||
| import io.vertx.ext.web.client.WebClient; | ||
| import io.vertx.ext.web.client.WebClientOptions; | ||
| import io.vertx.junit5.VertxExtension; | ||
| import io.vertx.junit5.VertxTestContext; | ||
| import iudx.aaa.server.apiserver.util.ComposeException; | ||
| import org.junit.jupiter.api.AfterAll; | ||
| import org.junit.jupiter.api.BeforeAll; | ||
| import org.junit.jupiter.api.DisplayName; | ||
| import org.junit.jupiter.api.MethodOrderer.OrderAnnotation; | ||
| import org.junit.jupiter.api.Order; | ||
| import org.junit.jupiter.api.RepeatedTest; | ||
| import org.junit.jupiter.api.Test; | ||
| import org.junit.jupiter.api.TestMethodOrder; | ||
| import org.junit.jupiter.api.extension.ExtendWith; | ||
|
|
||
| @ExtendWith({VertxExtension.class}) | ||
| @TestMethodOrder(OrderAnnotation.class) | ||
| public class ApdWebClientTest { | ||
| private static ApdWebClient apdWebClient; | ||
| private static TestApdServerVerticle verticle = new TestApdServerVerticle(); | ||
|
|
||
| @BeforeAll | ||
| @DisplayName("Deploying Verticle") | ||
| static void startVertx(Vertx vertx, VertxTestContext testContext) { | ||
|
|
||
| /* | ||
| * Deploying with timeout 4000 instead of picking up from config file. Deploying with no SSL | ||
| * checks. Redirects are always not allowed. | ||
| */ | ||
| JsonObject apdWebCliConfig = new JsonObject().put(Constants.CONFIG_WEBCLI_TIMEOUTMS, 4000); | ||
|
|
||
| WebClientOptions webClientOptions = new WebClientOptions().setSsl(false).setVerifyHost(false) | ||
| .setTrustAll(false).setFollowRedirects(false); | ||
| WebClient webClient = WebClient.create(vertx, webClientOptions); | ||
|
|
||
| /* TestApdServiceVerticle starts on port 7331, so using this constructor */ | ||
| apdWebClient = new ApdWebClient(webClient, apdWebCliConfig, 7331); | ||
|
|
||
| vertx.deployVerticle(verticle, handler -> { | ||
| if (handler.succeeded()) { | ||
| testContext.completeNow(); | ||
| } else { | ||
| handler.cause().printStackTrace(); | ||
| } | ||
| }); | ||
| } | ||
|
|
||
| @AfterAll | ||
| public static void finish(VertxTestContext testContext) { | ||
| verticle.stop(); | ||
| testContext.completeNow(); | ||
| } | ||
|
|
||
| @Order(1) | ||
| @RepeatedTest(TestApdServerVerticle.USERCLASS_ERRORS) | ||
| @DisplayName("Test get userclass error cases") | ||
| void testGetUserclassErrors(VertxTestContext testContext) { | ||
| testContext.assertFailure(apdWebClient.checkApdExists("localhost")).recover(r -> { | ||
| assertTrue(r instanceof ComposeException); | ||
| assertTrue(r.getLocalizedMessage().equals(ERR_DETAIL_APD_NOT_RESPOND)); | ||
| return Future.succeededFuture(); | ||
| }).onSuccess(x -> testContext.completeNow()); | ||
| } | ||
|
|
||
| @Order(2) | ||
| @Test | ||
| @DisplayName("Test get userclass success") | ||
| void testGetUserclassSuccess(VertxTestContext testContext) { | ||
| testContext.assertComplete(apdWebClient.checkApdExists("localhost")).compose(r -> { | ||
| assertTrue(r); | ||
| return Future.succeededFuture(); | ||
| }).onSuccess(x -> testContext.completeNow()); | ||
| } | ||
|
|
||
| @Order(3) | ||
| @RepeatedTest(TestApdServerVerticle.VERIFY_ERRORS) | ||
| @DisplayName("Test post verify error cases") | ||
| void testPostVerifyErrors(VertxTestContext testContext) { | ||
| /* | ||
| * We just put empty objects for user and provider and dummy placeholders for the auth token and | ||
| * resource ID | ||
| */ | ||
| JsonObject request = | ||
| new JsonObject().put(APD_REQ_USER, new JsonObject()).put(APD_REQ_PROVIDER, new JsonObject()) | ||
| .put(APD_REQ_RESOURCE, "resource").put(APD_REQ_USERCLASS, "TestError"); | ||
| testContext.assertFailure(apdWebClient.callVerifyApdEndpoint("localhost", "token", request)) | ||
| .recover(r -> { | ||
| assertTrue(r instanceof ComposeException); | ||
| assertTrue(r.getLocalizedMessage().equals(ERR_DETAIL_APD_NOT_RESPOND)); | ||
| return Future.succeededFuture(); | ||
| }).onSuccess(x -> testContext.completeNow()); | ||
| } | ||
|
|
||
| @Order(4) | ||
| @Test | ||
| @DisplayName("Test post verify allow") | ||
| void testPostVerifyAllow(VertxTestContext testContext) { | ||
| JsonObject request = | ||
| new JsonObject().put(APD_REQ_USER, new JsonObject()).put(APD_REQ_PROVIDER, new JsonObject()) | ||
| .put(APD_REQ_RESOURCE, "resource").put(APD_REQ_USERCLASS, "TestAllow"); | ||
| testContext.assertComplete(apdWebClient.callVerifyApdEndpoint("localhost", "token", request)) | ||
| .compose(r -> { | ||
| assertEquals(r.getString(APD_RESP_TYPE), APD_URN_ALLOW); | ||
| return Future.succeededFuture(); | ||
| }).onSuccess(x -> testContext.completeNow()); | ||
| } | ||
|
|
||
| @Order(5) | ||
| @Test | ||
| @DisplayName("Test post verify deny") | ||
| void testPostVerifyDeny(VertxTestContext testContext) { | ||
| JsonObject request = | ||
| new JsonObject().put(APD_REQ_USER, new JsonObject()).put(APD_REQ_PROVIDER, new JsonObject()) | ||
| .put(APD_REQ_RESOURCE, "resource").put(APD_REQ_USERCLASS, "TestDeny"); | ||
| testContext.assertComplete(apdWebClient.callVerifyApdEndpoint("localhost", "token", request)) | ||
| .compose(r -> { | ||
| assertEquals(r.getString(APD_RESP_TYPE), APD_URN_DENY); | ||
| assertTrue(r.containsKey(APD_RESP_DETAIL)); | ||
| assertTrue(r.getString(APD_RESP_DETAIL) != null); | ||
| return Future.succeededFuture(); | ||
| }).onSuccess(x -> testContext.completeNow()); | ||
| } | ||
|
|
||
| @Order(6) | ||
| @Test | ||
| @DisplayName("Test post verify deny-needs-interaction") | ||
| void testPostVerifyDenyNeedsInteraction(VertxTestContext testContext) { | ||
| JsonObject request = | ||
| new JsonObject().put(APD_REQ_USER, new JsonObject()).put(APD_REQ_PROVIDER, new JsonObject()) | ||
| .put(APD_REQ_RESOURCE, "resource").put(APD_REQ_USERCLASS, "TestDenyNInteraction"); | ||
| testContext.assertComplete(apdWebClient.callVerifyApdEndpoint("localhost", "token", request)) | ||
| .compose(r -> { | ||
| assertEquals(r.getString(APD_RESP_TYPE), APD_URN_DENY_NEEDS_INT); | ||
| assertTrue(r.containsKey(APD_RESP_DETAIL)); | ||
| assertTrue(r.containsKey(APD_RESP_SESSIONID)); | ||
| assertTrue(r.getString(APD_RESP_DETAIL) != null); | ||
| assertTrue(r.getString(APD_RESP_SESSIONID) != null); | ||
| return Future.succeededFuture(); | ||
| }).onSuccess(x -> testContext.completeNow()); | ||
| } | ||
| } |
Oops, something went wrong.