Fix handling of multiple syntax elements

- Add tests for several exception cases
xlate · May 11, 2020 · 8752f4c · 8752f4c
1 parent 862d0e7
commit 8752f4c
Show file tree

Hide file tree

Showing 7 changed files with 2,600 additions and 2 deletions.
diff --git a/src/main/java/io/xlate/edi/internal/schema/SchemaReaderBase.java b/src/main/java/io/xlate/edi/internal/schema/SchemaReaderBase.java
@@ -362,8 +362,10 @@ StructureType readComplexType(XMLStreamReader reader,
 
         if (event == XMLStreamConstants.START_ELEMENT) {
             requireElementStart(qnSyntax, reader);
-            readSyntax(reader, rules);
-            event = reader.nextTag();
+            do {
+                readSyntax(reader, rules);
+                event = reader.nextTag();
+            } while (event == XMLStreamConstants.START_ELEMENT && qnSyntax.equals(reader.getName()));
         }
 
         if (event == XMLStreamConstants.END_ELEMENT) {

diff --git a/src/test/java/io/xlate/edi/internal/schema/StaEDISchemaFactoryTest.java b/src/test/java/io/xlate/edi/internal/schema/StaEDISchemaFactoryTest.java
@@ -223,4 +223,34 @@ public void testDiscriminatorMissingEnumeration() {
         assertNotNull(cause);
         assertEquals("Discriminator element does not specify value enumeration: 2.2", cause.getMessage());
     }
+
+    @Test
+    public void testInvalidSyntaxTypeValue() {
+        SchemaFactory factory = SchemaFactory.newFactory();
+        InputStream stream = getClass().getResourceAsStream("/EDIFACT/fragment-uci-invalid-syntax-type.xml");
+        EDISchemaException thrown = assertThrows(EDISchemaException.class, () -> factory.createSchema(stream));
+        Throwable cause = thrown.getCause();
+        assertNotNull(cause);
+        assertEquals("Invalid syntax 'type': [conditional-junk]", cause.getMessage());
+    }
+
+    @Test
+    public void testInvalidMinOccursValue() {
+        SchemaFactory factory = SchemaFactory.newFactory();
+        InputStream stream = getClass().getResourceAsStream("/EDIFACT/fragment-uci-invalid-min-occurs.xml");
+        EDISchemaException thrown = assertThrows(EDISchemaException.class, () -> factory.createSchema(stream));
+        Throwable cause = thrown.getCause();
+        assertNotNull(cause);
+        assertEquals("Invalid minOccurs", cause.getMessage());
+    }
+
+    @Test
+    public void testDuplicateElementTypeNames() {
+        SchemaFactory factory = SchemaFactory.newFactory();
+        InputStream stream = getClass().getResourceAsStream("/EDIFACT/fragment-uci-duplicate-element-names.xml");
+        EDISchemaException thrown = assertThrows(EDISchemaException.class, () -> factory.createSchema(stream));
+        Throwable cause = thrown.getCause();
+        assertNotNull(cause);
+        assertEquals("duplicate name: DE0004", cause.getMessage());
+    }
 }
diff --git a/src/test/java/io/xlate/edi/internal/schema/StaEDISchemaTest.java b/src/test/java/io/xlate/edi/internal/schema/StaEDISchemaTest.java
@@ -28,6 +28,7 @@
 
 import org.junit.jupiter.api.Test;
 
+import io.xlate.edi.schema.EDIComplexType;
 import io.xlate.edi.schema.EDISchemaException;
 import io.xlate.edi.schema.EDIType;
 
@@ -69,4 +70,19 @@ public void testRootTypeIsInterchangeV3() throws EDISchemaException, XMLStreamEx
 
         assertEquals(EDIType.Type.INTERCHANGE, schema.getType(INTERCHANGE_V3).getType());
     }
+
+    @Test
+    public void testLoadV3TransactionMultipleSyntaxElements_EDIFACT_CONTRL() throws EDISchemaException, XMLStreamException, FactoryConfigurationError {
+        StaEDISchema schema = new StaEDISchema(INTERCHANGE_V3, TRANSACTION_V3);
+        InputStream schemaStream = getClass().getResourceAsStream("/EDIFACT/CONTRL-v4r02.xml");
+        XMLStreamReader reader = XMLInputFactory.newInstance().createXMLStreamReader(schemaStream);
+        reader.nextTag(); // Pass by <schema> element
+        Map<String, EDIType> types = new SchemaReaderV3(reader).readTypes();
+        schema.setTypes(types);
+
+        assertEquals(EDIType.Type.TRANSACTION, schema.getType(TRANSACTION_V3).getType());
+        assertEquals(4, ((EDIComplexType) schema.getType("UCF")).getSyntaxRules().size());
+        assertEquals(4, ((EDIComplexType) schema.getType("UCI")).getSyntaxRules().size());
+        assertEquals(7, ((EDIComplexType) schema.getType("UCM")).getSyntaxRules().size());
+    }
 }