From 5bc53853c90f92c005a6513007ba1bef90ad8e48 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Stefan=20B=C3=BCrk?= <stefan@buerk.tech> Date: Wed, 26 Jun 2024 13:26:36 +0200 Subject: [PATCH] [TASK] Disable failing unit-case test data set MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Since recently upgraded PHP-version the 10mb big xml unit test failes. The `\LIBXML_PARSEHUGE` xml option could be used to mitigate this case but would not explain the change from PHP yet. Thus the decision has been made to disable the failing data set as a temporary solution to avoid disturbance for core contribution until this issue has been investigated and reported to PHP itself. Resolves: #104218 Releases: main, 12.4, 11.5 Change-Id: Ifef4f18818e661312b9e56e013bb8f479146b806 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84850 Tested-by: Stefan Bürk <stefan@buerk.tech> Tested-by: core-ci <typo3@b13.com> Reviewed-by: Stefan Bürk <stefan@buerk.tech> --- .../core/Tests/Unit/Serializer/Typo3XmlParserTest.php | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/typo3/sysext/core/Tests/Unit/Serializer/Typo3XmlParserTest.php b/typo3/sysext/core/Tests/Unit/Serializer/Typo3XmlParserTest.php index 81c0b556b786..220571807582 100644 --- a/typo3/sysext/core/Tests/Unit/Serializer/Typo3XmlParserTest.php +++ b/typo3/sysext/core/Tests/Unit/Serializer/Typo3XmlParserTest.php @@ -362,7 +362,9 @@ class Typo3XmlParserTest extends UnitTestCase return [ '1mb' => [1], '5mb' => [5], - '10mb' => [10], + // @todo Since 8.1.29 | 8.2.20 | 8.3.8 this case is broken. Needs deeper investigation and issue report + // on php itself. Using `\LIBXML_PARSEHUGE` would mitigate the issue, but have not been used before. + // '10mb' => [10], ]; } @@ -372,15 +374,15 @@ class Typo3XmlParserTest extends UnitTestCase */ public function decodeHandlesBigXmlContent(int $megabytes): void { + $testValue = str_repeat('1', $megabytes * 1024 * 1024); $input = '<?xml version="1.0" encoding="utf-8" standalone="yes"?> <T3FlexForms> <data> <field index="settings.persistenceIdentifier"> - <value index="vDEF">' . str_repeat('1', $megabytes * 1024 * 1024) . '</value> + <value index="vDEF">' . $testValue . '</value> </field> </data> </T3FlexForms>'; - $testValue = str_repeat('1', $megabytes * 1024 * 1024); $xmlDecoder = new Typo3XmlParser(); $expected = [ 'data' => [ -- GitLab