Fix tests

Author: feyst

phpunit.xml

@@ -2,10 +2,7 @@
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd"
          bootstrap="tests/bootstrap.php"
-         cacheDirectory=".phpunit.cache"
          executionOrder="depends,defects"
-         requireCoverageMetadata="false"
-         beStrictAboutCoverageMetadata="false"
          beStrictAboutOutputDuringTests="true"
          failOnRisky="true"
          failOnWarning="true">

src/EDI/Interpreter.php

@@ -631,7 +631,7 @@ private function processXmlSegment(\SimpleXMLElement $elm, array &$message, int
             } else {
                 if (! $segmentVisited && isset($elm['required'])) {
                     $segmentVisited = true;
-                    if (\call_user_func($this->comparisonFunction, $message[$segmentIdx+1], $elm)) {
+                    if (isset($message[$segmentIdx+1]) && \call_user_func($this->comparisonFunction, $message[$segmentIdx+1], $elm)) {
                         $errors[] = [
                             'text' => $this->messageTextConf['SPURIOUSSEGMENT'].($this->patchFiles ? ' (skipped)' : ''),
                             'position' => $segmentIdx,

tests/EDITest/InterpreterTest.php

@@ -91,8 +91,8 @@ public function testDESADV()
             'JSON does not match expected output'
         );
 
-        static::assertSame(3598, \strlen($interpreter->getJson()));
-        static::assertSame(9383, \strlen($interpreter->getJson(true)));
+        static::assertSame(3594, \strlen($interpreter->getJson()));
+        static::assertSame(9379, \strlen($interpreter->getJson(true)));
 
         static::assertCount(2, $interpreter->getMessages());
         static::assertCount(0, $interpreter->getErrors());

tests/files/D96ADESADV.json

@@ -227,7 +227,7 @@
         "messageTrailer": {
             "segmentIdx": 20,
             "segmentCode": "UNT",
-            "segmentGroup": "SG16",
+            "segmentGroup": "",
             "numberOfSegmentsInTheMessage": "21",
             "messageReferenceNumber": "142"
         }