From a41b072b251df7a07c4c25b7834de53aa30830e1 Mon Sep 17 00:00:00 2001
From: palatsangeetha <138600841+palatsangeetha@users.noreply.github.com>
Date: Thu, 12 Dec 2024 13:43:49 +0100
Subject: [PATCH] SD-603 Update PartyFunction EBL (#243)

---
 .../standards/ebl/checks/EBLChecks.java       | 20 ++++++++++-
 .../standards/ebl/checks/EblDatasets.java     |  4 +++
 .../ebl/messages/ebl-api-3.0.0-request.json   |  2 +-
 .../standards/ebl/checks/EBLChecksTest.java   | 34 +++++++++++++++++++
 4 files changed, 58 insertions(+), 2 deletions(-)

diff --git a/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EBLChecks.java b/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EBLChecks.java
index 5c846917..451529ce 100644
--- a/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EBLChecks.java
+++ b/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EBLChecks.java
@@ -5,6 +5,8 @@
 import static org.dcsa.conformance.standards.ebl.checks.EblDatasets.EXEMPT_PACKAGE_CODES;
 import static org.dcsa.conformance.standards.ebl.checks.EblDatasets.MODE_OF_TRANSPORT;
 import static org.dcsa.conformance.standards.ebl.checks.EblDatasets.NATIONAL_COMMODITY_CODES;
+import static org.dcsa.conformance.standards.ebl.checks.EblDatasets.PARTY_FUNCTION_CODE;
+import static org.dcsa.conformance.standards.ebl.checks.EblDatasets.PARTY_FUNCTION_CODE_HBL;
 import static org.dcsa.conformance.standards.ebl.checks.EblDatasets.REQUESTED_CARRIER_CLAUSES;
 
 
@@ -666,6 +668,19 @@ private static Consumer<MultiAttributeValidator> allDg(Consumer<MultiAttributeVa
               JsonAttribute.mustBeAbsent(SI_REQUEST_SEND_TO_PLATFORM)),
           JsonAttribute.mustBeAbsent(SI_REQUEST_SEND_TO_PLATFORM));
 
+  static final JsonRebaseableContentCheck VALID_PARTY_FUNCTION =
+      JsonAttribute.allIndividualMatchesMustBeValid(
+          "The partyFunction in OtherDocumentParty is valid",
+          mav -> mav.submitAllMatching("documentParties.other.*.partyFunction"),
+          JsonAttribute.matchedMustBeDatasetKeywordIfPresent(PARTY_FUNCTION_CODE));
+
+  static final JsonRebaseableContentCheck VALID_PARTY_FUNCTION_HBL =
+      JsonAttribute.allIndividualMatchesMustBeValid(
+          "The partyFunction in OtherDocumentParty of houseBillOfLadings is valid",
+          mav ->
+              mav.submitAllMatching("houseBillOfLadings.*.documentParties.other.*.partyFunction"),
+          JsonAttribute.matchedMustBeDatasetKeywordIfPresent(PARTY_FUNCTION_CODE_HBL));
+
   private static final List<JsonContentCheck> STATIC_SI_CHECKS = Arrays.asList(
     JsonAttribute.mustBeDatasetKeywordIfPresent(
       SI_REQUEST_SEND_TO_PLATFORM,
@@ -684,6 +699,8 @@ private static Consumer<MultiAttributeValidator> allDg(Consumer<MultiAttributeVa
     ROUTING_OF_CONSIGNMENT_COUNTRIES_CHECK,
     VALID_REQUESTED_CARRIER_CLAUSES,
     BUYER_AND_SELLER_CONDITIONAL_CHECK,
+    VALID_PARTY_FUNCTION,
+    VALID_PARTY_FUNCTION_HBL,
     ONLY_EBLS_CAN_BE_NEGOTIABLE,
     EBL_AT_MOST_ONE_COPY_WITHOUT_CHARGES,
     EBL_AT_MOST_ONE_COPY_WITH_CHARGES,
@@ -811,7 +828,8 @@ private static Consumer<MultiAttributeValidator> allDg(Consumer<MultiAttributeVa
       JsonAttribute.matchedMustBeDatasetKeywordIfPresent(MODE_OF_TRANSPORT)
     ),
     NOTIFY_PARTIES_REQUIRED_IN_NEGOTIABLE_BLS,
-    TLR_CC_T_COMBINATION_UNIQUE
+    TLR_CC_T_COMBINATION_UNIQUE,
+    VALID_PARTY_FUNCTION
   );
 
   public static final JsonContentCheck SIR_REQUIRED_IN_NOTIFICATION = JsonAttribute.mustBePresent(SI_NOTIFICATION_SIR_PTR);
diff --git a/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EblDatasets.java b/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EblDatasets.java
index a15bbe16..3cd27acc 100644
--- a/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EblDatasets.java
+++ b/ebl/src/main/java/org/dcsa/conformance/standards/ebl/checks/EblDatasets.java
@@ -92,4 +92,8 @@ public class EblDatasets {
     "INTRANSITCLAUSE"
   );
 
+  public static final KeywordDataset PARTY_FUNCTION_CODE =
+      KeywordDataset.staticDataset("SCO", "DDR", "DDS", "COW", "COX", "CS", "MF", "WH");
+  public static final KeywordDataset PARTY_FUNCTION_CODE_HBL =
+      KeywordDataset.staticDataset("DDR", "DDS", "CS", "MF", "WH");
 }
diff --git a/ebl/src/main/resources/standards/ebl/messages/ebl-api-3.0.0-request.json b/ebl/src/main/resources/standards/ebl/messages/ebl-api-3.0.0-request.json
index 373a3a11..2d6a407e 100644
--- a/ebl/src/main/resources/standards/ebl/messages/ebl-api-3.0.0-request.json
+++ b/ebl/src/main/resources/standards/ebl/messages/ebl-api-3.0.0-request.json
@@ -148,7 +148,7 @@
     },
     "issueTo": {
       "partyName": "DCSA CTK",
-      "identifyingCodes" : [
+      "identifyingCodes": [
         {
           "codeListProvider": "W3C",
           "codeListName": "LCL",
diff --git a/ebl/src/test/java/org/dcsa/conformance/standards/ebl/checks/EBLChecksTest.java b/ebl/src/test/java/org/dcsa/conformance/standards/ebl/checks/EBLChecksTest.java
index 3bf8914e..59241b4b 100644
--- a/ebl/src/test/java/org/dcsa/conformance/standards/ebl/checks/EBLChecksTest.java
+++ b/ebl/src/test/java/org/dcsa/conformance/standards/ebl/checks/EBLChecksTest.java
@@ -19,6 +19,8 @@
 import static org.dcsa.conformance.standards.ebl.checks.EBLChecks.LOCATION_NAME_CONDITIONAL_VALIDATION_POA;
 import static org.dcsa.conformance.standards.ebl.checks.EBLChecks.LOCATION_NAME_CONDITIONAL_VALIDATION_POFD;
 import static org.dcsa.conformance.standards.ebl.checks.EBLChecks.VALID_CONSIGMENT_ITEMS_REFERENCE_TYPES;
+import static org.dcsa.conformance.standards.ebl.checks.EBLChecks.VALID_PARTY_FUNCTION;
+import static org.dcsa.conformance.standards.ebl.checks.EBLChecks.VALID_PARTY_FUNCTION_HBL;
 import static org.dcsa.conformance.standards.ebl.checks.EBLChecks.VALID_REQUESTED_CARRIER_CLAUSES;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
@@ -295,4 +297,36 @@ void testValidConsignmentItemsReferenceTypes() {
     Set<String> invalidErrors = VALID_CONSIGMENT_ITEMS_REFERENCE_TYPES.validate(rootNode);
     assertEquals(1, invalidErrors.size());
   }
+
+  @Test
+  void testValidPartyFunction() {
+    ObjectNode documentParties = rootNode.putObject("documentParties");
+    ArrayNode otherParties = documentParties.putArray("other");
+    ObjectNode otherParty = otherParties.addObject();
+    otherParty.put("partyFunction", "SCO");
+
+    Set<String> errors = VALID_PARTY_FUNCTION.validate(rootNode);
+    assertEquals(0, errors.size());
+
+    otherParty.put("partyFunction", "SSS");
+    errors = VALID_PARTY_FUNCTION.validate(rootNode);
+    assertEquals(1, errors.size());
+  }
+
+  @Test
+  void testValidPartyFunctionHBL() {
+    ArrayNode houseBillOfLadings = rootNode.putArray("houseBillOfLadings");
+    ObjectNode hbl = houseBillOfLadings.addObject();
+    ObjectNode documentParties = hbl.putObject("documentParties");
+    ArrayNode otherParties = documentParties.putArray("other");
+    ObjectNode otherParty = otherParties.addObject();
+    otherParty.put("partyFunction", "CS");
+
+    Set<String> errors = VALID_PARTY_FUNCTION_HBL.validate(rootNode);
+    assertEquals(0, errors.size());
+
+    otherParty.put("partyFunction", "SSS");
+    errors = VALID_PARTY_FUNCTION_HBL.validate(rootNode);
+    assertEquals(1, errors.size());
+  }
 }