JiriOndrusek commented on code in PR #3919:
URL: https://github.com/apache/camel-quarkus/pull/3919#discussion_r947630120
##########
integration-tests/google-pubsub/src/test/java/org/apache/camel/quarkus/component/google/pubsub/it/GooglePubsubTest.java:
##########
@@ -56,6 +74,220 @@ public void jacksonSerializer() {
RestAssured.get("/google-pubsub/pojo")
.then()
.statusCode(200)
- .body("name", is(fruitName));
+ .body("name", Matchers.is(fruitName));
+
+ }
+
+ @Test
+ @Order(3)
+ public void testGrouped() throws Exception {
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.GROUP_DIRECT_AGGREGATOR)
+ .body("body1")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+
+ RestAssured.given()
+
.get("/google-pubsub/receive/subscription/google-pubsub.grouped-subscription-name")
+ .then()
+ .statusCode(204);
+
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.GROUP_DIRECT_AGGREGATOR)
+ .body("body2")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+
+ Set<String> results = new HashSet<>();
+ results.add(RestAssured.given()
+
.get("/google-pubsub/receive/subscription/google-pubsub.grouped-subscription-name")
+ .then()
+ .statusCode(200).extract().asString());
+
+ results.add(RestAssured.given()
+
.get("/google-pubsub/receive/subscription/google-pubsub.grouped-subscription-name")
+ .then()
+ .statusCode(200).extract().asString());
+
+ Assertions.assertTrue(results.contains("body1"));
+ Assertions.assertTrue(results.contains("body2"));
+
+ }
+
+ //Disabled on real account because of
https://issues.apache.org/jira/browse/CAMEL-18277
+ @DisabledIfEnvironmentVariable(named = "GOOGLE_APPLICATION_CREDENTIALS",
matches = ".+")
+ // https://github.com/apache/camel-quarkus/issues/3944
+ @DisabledIfEnvironmentVariable(named = "CI", matches = "true")
+ @Test
+ @Order(4)
+ public void testOrdering() throws Exception {
+ LOG.info("Start of the ordering test");
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.ORDERING_DIRECT_IN)
+ .body("1")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.ORDERING_DIRECT_IN)
+ .body("2")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.ORDERING_DIRECT_IN)
+ .body("3")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.ORDERING_DIRECT_IN)
+ .body("4")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.ORDERING_DIRECT_IN)
+ .body("5")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ RestAssured.given()
+ .queryParam("toEndpoint",
GooglePubSubRoutes.ORDERING_DIRECT_IN)
+ .body("6")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ LOG.info("All messages were sent");
+
+ await().atMost(5, TimeUnit.SECONDS).until(() -> RestAssured.given()
+
.get("/google-pubsub/receive/subscriptionOrdering/google-pubsub.ordering-subscription-name")
+ .then()
+ .statusCode(200)
+ .extract().asString(),
+ Matchers.is("1"));
+ LOG.info("Message \"1\" received.");
+ await().atMost(5, TimeUnit.SECONDS).until(() -> RestAssured.given()
+
.get("/google-pubsub/receive/subscriptionOrdering/google-pubsub.ordering-subscription-name")
+ .then()
+ .statusCode(200)
+ .extract().asString(),
+ Matchers.is("2"));
+ LOG.info("Message \"2\" received.");
+ await().atMost(5, TimeUnit.SECONDS).until(() -> RestAssured.given()
+
.get("/google-pubsub/receive/subscriptionOrdering/google-pubsub.ordering-subscription-name")
+ .then()
+ .statusCode(200)
+ .extract().asString(),
+ Matchers.is("3"));
+ LOG.info("Message \"3\" received.");
+ await().atMost(5, TimeUnit.SECONDS).until(() -> RestAssured.given()
+
.get("/google-pubsub/receive/subscriptionOrdering/google-pubsub.ordering-subscription-name")
+ .then()
+ .statusCode(200)
+ .extract().asString(),
+ Matchers.is("4"));
+ LOG.info("Message \"4\" received.");
+ await().atMost(5, TimeUnit.SECONDS).until(() -> RestAssured.given()
+
.get("/google-pubsub/receive/subscriptionOrdering/google-pubsub.ordering-subscription-name")
+ .then()
+ .statusCode(200)
+ .extract().asString(),
+ Matchers.is("5"));
+ LOG.info("Message \"5\" received.");
+ await().atMost(5, TimeUnit.SECONDS).until(() -> RestAssured.given()
+
.get("/google-pubsub/receive/subscriptionOrdering/google-pubsub.ordering-subscription-name")
+ .then()
+ .statusCode(200)
+ .extract().asString(),
+ Matchers.is("6"));
+ LOG.info("Message \"6\" received.");
+
+ }
+
+ @Test
+ @Order(5)
+ public void testAck() throws Exception {
+ LOG.info("Start of the acking test");
+ //enable ack
+ RestAssured.given()
+ .body(false)
+ .post("/google-pubsub/setFail")
+ .then()
+ .statusCode(201);
+ //successful run
+ RestAssured.given()
+ .queryParam("toEndpoint", GooglePubSubRoutes.ACK_DIRECT_IN)
+ .body("1")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ LOG.info("Message \"1\" was sent and should be Acked.");
+
+ await().atMost(5, TimeUnit.SECONDS).untilAsserted(() ->
RestAssured.given()
+ .get("/google-pubsub/receive/mock/" +
GooglePubSubRoutes.ACK_MOCK_RESULT)
+ .then()
+ .statusCode(200)
+ .body(Matchers.is("1")));
+ LOG.info("Message \"1\" was received.");
+ //once message is received, reset mock
+ RestAssured.given()
+ .get("/google-pubsub/resetMock/" +
GooglePubSubRoutes.ACK_MOCK_RESULT)
+ .then()
+ .statusCode(201);
+ //failing run
+ //disable ack
+ RestAssured.given()
+ .body(true)
+ .post("/google-pubsub/setFail")
+ .then()
+ .statusCode(201);
+
+ RestAssured.given()
+ .queryParam("toEndpoint", GooglePubSubRoutes.ACK_DIRECT_IN)
+ .body("2")
+ .post("/google-pubsub/sendToEndpoint")
+ .then()
+ .statusCode(201);
+ LOG.info("Message \"2\" was sent and should be Nacked.");
+ //wait to be sure that the nacked message was not delivered
+ Thread.sleep(1000);
+
+ RestAssured.given()
+ .get("/google-pubsub/receive/mock/" +
GooglePubSubRoutes.ACK_MOCK_RESULT)
+ .then()
+ .statusCode(200)
+ .body(Matchers.is(""));
+ LOG.info("Message \"2\" was not delivered.");
+
+ //enable ack
+ RestAssured.given()
+ .body(false)
+ .post("/google-pubsub/setFail")
+ .then()
+ .statusCode(201);
+ LOG.info("Acking was enabled.");
+
+ //assert redelivered message
+ await().atMost(20, TimeUnit.SECONDS).untilAsserted(() ->
RestAssured.given()
+ .get("/google-pubsub/receive/mock/" +
GooglePubSubRoutes.ACK_MOCK_RESULT)
+ .then()
+ .statusCode(200)
+ .body(Matchers.is("2")));
+
+ LOG.info("Message \"2\" was delivered after acking was enabled.");
+ }
+
+ @Test
+ @Order(6)
+ public void stopConsumerTest() throws Exception {
+ RestAssured.given()
Review Comment:
I added a comment to the follow-up issue to answer you
(https://github.com/apache/camel-quarkus/issues/3957#issuecomment-1217698290)
>The problem seems to be somehow connected to the close of the consumer in
the native mode.There is a test stopConsumerTest, which is closing the consumer
after all the tests (and requires method order to achieve this)
I would keep the method here, because it forces the right closure of the
consumer in the native mode. Once
https://github.com/apache/camel-quarkus/issues/3957 is solved, this method
should be removed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
