Skip to content

Commit

Permalink
Do not check logs in serverless tests
Browse files Browse the repository at this point in the history
These apps do not have them
  • Loading branch information
fedinskiy committed Jan 10, 2025
1 parent 106bd84 commit e527a5c
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
import static org.hamcrest.Matchers.notNullValue;
import static org.hamcrest.Matchers.nullValue;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertNotEquals;
import static org.junit.jupiter.api.Assertions.assertTrue;

import java.util.List;
Expand Down Expand Up @@ -90,7 +91,7 @@ public void shortRecordReturnedDirectly() {

@Test
@Tag("https://github.com/quarkusio/quarkus/issues/44564")
void interceptedMethodFound() throws InterruptedException {
void interceptedMethodFound() {
Response operator = givenSpec()
.when()
.contentType(ContentType.JSON)
Expand All @@ -99,7 +100,7 @@ void interceptedMethodFound() throws InterruptedException {

assertEquals(200, operator.statusCode(), "Intercepted request was not processed");
assertEquals("Hello operator", operator.body().asString(), "Intercepted request was not processed properly");

assertNotEquals(0, app.getLogs().size(), "App logs are empty!");
await().atMost(10, TimeUnit.SECONDS).untilAsserted(() -> {
List<String> logs = app.getLogs();
boolean startFlag = false;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import static io.restassured.RestAssured.given;

import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Tag;

import io.quarkus.test.scenarios.OpenShiftDeploymentStrategy;
Expand All @@ -18,4 +19,9 @@ public class ServerlessExtensionDockerBuildStrategyOpenShiftHttpMinimumReactiveI
protected RequestSpecification givenSpec() {
return HTTPS_CLIENT_SPEC;
}

@Override
@Disabled("Serverless deployments do not have logs which we can check")
void interceptedMethodFound() {
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import static io.restassured.RestAssured.given;

import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Tag;

import io.quarkus.test.scenarios.OpenShiftDeploymentStrategy;
Expand All @@ -19,4 +20,9 @@ public class ServerlessExtensionOpenShiftHttpMinimumReactiveIT extends HttpMinim
protected RequestSpecification givenSpec() {
return HTTPS_CLIENT_SPEC;
}

@Override
@Disabled("Serverless deployments do not have logs which we can check")
void interceptedMethodFound() {
}
}

0 comments on commit e527a5c

Please sign in to comment.