Skip to content

Commit e509b7c

Browse files
dolfinuswendigo
authored andcommitted
Make trino-openlineage tests output less verbose
1 parent aaccb63 commit e509b7c

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

plugin/trino-openlineage/src/test/java/io/trino/plugin/openlineage/TestOpenLineageEventListenerMarquezIntegration.java

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
*/
1414
package io.trino.plugin.openlineage;
1515

16-
import io.airlift.log.Logger;
1716
import io.trino.SessionRepresentation;
1817
import io.trino.testing.QueryRunner;
1918

@@ -24,14 +23,11 @@
2423
import java.net.http.HttpResponse;
2524

2625
import static java.nio.charset.StandardCharsets.UTF_8;
27-
import static java.util.Locale.ENGLISH;
2826
import static org.assertj.core.api.Assertions.assertThat;
2927

3028
final class TestOpenLineageEventListenerMarquezIntegration
3129
extends BaseTestOpenLineageQueries
3230
{
33-
private static final Logger logger = Logger.get(TestOpenLineageEventListenerMarquezIntegration.class);
34-
3531
private static MarquezServer server;
3632
private static String marquezURI;
3733
private static final HttpClient client = HttpClient.newHttpClient();
@@ -191,10 +187,13 @@ private void checkJobRegistration(HttpClient client, String expectedJobName)
191187

192188
HttpResponse<String> responseJob = client.send(requestJob, HttpResponse.BodyHandlers.ofString());
193189

194-
logger.info(responseJob.body());
190+
assertThat(responseJob.statusCode())
191+
.withFailMessage("Marquez responded with status %s: %s", responseJob.statusCode(), responseJob.body())
192+
.isEqualTo(200);
195193

196-
assertThat(responseJob.statusCode()).isEqualTo(200);
197-
assertThat(responseJob.body().toLowerCase(ENGLISH)).contains("complete");
194+
assertThat(responseJob.body())
195+
.withFailMessage("Expected job %s to be completed: %s", expectedJobName, responseJob.body())
196+
.containsIgnoringCase("complete");
198197
}
199198
catch (Exception e) {
200199
throw new RuntimeException(e);

0 commit comments

Comments
 (0)