Fixing various tests and deprecations

symfony · Aug 24, 2023 · ac930c9 · ac930c9
1 parent 02d661b
commit ac930c9
Show file tree

Hide file tree

Showing 8 changed files with 50 additions and 37 deletions.
diff --git a/src/Cropperjs/tests/Form/CropperTypeTest.php b/src/Cropperjs/tests/Form/CropperTypeTest.php
@@ -44,21 +44,13 @@ public function testRenderFull()
 
         $rendered = $container->get(Environment::class)->render('cropper_form.html.twig', ['form' => $form->createView()]);
 
-        $this->assertSame(
-            '<form name="form" method="post">'.
-                '<div id="form">'.
-                    '<div>'.
-                        '<div id="form_photo" class="cropperjs">'.
-                            '<input type="hidden" id="form_photo_options" name="form[photo][options]" '.
-                                'data-controller="mycropper symfony--ux-cropperjs--cropper" '.
-                                'data-symfony--ux-cropperjs--cropper-public-url-value="/public/url.jpg" '.
-                                'data-symfony--ux-cropperjs--cropper-options-value="{&quot;viewMode&quot;:1,&quot;dragMode&quot;:&quot;move&quot;}" />'.
-                        '</div>'.
-                    '</div>'.
-                '</div>'.
-            '</form>
-',
-            str_replace(' >', '>', $rendered)
+        $this->assertStringContainsString('<div id="form_photo" class="cropperjs">', $rendered);
+        $this->assertStringContainsString(
+            '<input type="hidden" id="form_photo_options" name="form[photo][options]" '.
+                'data-controller="mycropper symfony--ux-cropperjs--cropper" '.
+                'data-symfony--ux-cropperjs--cropper-public-url-value="/public/url.jpg" '.
+                'data-symfony--ux-cropperjs--cropper-options-value="{&quot;viewMode&quot;:1,&quot;dragMode&quot;:&quot;move&quot;}"',
+            $rendered,
         );
     }
 
@@ -78,20 +70,11 @@ public function testRenderNoOptions()
 
         $rendered = $container->get(Environment::class)->render('cropper_form.html.twig', ['form' => $form->createView()]);
 
-        $this->assertSame(
-            '<form name="form" method="post">'.
-                '<div id="form">'.
-                    '<div>'.
-                        '<div id="form_photo" class="cropperjs">'.
-                            '<input type="hidden" id="form_photo_options" name="form[photo][options]" '.
-                                'data-controller="mycropper symfony--ux-cropperjs--cropper" '.
-                                'data-symfony--ux-cropperjs--cropper-public-url-value="/public/url.jpg" '.
-                                'data-symfony--ux-cropperjs--cropper-options-value="[]" />'.
-                        '</div>'.
-                    '</div>'.
-                '</div>'.
-            '</form>
-',
+        $this->assertStringContainsString(
+            '<input type="hidden" id="form_photo_options" name="form[photo][options]" '.
+                'data-controller="mycropper symfony--ux-cropperjs--cropper" '.
+                'data-symfony--ux-cropperjs--cropper-public-url-value="/public/url.jpg" '.
+                'data-symfony--ux-cropperjs--cropper-options-value="[]"',
             str_replace(' >', '>', $rendered)
         );
     }

diff --git a/src/Cropperjs/tests/Kernel/FrameworkAppKernel.php b/src/Cropperjs/tests/Kernel/FrameworkAppKernel.php
@@ -34,7 +34,16 @@ public function registerBundles(): iterable
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
         $loader->load(function (ContainerBuilder $container) {
-            $container->loadFromExtension('framework', ['secret' => '$ecret', 'test' => true]);
+            $container->loadFromExtension('framework', [
+                'secret' => '$ecret',
+                'test' => true,
+                'http_method_override' => false,
+                'handle_all_throwables' => true,
+                'php_errors' => ['log' => true],
+                'validation' => [
+                    'email_validation_mode' => 'html5',
+                ],
+            ]);
         });
     }
 }
diff --git a/src/Cropperjs/tests/Kernel/TwigAppKernel.php b/src/Cropperjs/tests/Kernel/TwigAppKernel.php
@@ -37,7 +37,16 @@ public function registerBundles(): iterable
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
         $loader->load(function (ContainerBuilder $container) {
-            $container->loadFromExtension('framework', ['secret' => '$ecret', 'test' => true, 'http_method_override' => false]);
+            $container->loadFromExtension('framework', [
+                'secret' => '$ecret',
+                'test' => true,
+                'http_method_override' => false,
+                'handle_all_throwables' => true,
+                'php_errors' => ['log' => true],
+                'validation' => [
+                    'email_validation_mode' => 'html5',
+                ],
+            ]);
             $container->loadFromExtension('twig', ['default_path' => __DIR__.'/templates', 'strict_variables' => true, 'exception_controller' => null]);
 
             // create a public alias - FormFactoryInterface is removed otherwise

diff --git a/src/StimulusBundle/composer.json b/src/StimulusBundle/composer.json
@@ -21,7 +21,7 @@
         "twig/twig": "^2.15.3|^3.4.3"
     },
     "require-dev": {
-        "symfony/asset-mapper": "6.3.x-dev",
+        "symfony/asset-mapper": "^6.3",
         "symfony/framework-bundle": "^5.4|^6.0",
         "symfony/phpunit-bridge": "^5.4|^6.0",
         "symfony/twig-bundle": "^5.4|^6.0",

diff --git a/src/StimulusBundle/phpunit.xml.dist b/src/StimulusBundle/phpunit.xml.dist
@@ -31,4 +31,8 @@
     <listeners>
         <listener class="Symfony\Bridge\PhpUnit\SymfonyTestsListener"/>
     </listeners>
+
+    <extensions>
+        <extension class="Zenstruck\Browser\Test\BrowserExtension" />
+    </extensions>
 </phpunit>
diff --git a/src/StimulusBundle/tests/fixtures/StimulusTestKernel.php b/src/StimulusBundle/tests/fixtures/StimulusTestKernel.php
@@ -57,6 +57,8 @@ protected function configureContainer(ContainerConfigurator $container): void
                 ],
             ],
             'test' => true,
+            'handle_all_throwables' => true,
+            'php_errors' => ['log' => true],
         ]);
 
         $container->extension('twig', [

diff --git a/src/TogglePassword/tests/Form/TogglePasswordTypeTest.php b/src/TogglePassword/tests/Form/TogglePasswordTypeTest.php
@@ -22,8 +22,7 @@ public function testRenderFormWithToggle(): void
 
         $rendered = $container->get(Environment::class)->render('toggle_password_form.html.twig', ['form' => $form->createView()]);
 
-        self::assertSame('<form name="form" method="post"><div id="form"><div><label for="form_password" class="required">Password</label><div class="toggle-password-container"><input type="password" id="form_password" name="form[password]" required="required" data-controller="symfony--ux-toggle-password--toggle-password" data-symfony--ux-toggle-password--toggle-password-hidden-label-value="Hide" data-symfony--ux-toggle-password--toggle-password-visible-label-value="Show" data-symfony--ux-toggle-password--toggle-password-hidden-icon-value="Default" data-symfony--ux-toggle-password--toggle-password-visible-icon-value="Default" data-symfony--ux-toggle-password--toggle-password-button-classes-value="[&quot;toggle-password-button&quot;]" /></div></div></div></form>
-', $rendered);
+        self::assertStringContainsString('<div class="toggle-password-container"><input type="password" id="form_password" name="form[password]" required="required" data-controller="symfony--ux-toggle-password--toggle-password" data-symfony--ux-toggle-password--toggle-password-hidden-label-value="Hide" data-symfony--ux-toggle-password--toggle-password-visible-label-value="Show" data-symfony--ux-toggle-password--toggle-password-hidden-icon-value="Default" data-symfony--ux-toggle-password--toggle-password-visible-icon-value="Default" data-symfony--ux-toggle-password--toggle-password-button-classes-value="[&quot;toggle-password-button&quot;]"', $rendered);
     }
 
     public function testRenderFormWithoutToggle(): void
@@ -36,8 +35,7 @@ public function testRenderFormWithoutToggle(): void
 
         $rendered = $container->get(Environment::class)->render('toggle_password_form.html.twig', ['form' => $form->createView()]);
 
-        self::assertSame('<form name="form" method="post"><div id="form"><div><label for="form_password" class="required">Password</label><input type="password" id="form_password" name="form[password]" required="required" /></div></div></form>
-', $rendered);
+        self::assertStringContainsString('<input type="password" id="form_password" name="form[password]" required="required"', $rendered);
     }
 
     private function givenServiceContainer(): ?object

diff --git a/src/TogglePassword/tests/Kernel/TwigAppKernel.php b/src/TogglePassword/tests/Kernel/TwigAppKernel.php
@@ -35,7 +35,15 @@ public function registerBundles(): iterable
     public function registerContainerConfiguration(LoaderInterface $loader): void
     {
         $loader->load(function (ContainerBuilder $container) {
-            $container->loadFromExtension('framework', ['secret' => '$ecret', 'test' => true, 'http_method_override' => false]);
+            $container->loadFromExtension('framework', [
+                'secret' => '$ecret',
+                'test' => true,
+                'http_method_override' => false,
+                'handle_all_throwables' => true,
+                'php_errors' => [
+                    'log' => true,
+                ],
+            ]);
             $container->loadFromExtension('twig', [
                 'default_path' => __DIR__.'/templates',
                 'strict_variables' => true,