Changeset 42343 for trunk/tests/phpunit/tests/category.php

Timestamp:

11/30/2017 11:09:33 PM (8 years ago)

Author:

pento

Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:

• trunk/tests/phpunit/tests/category.php (modified) (10 diffs)

trunk/tests/phpunit/tests/category.php

@@ -26 +26 @@
         // create new taxonomy to ensure not included
         register_taxonomy( 'test_tax_cat', 'post' );
-        wp_insert_term( "test1", 'test_tax_cat' );
+        wp_insert_term( 'test1', 'test_tax_cat' );
 
         // Validate length is 1 + created due to uncategorized
         $cat_ids = get_all_category_ids();
-        $this->assertEquals( 3, count($cat_ids));
+        $this->assertEquals( 3, count( $cat_ids ) );
     }
 
@@ -39 +39 @@
 
         // create Test Categories
-        $testcat = self::factory()->category->create_and_get(
+        $testcat  = self::factory()->category->create_and_get(
             array(
                 'slug' => 'testcat',
-                'name' => 'Test Category 1'
+                'name' => 'Test Category 1',
             )
         );
@@ -48 +48 @@
             array(
                 'slug' => 'testcat2',
-                'name' => 'Test Category 2'
+                'name' => 'Test Category 2',
             )
         );
@@ -69 +69 @@
 
         // create Test Categories and Array Representations
-        $testcat_array = array(
-            'slug' => 'testmcc',
-            'name' => 'Test MCC',
-            'description' => 'Category Test'
-        );
-        $testcat = self::factory()->category->create_and_get( $testcat_array );
+        $testcat_array            = array(
+            'slug'        => 'testmcc',
+            'name'        => 'Test MCC',
+            'description' => 'Category Test',
+        );
+        $testcat                  = self::factory()->category->create_and_get( $testcat_array );
         $testcat_array['term_id'] = $testcat->term_id;
 
-        $testcat2_array = array(
-            'slug' => 'testmcc',
-            'name' => 'Test MCC',
+        $testcat2_array            = array(
+            'slug'        => 'testmcc',
+            'name'        => 'Test MCC',
             'description' => 'Category Test',
-            'parent' => $testcat->term_id
-        );
-        $testcat2 = self::factory()->category->create_and_get( $testcat2_array );
+            'parent'      => $testcat->term_id,
+        );
+        $testcat2                  = self::factory()->category->create_and_get( $testcat2_array );
         $testcat2_array['term_id'] = $testcat2->term_id;
 
@@ -149 +149 @@
             array(
                 'slug' => 'testcat',
-                'name' => 'Test Category 1'
+                'name' => 'Test Category 1',
             )
         );
@@ -169 +169 @@
             array(
                 'slug' => 'testcat',
-                'name' => 'Test Category 1'
+                'name' => 'Test Category 1',
             )
         );
@@ -175 +175 @@
         // Validate
         $this->assertEquals( $testcat->term_id, get_cat_ID( $testcat->name ) );
-        $this->assertEquals( 0, get_cat_ID( "NO CAT" ) );
+        $this->assertEquals( 0, get_cat_ID( 'NO CAT' ) );
         $this->assertEquals( 0, get_cat_ID( 12 ) );
 
@@ -186 +186 @@
 
         // create Test Categories
-        $root_id = self::factory()->category->create(
+        $root_id           = self::factory()->category->create(
             array(
                 'slug' => 'root',
             )
         );
-        $root_cat_id = self::factory()->category->create(
-            array(
-                'slug' => 'cat',
-                'parent' => $root_id
-            )
-        );
-        $root_cat_cat_id = self::factory()->category->create(
-            array(
-                'slug' => 'cat', //note this is modified on create
-                'parent' => $root_cat_id
-            )
-        );
-        $root_path_id = self::factory()->category->create(
-            array(
-                'slug' => 'path',
-                'parent' => $root_id
-            )
-        );
-        $root_path_cat_id = self::factory()->category->create(
-            array(
-                'slug' => 'cat', //note this is modified on create
-                'parent' => $root_path_id
-            )
-        );
-        $root_level_id = self::factory()->category->create(
-            array(
-                'slug' => 'level-1',
-                'parent' => $root_id
+        $root_cat_id       = self::factory()->category->create(
+            array(
+                'slug'   => 'cat',
+                'parent' => $root_id,
+            )
+        );
+        $root_cat_cat_id   = self::factory()->category->create(
+            array(
+                'slug'   => 'cat', //note this is modified on create
+                'parent' => $root_cat_id,
+            )
+        );
+        $root_path_id      = self::factory()->category->create(
+            array(
+                'slug'   => 'path',
+                'parent' => $root_id,
+            )
+        );
+        $root_path_cat_id  = self::factory()->category->create(
+            array(
+                'slug'   => 'cat', //note this is modified on create
+                'parent' => $root_path_id,
+            )
+        );
+        $root_level_id     = self::factory()->category->create(
+            array(
+                'slug'   => 'level-1',
+                'parent' => $root_id,
             )
         );
         $root_level_cat_id = self::factory()->category->create(
             array(
-                'slug' => 'cat', //note this is modified on create
-                'parent' => $root_level_id
+                'slug'   => 'cat', //note this is modified on create
+                'parent' => $root_level_id,
             )
         );
@@ -232 +232 @@
         $this->assertEquals( $root_level_id, $ret_cat->term_id );
         $this->assertNull( get_category_by_path( 'level-1', true ) );
-        $this->assertNull( get_category_by_path( 'nocat/nocat/', true) );
+        $this->assertNull( get_category_by_path( 'nocat/nocat/', true ) );
 
         // Validate Partial Match
@@ -241 +241 @@
         $ret_cat = get_category_by_path( 'root$2Fcat%20%2Flevel-1', false );
         $this->assertEquals( $root_level_id, $ret_cat->term_id );
-        $this->assertNull( get_category_by_path( 'nocat/nocat/', false) );
+        $this->assertNull( get_category_by_path( 'nocat/nocat/', false ) );
     }
 }