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

KOP-3126 Fix RemoteITCase

parent 70b55a78
No related branches found
No related tags found
1 merge request!3Kop 3126 adjust tests
Pipeline #2258 failed
...@@ -174,12 +174,6 @@ ...@@ -174,12 +174,6 @@
<plugin> <plugin>
<groupId>org.apache.maven.plugins</groupId> <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-failsafe-plugin</artifactId> <artifactId>maven-failsafe-plugin</artifactId>
<configuration>
<excludes>
<!--docker environment in der pipeline laeuft noch nicht-->
<exclude>**/XtaClientITCase.java</exclude>
</excludes>
</configuration>
<executions> <executions>
<execution> <execution>
<goals> <goals>
......
...@@ -7,6 +7,7 @@ import static org.assertj.core.api.Assertions.*; ...@@ -7,6 +7,7 @@ import static org.assertj.core.api.Assertions.*;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import java.util.Optional;
import java.util.function.Consumer; import java.util.function.Consumer;
import java.util.function.Predicate; import java.util.function.Predicate;
import java.util.stream.Stream; import java.util.stream.Stream;
...@@ -65,10 +66,10 @@ class XtaClientRemoteITCase { ...@@ -65,10 +66,10 @@ class XtaClientRemoteITCase {
@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<>();
testClient = XTA_REMOTE_SERVER_SETUP_EXTENSION.getTestClient(); testClient = XTA_REMOTE_SERVER_SETUP_EXTENSION.getTestClient();
...@@ -76,6 +77,11 @@ class XtaClientRemoteITCase { ...@@ -76,6 +77,11 @@ class XtaClientRemoteITCase {
devClient = XTA_REMOTE_SERVER_SETUP_EXTENSION.getDevClient(); devClient = XTA_REMOTE_SERVER_SETUP_EXTENSION.getDevClient();
silentDevClient = XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentDevClient(); silentDevClient = XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentDevClient();
if (Optional.ofNullable(System.getenv("CLEAR_ALL_MESSAGES")).map(d -> d.equals("yes")).orElse(false)) {
closeAllMessages(XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentDevClientConfig(), DEV_READER_CLIENT_IDENTIFIER);
closeAllMessages(XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentTestClientConfig(), TEST_READER_CLIENT_IDENTIFIER);
}
failIfAnyMessagePending(XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentDevClientConfig(), DEV_READER_CLIENT_IDENTIFIER); failIfAnyMessagePending(XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentDevClientConfig(), DEV_READER_CLIENT_IDENTIFIER);
failIfAnyMessagePending(XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentTestClientConfig(), TEST_READER_CLIENT_IDENTIFIER); failIfAnyMessagePending(XTA_REMOTE_SERVER_SETUP_EXTENSION.getSilentTestClientConfig(), TEST_READER_CLIENT_IDENTIFIER);
......
...@@ -16,6 +16,7 @@ import de.ozgcloud.xta.client.core.WrappedXtaService; ...@@ -16,6 +16,7 @@ import de.ozgcloud.xta.client.core.WrappedXtaService;
import de.ozgcloud.xta.client.core.WrappedXtaServiceFactory; import de.ozgcloud.xta.client.core.WrappedXtaServiceFactory;
import de.ozgcloud.xta.client.exception.XtaClientRuntimeException; import de.ozgcloud.xta.client.exception.XtaClientRuntimeException;
import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.CodeFehlernummer; import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.CodeFehlernummer;
import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.InvalidMessageIDException;
import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.MessageSchemaViolationException; import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.MessageSchemaViolationException;
import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.ParameterIsNotValidException; import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.ParameterIsNotValidException;
import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.PermissionDeniedException; import de.ozgcloud.xta.client.gen.v3.de.xoev.transport.xta.x211.PermissionDeniedException;
...@@ -122,11 +123,19 @@ public class XtaServerSetupExtensionTestUtil { ...@@ -122,11 +123,19 @@ public class XtaServerSetupExtensionTestUtil {
closeMessagesById(config, clientId, messageIds); closeMessagesById(config, clientId, messageIds);
} }
@SneakyThrows
public static void closeMessagesById(XtaClientConfig config, XtaIdentifier clientId, List<String> messageIds) { public static void closeMessagesById(XtaClientConfig config, XtaIdentifier clientId, List<String> messageIds) {
var wrappedService = createWrappedService(config); var wrappedService = createWrappedService(config);
for (var messageId : messageIds) { for (var messageId : messageIds) {
closeMessageById(wrappedService, messageId, clientId);
}
}
@SneakyThrows
private static void closeMessageById(WrappedXtaService wrappedService, String messageId, XtaIdentifier clientId) {
try {
wrappedService.close(messageId, clientId); wrappedService.close(messageId, clientId);
} catch (InvalidMessageIDException e) {
log.warn("Failed closing message, possibly it has already been closed! '{}' (reader: {})", messageId, clientId);
} }
} }
...@@ -135,10 +144,12 @@ public class XtaServerSetupExtensionTestUtil { ...@@ -135,10 +144,12 @@ public class XtaServerSetupExtensionTestUtil {
return WrappedXtaServiceFactory.from(config).create(); return WrappedXtaServiceFactory.from(config).create();
} }
public static XtaMessageMetaData withoutMessageIdAndSize(XtaMessageMetaData metaData) { public static XtaMessageMetaData withoutMessageIdAndSizeAndDelivery(XtaMessageMetaData metaData) {
return metaData.toBuilder() return metaData.toBuilder()
.messageId(null) .messageId(null)
.messageSize(null) .messageSize(null)
.deliveryAttributesDelivery(null)
.deliveryAttributesOrigin(null)
.build(); .build();
} }
...@@ -268,9 +279,9 @@ public class XtaServerSetupExtensionTestUtil { ...@@ -268,9 +279,9 @@ public class XtaServerSetupExtensionTestUtil {
// Assert equal metadata (ignoring message id and size since they should be null before sending) // Assert equal metadata (ignoring message id and size since they should be null before sending)
assertThat(processedMessages) assertThat(processedMessages)
.extracting(XtaMessage::metaData) .extracting(XtaMessage::metaData)
.extracting(XtaServerSetupExtensionTestUtil::withoutMessageIdAndSize) .extracting(XtaServerSetupExtensionTestUtil::withoutMessageIdAndSizeAndDelivery)
.containsExactlyInAnyOrderElementsOf(Arrays.stream(messageMetaDataItems) .containsExactlyInAnyOrderElementsOf(Arrays.stream(messageMetaDataItems)
.map(XtaServerSetupExtensionTestUtil::withoutMessageIdAndSize) .map(XtaServerSetupExtensionTestUtil::withoutMessageIdAndSizeAndDelivery)
.toList()); .toList());
} catch (AssertionError | RuntimeException e) { } catch (AssertionError | RuntimeException e) {
log.error("Messages do not exactly contain excepted metadata!"); log.error("Messages do not exactly contain excepted metadata!");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment