Changeset 53153 for trunk/tests/phpunit/tests/block-supports/border.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/border.php (modified) (10 diffs)

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

@@ -5 +5 @@
 class Test_Block_Supports_Border 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 55505
      *
@@ -10 +26 @@
      */
     function test_border_color_slug_with_numbers_is_kebab_cased_properly() {
-        $block_name = 'test/border-color-slug-with-numbers-is-kebab-cased-properly';
+        $this->test_block_name = 'test/border-color-slug-with-numbers-is-kebab-cased-properly';
         register_block_type(
-            $block_name,
+            $this->test_block_name,
             array(
                 'api_version' => 2,
@@ -34 +50 @@
         );
         $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(
             'borderColor' => 'red',
@@ -53 +69 @@
 
         $this->assertSame( $expected, $actual );
-        unregister_block_type( $block_name );
     }
 
@@ -62 +77 @@
      */
     function test_border_with_skipped_serialization_block_supports() {
-        $block_name = 'test/border-with-skipped-serialization-block-supports';
+        $this->test_block_name = 'test/border-with-skipped-serialization-block-supports';
         register_block_type(
-            $block_name,
+            $this->test_block_name,
             array(
                 'api_version' => 2,
@@ -84 +99 @@
         );
         $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(
@@ -100 +115 @@
 
         $this->assertSame( $expected, $actual );
-        unregister_block_type( $block_name );
     }
 
@@ -109 +123 @@
      */
     function test_radius_with_individual_skipped_serialization_block_supports() {
-        $block_name = 'test/radius-with-individual-skipped-serialization-block-supports';
+        $this->test_block_name = 'test/radius-with-individual-skipped-serialization-block-supports';
         register_block_type(
-            $block_name,
+            $this->test_block_name,
             array(
                 'api_version' => 2,
@@ -131 +145 @@
         );
         $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(
@@ -149 +163 @@
 
         $this->assertSame( $expected, $actual );
-        unregister_block_type( $block_name );
     }
 }