Changeset 35225 for trunk/tests/phpunit/tests/link/getAdjacentPostLink.php

Timestamp:

10/16/2015 09:04:12 PM (10 years ago)

Author:

wonderboymusic

Message:

Unit Tests: one $factory to rule them all, and it shall be static.

Using more than one instance of WP_UnitTest_Factory causes all kinds of craziness, due to out-of-sync internal generator sequences. Since we want to use setUpBeforeClass, we were creating ad hoc instances. To avoid that, we were injecting one static instance via Dependency Injection in wpSetUpBeforeClass. All tests should really use the static instance, so we will remove the instance prop $factory.

Replace $this->factory with self::$factory over 2000 times.
Rewrite all of the tests that were hard-coding dynamic values.

#YOLOFriday

File:

• trunk/tests/phpunit/tests/link/getAdjacentPostLink.php (modified) (7 diffs)

trunk/tests/phpunit/tests/link/getAdjacentPostLink.php

@@ -11 +11 @@
     public function setUp(){
         parent::setUp();
-        $this->cat_id = $this->factory->category->create( array( 'name' => 'other' ) );
+        $this->cat_id = self::$factory->category->create( array( 'name' => 'other' ) );
         $this->post_ids = array();
-        $this->post_ids[] = $this->factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 05:32:29', 'category_id' => 1 ) );
-        $this->post_ids[] = $this->factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 04:32:29', 'category_id' => $this->cat_id ) );
-        $this->post_ids[] = $this->factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 03:32:29', 'category_id' => 1 ) );
-        $this->post_ids[] = $this->factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 02:32:29', 'category_id' => $this->cat_id ) );
-        $this->post_ids[] = $this->factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 01:32:29', 'category_id' => 1 ) );
+        $this->post_ids[] = self::$factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 05:32:29', 'category_id' => 1 ) );
+        $this->post_ids[] = self::$factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 04:32:29', 'category_id' => $this->cat_id ) );
+        $this->post_ids[] = self::$factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 03:32:29', 'category_id' => 1 ) );
+        $this->post_ids[] = self::$factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 02:32:29', 'category_id' => $this->cat_id ) );
+        $this->post_ids[] = self::$factory->post->create( array( 'post_type' => 'post', 'post_date' => '2014-10-26 01:32:29', 'category_id' => 1 ) );
 
         //set current post (has 2 on each end)
@@ -26 +26 @@
     public function test_get_next_post_link_default() {
         $actual = get_next_post_link();
-        $expected = '<a href="' . home_url( '?p=' . $this->post_ids[1] ) . '" rel="next">Post title 2</a> &raquo;';
+        $title = get_post( $this->post_ids[1] )->post_title;
+        $expected = '<a href="' . home_url( '?p=' . $this->post_ids[1] ) . '" rel="next">' . $title . '</a> &raquo;';
         $this->assertSame( $expected, $actual );
     }
@@ -32 +33 @@
     public function test_get_previous_post_link_default() {
         $actual = get_previous_post_link();
-        $expected = '&laquo; <a href="' . home_url( '?p=' . $this->post_ids[3] ) . '" rel="prev">Post title 4</a>';
+        $title = get_post( $this->post_ids[3] )->post_title;
+        $expected = '&laquo; <a href="' . home_url( '?p=' . $this->post_ids[3] ) . '" rel="prev">' . $title . '</a>';
         $this->assertSame( $expected, $actual );
     }
@@ -38 +40 @@
     public function test_get_next_post_link_same_category() {
         $actual = get_next_post_link( '%link &raquo;', '%title', true );
-        $expected = '<a href="' . home_url( '?p=' . $this->post_ids[1] ) . '" rel="next">Post title 2</a> &raquo;';
+        $title = get_post( $this->post_ids[1] )->post_title;
+        $expected = '<a href="' . home_url( '?p=' . $this->post_ids[1] ) . '" rel="next">' . $title . '</a> &raquo;';
         $this->assertSame( $expected, $actual );
     }
@@ -44 +47 @@
     public function test_get_previous_post_link_same_category() {
         $actual = get_previous_post_link( '&laquo; %link', '%title', true );
-        $expected = '&laquo; <a href="' . home_url( '?p=' . $this->post_ids[3] ) . '" rel="prev">Post title 4</a>';
+        $title = get_post( $this->post_ids[3] )->post_title;
+        $expected = '&laquo; <a href="' . home_url( '?p=' . $this->post_ids[3] ) . '" rel="prev">' . $title . '</a>';
         $this->assertSame( $expected, $actual );
     }
@@ -50 +54 @@
     public function test_get_next_post_link_exclude_category() {
         $actual = get_next_post_link( '%link &raquo;', '%title', false, $this->cat_id );
-        $expected = '<a href="' . home_url( '?p=' . $this->post_ids[1] ) . '" rel="next">Post title 2</a> &raquo;';
+        $title = get_post( $this->post_ids[1] )->post_title;
+        $expected = '<a href="' . home_url( '?p=' . $this->post_ids[1] ) . '" rel="next">' . $title . '</a> &raquo;';
         $this->assertSame( $expected, $actual );
     }
@@ -56 +61 @@
     public function test_get_previous_post_link_exclude_category() {
         $actual = get_previous_post_link( '&laquo; %link', '%title', false, $this->cat_id );
-        $expected = '&laquo; <a href="' . home_url( '?p=' . $this->post_ids[3] ) . '" rel="prev">Post title 4</a>';
+        $title = get_post( $this->post_ids[3] )->post_title;
+        $expected = '&laquo; <a href="' . home_url( '?p=' . $this->post_ids[3] ) . '" rel="prev">' . $title . '</a>';
         $this->assertSame( $expected, $actual );
     }