minor #53335 [Form] skip some tests that do not work with ICU 71.1/72.1 (xabbuh)

xabbuh · xabbuh · commit 37282f7cb6b3 · 2024-01-01T13:23:37.000+01:00
This PR was merged into the 5.4 branch. Discussion ---------- [Form] skip some tests that do not work with ICU 71.1/72.1 | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | replaces #52967 | License | MIT Commits ------- 8ef61d9 skip some tests that do not work with ICU 71.1/72.1
diff --git a/src/Symfony/Component/Form/Tests/Extension/Core/Type/DateTypeTest.php b/src/Symfony/Component/Form/Tests/Extension/Core/Type/DateTypeTest.php
@@ -91,8 +91,8 @@ public function testSubmitFromSingleTextDateTime()
         // we test against "de_DE", so we need the full implementation
         IntlTestHelper::requireFullIntl($this, false);
 
-        if ('71.1' === Intl::getIcuVersion()) {
-            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1.');
+        if (\in_array(Intl::getIcuVersion(), ['71.1', '72.1'], true)) {
+            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1/72.1.');
         }
 
         \Locale::setDefault('de_DE');
@@ -117,8 +117,8 @@ public function testSubmitFromSingleTextDateTimeImmutable()
         // we test against "de_DE", so we need the full implementation
         IntlTestHelper::requireFullIntl($this, false);
 
-        if ('71.1' === Intl::getIcuVersion()) {
-            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1.');
+        if (\in_array(Intl::getIcuVersion(), ['71.1', '72.1'], true)) {
+            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1/72.1.');
         }
 
         \Locale::setDefault('de_DE');
@@ -144,8 +144,8 @@ public function testSubmitFromSingleTextString()
         // we test against "de_DE", so we need the full implementation
         IntlTestHelper::requireFullIntl($this, false);
 
-        if ('71.1' === Intl::getIcuVersion()) {
-            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1.');
+        if (\in_array(Intl::getIcuVersion(), ['71.1', '72.1'], true)) {
+            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1/72.1.');
         }
 
         \Locale::setDefault('de_DE');
@@ -170,8 +170,8 @@ public function testSubmitFromSingleTextTimestamp()
         // we test against "de_DE", so we need the full implementation
         IntlTestHelper::requireFullIntl($this, false);
 
-        if ('71.1' === Intl::getIcuVersion()) {
-            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1.');
+        if (\in_array(Intl::getIcuVersion(), ['71.1', '72.1'], true)) {
+            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1/72.1.');
         }
 
         \Locale::setDefault('de_DE');
@@ -198,8 +198,8 @@ public function testSubmitFromSingleTextRaw()
         // we test against "de_DE", so we need the full implementation
         IntlTestHelper::requireFullIntl($this, false);
 
-        if ('71.1' === Intl::getIcuVersion()) {
-            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1.');
+        if (\in_array(Intl::getIcuVersion(), ['71.1', '72.1'], true)) {
+            $this->markTestSkipped('Skipping test due to a bug in ICU 71.1/72.1.');
         }
 
         \Locale::setDefault('de_DE');
