DT-1680 DT-1681 Not expecting exchange in out-of-band actions

dcsaorg · Sep 30, 2024 · 7017785 · 7017785
1 parent 5f948bb
commit 7017785
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 3 deletions.
diff --git a/core/src/main/java/org/dcsa/conformance/core/scenario/ConformanceAction.java b/core/src/main/java/org/dcsa/conformance/core/scenario/ConformanceAction.java
@@ -105,8 +105,8 @@ public boolean isInputRequired() {
     return false;
   }
 
-  public boolean hasMatchedExchange() {
-    return matchedExchangeUuid != null;
+  public boolean isMissingMatchedExchange() {
+    return matchedExchangeUuid == null;
   }
 
   /**
@@ -175,6 +175,7 @@ protected boolean expectsNotificationExchange() {
     return false;
   }
 
+  @SuppressWarnings("unused")
   protected void doHandleNotificationExchange(ConformanceExchange exchange) {}
 
   public final ConformanceCheck createFullCheck(String expectedApiVersion) {

diff --git a/...rc/main/java/org/dcsa/conformance/standards/eblsurrender/action/VoidAndReissueAction.java b/...rc/main/java/org/dcsa/conformance/standards/eblsurrender/action/VoidAndReissueAction.java
@@ -44,6 +44,11 @@ public boolean isConfirmationRequired() {
     return true;
   }
 
+  @Override
+  public boolean isMissingMatchedExchange() {
+    return false;
+  }
+
   @Override
   public Supplier<String> getSrrSupplier() {
     return null;

diff --git a/...X_Carrier_TDOnlyProcessOutOfBandUpdateOrAmendmentRequestDraftTransportDocumentAction.java b/...X_Carrier_TDOnlyProcessOutOfBandUpdateOrAmendmentRequestDraftTransportDocumentAction.java
@@ -28,4 +28,14 @@ public ObjectNode asJsonNode() {
     node.set("csp", getCspSupplier().get().toJson());
     return node;
   }
+
+  @Override
+  public boolean isConfirmationRequired() {
+    return true;
+  }
+
+  @Override
+  public boolean isMissingMatchedExchange() {
+    return false;
+  }
 }
diff --git a/sandbox/src/main/java/org/dcsa/conformance/sandbox/ConformanceOrchestrator.java b/sandbox/src/main/java/org/dcsa/conformance/sandbox/ConformanceOrchestrator.java
@@ -321,7 +321,7 @@ public void completeCurrentAction() {
               .formatted(currentScenario.toString()));
       return;
     }
-    if (!nextAction.hasMatchedExchange()) {
+    if (nextAction.isMissingMatchedExchange()) {
       throw new UserFacingException(
           "A required API exchange was not yet detected for action '%s'"
               .formatted(nextAction.getActionTitle()));