Changeset 53153 for trunk/tests/phpunit/tests/block-supports/colors.php

Timestamp:

04/12/2022 09:38:18 AM (3 years ago)

Author:

gziolo

Message:

Tests: Improve test cleanup for block supports

Let's unregister the block in the tear_down method. The block will be unregistered even if the test fails.

Props ramonopoly, antonvlasenko.
See #55505.
Follow-up [53085], [53076].

File:

• trunk/tests/phpunit/tests/block-supports/colors.php (modified) (10 diffs)

trunk/tests/phpunit/tests/block-supports/colors.php

@@ -5 +5 @@
 class Tests_Block_Supports_Colors extends WP_UnitTestCase {
     /**
+     * @var string|null
+     */
+    private $test_block_name;
+
+    function set_up() {
+        parent::set_up();
+        $this->test_block_name = null;
+    }
+
+    function tear_down() {
+        unregister_block_type( $this->test_block_name );
+        $this->test_block_name = null;
+        parent::set_up();
+    }
+
+    /**
      * @ticket 54337
      *
@@ -10 +26 @@
      */
     function test_color_slugs_with_numbers_are_kebab_cased_properly() {
+        $this->test_block_name = 'test/color-slug-with-numbers';
         register_block_type(
-            'test/color-slug-with-numbers',
+            $this->test_block_name,
             array(
                 'api_version' => 2,
@@ -35 +52 @@
         );
         $registry   = WP_Block_Type_Registry::get_instance();
-        $block_type = $registry->get_registered( 'test/color-slug-with-numbers' );
+        $block_type = $registry->get_registered( $this->test_block_name );
 
         $block_atts = array(
@@ -47 +64 @@
 
         $this->assertSame( $expected, $actual );
-        unregister_block_type( 'test/color-slug-with-numbers' );
     }
 
@@ -56 +72 @@
      */
     function test_color_with_skipped_serialization_block_supports() {
-        $block_name = 'test/color-with-skipped-serialization-block-supports';
+        $this->test_block_name = 'test/color-with-skipped-serialization-block-supports';
         register_block_type(
-            $block_name,
+            $this->test_block_name,
             array(
                 'api_version' => 2,
@@ -77 +93 @@
 
         $registry   = WP_Block_Type_Registry::get_instance();
-        $block_type = $registry->get_registered( $block_name );
+        $block_type = $registry->get_registered( $this->test_block_name );
         $block_atts = array(
             'style' => array(
@@ -91 +107 @@
 
         $this->assertSame( $expected, $actual );
-        unregister_block_type( $block_name );
     }
 
@@ -100 +115 @@
      */
     function test_gradient_with_individual_skipped_serialization_block_supports() {
-        $block_name = 'test/gradient-with-individual-skipped-serialization-block-support';
+        $this->test_block_name = 'test/gradient-with-individual-skipped-serialization-block-support';
         register_block_type(
-            $block_name,
+            $this->test_block_name,
             array(
                 'api_version' => 2,
@@ -121 +136 @@
 
         $registry   = WP_Block_Type_Registry::get_instance();
-        $block_type = $registry->get_registered( $block_name );
+        $block_type = $registry->get_registered( $this->test_block_name );
         $block_atts = array(
             'style' => array(
@@ -137 +152 @@
 
         $this->assertSame( $expected, $actual );
-        unregister_block_type( $block_name );
     }
 }