Skip to content
Snippets Groups Projects
Commit bba8694e authored by Jan Zickermann's avatar Jan Zickermann
Browse files

KOP-3126 Fix ClientITCase

parent b4fa8782
No related branches found
No related tags found
1 merge request!3Kop 3126 adjust tests
Pipeline #2259 failed
...@@ -44,10 +44,10 @@ class XtaClientITCase { ...@@ -44,10 +44,10 @@ class XtaClientITCase {
@BeforeEach @BeforeEach
@SneakyThrows @SneakyThrows
void setup() { void setup() {
processMessageDummy = (message) -> { processMessageDummy = message -> {
}; };
supportCheckedMetadataItems = new ArrayList<>(); supportCheckedMetadataItems = new ArrayList<>();
isSupportedDummy = (metaData) -> true; isSupportedDummy = metaData -> true;
processedMessages = new ArrayList<>(); processedMessages = new ArrayList<>();
silentTestClient = XTA_TEST_SERVER_SETUP_EXTENSION.getSilentTestClient(); silentTestClient = XTA_TEST_SERVER_SETUP_EXTENSION.getSilentTestClient();
...@@ -99,7 +99,7 @@ class XtaClientITCase { ...@@ -99,7 +99,7 @@ class XtaClientITCase {
void shouldThrowExceptionOnConnectionFailure() { void shouldThrowExceptionOnConnectionFailure() {
setupClientWithoutTrustStore(); setupClientWithoutTrustStore();
assertThatThrownBy(() -> testClient.fetchMessages((message) -> fail("Should not process any message!"))) assertThatThrownBy(() -> testClient.fetchMessages(message -> fail("Should not process any message!")))
.isInstanceOf(XtaClientException.class); .isInstanceOf(XtaClientException.class);
} }
...@@ -303,7 +303,7 @@ class XtaClientITCase { ...@@ -303,7 +303,7 @@ class XtaClientITCase {
@SneakyThrows @SneakyThrows
private List<XtaTransportReport> fetchMessages() { private List<XtaTransportReport> fetchMessages() {
return testClient.fetchMessages((message) -> { return testClient.fetchMessages(message -> {
processedMessages.add(message); processedMessages.add(message);
processMessageDummy.accept(message); processMessageDummy.accept(message);
}); });
......
...@@ -153,9 +153,11 @@ public class XtaServerSetupExtensionTestUtil { ...@@ -153,9 +153,11 @@ public class XtaServerSetupExtensionTestUtil {
.build(); .build();
} }
public static XtaMessageMetaData withoutMessageSize(XtaMessageMetaData metaData) { public static XtaMessageMetaData withoutMessageSizeAndDelivery(XtaMessageMetaData metaData) {
return metaData.toBuilder() return metaData.toBuilder()
.messageSize(null) .messageSize(null)
.deliveryAttributesDelivery(null)
.deliveryAttributesOrigin(null)
.build(); .build();
} }
...@@ -325,10 +327,10 @@ public class XtaServerSetupExtensionTestUtil { ...@@ -325,10 +327,10 @@ public class XtaServerSetupExtensionTestUtil {
// ignoring size since it may be null for transport report (due to message closed?) // ignoring size since it may be null for transport report (due to message closed?)
assertThat(transportReports) assertThat(transportReports)
.extracting(XtaTransportReport::metaData) .extracting(XtaTransportReport::metaData)
.extracting(XtaServerSetupExtensionTestUtil::withoutMessageSize) .extracting(XtaServerSetupExtensionTestUtil::withoutMessageSizeAndDelivery)
.containsExactlyElementsOf(processedMessages.stream() .containsExactlyElementsOf(processedMessages.stream()
.map(XtaMessage::metaData) .map(XtaMessage::metaData)
.map(XtaServerSetupExtensionTestUtil::withoutMessageSize) .map(XtaServerSetupExtensionTestUtil::withoutMessageSizeAndDelivery)
.toList()); .toList());
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment