fix: reuse overseas receiving resources in acceptance

lastleg-service/src/main/java/com/baiyue/lastleg/relabel/service/RelabelApplicationService.java

@@ -238,7 +238,7 @@ public class RelabelApplicationService implements WalletResultHandler {
         if (!StringUtils.hasText(request.operationWarehouseId())) throw badRequest("Operation warehouse is required");
         boolean requiresOutbound = request.items().stream().anyMatch(item -> !"SKU_LABEL".equals(operationType(item)));
         String deliveryMethod = trim(request.deliveryMethod());
-        if (requiresOutbound && !DELIVERY_METHODS.contains(deliveryMethod)) {
+        if (requiresOutbound && (!StringUtils.hasText(deliveryMethod) || !DELIVERY_METHODS.contains(deliveryMethod))) {
             throw badRequest("Delivery method must be TRUCK, PICKUP or EXPRESS when carton labels are changed");
         }
         if (!requiresOutbound && StringUtils.hasText(deliveryMethod)) {

lastleg-service/src/test/java/com/baiyue/lastleg/relabel/RelabelContractTest.java

@@ -90,4 +90,12 @@ class RelabelContractTest {
             openApi.indexOf("    RelabelItem:"));
         assertFalse(relabelCreate.contains("customerId"));
     }
+
+    @Test
+    void missingDeliveryMethodIsValidatedWithoutCallingNullRejectingSetContains() throws Exception {
+        String service = Files.readString(Path.of("src/main/java/com/baiyue/lastleg/relabel/service/RelabelApplicationService.java"),
+            StandardCharsets.UTF_8);
+        assertTrue(service.contains("requiresOutbound && (!StringUtils.hasText(deliveryMethod)"
+            + " || !DELIVERY_METHODS.contains(deliveryMethod))"));
+    }
 }