From de55e39c66d739fcba6e1d4cd1dbd85e1d4220ec Mon Sep 17 00:00:00 2001 From: Eddie Carrasco Date: Tue, 2 Jul 2024 18:14:09 -0400 Subject: [PATCH] fix: removing repetitive test. --- tests/Integration/CoAuthorsPlusTest.php | 211 ------------------------ 1 file changed, 211 deletions(-) diff --git a/tests/Integration/CoAuthorsPlusTest.php b/tests/Integration/CoAuthorsPlusTest.php index 15a4e149..016fa5ae 100644 --- a/tests/Integration/CoAuthorsPlusTest.php +++ b/tests/Integration/CoAuthorsPlusTest.php @@ -1917,217 +1917,6 @@ public function test_assign_multiple_post_authors_one_linked_passed_using_user_l $this->assertEquals( 'cap-' . $linked_author_1->user_login, $guest_author_term[0]->slug ); } - /** - * This test is a combination of a few different aspects that establish some expected plugin behavior. - * Two posts are created to test with. Both have no WP_User as the author (i.e. wp_posts.post_author = 0). - * For the first post: - * 1. A guest author is created, and assigned to the post. - * 2. Since there is no WP_User which is passed, and the GA is NOT being appended, the result should be false. - * 3. The wp_posts.post_author column should still be 0 since this is a GA and not a WP_User or a linked GA. - * - * For the second post: - * 1. A guest author is created, and appended to the post. - * 2. Since we are appending a coauthor, it does not matter if there was already a WP_User author, so result should be true. - * 3. The wp_posts.post_author column should still be 0 since this is a GA and not a WP_User or a linked GA. - * - * Going back to the first post: - * 1. A linked user and a WP_User are created and assigned to this post. - * 2. Result should be true since here we essentially have 2 WP_Users. - * 3. Since there is a WP_User, and it is passed first, the wp_posts.post_author column should be set to the ID for that WP_User. - * 4. There should only be 2 author terms for this post, one for the WP_User, and one for the linked account. - * The term for the GA which was previously assigned is deleted. - * - * Finally, going back to the second post: - * 1. A linked user and a WP_User are created and appended to this post. - * 2. Result should be true since we have 2 WP_Users. - * 3. Here we passed the linked author first, so the wp_posts.post_author column should match the ID for the linked WP_user account. - * 4. There should be 3 author terms for this post, one for the GA, the WP_user, and linked account. - * @return void - */ - public function test_assign_post_authors_from_post_with_no_author() { - $post_id_1 = $this->factory()->post->create( - array( - 'post_author' => 0, - 'post_status' => 'publish', - 'post_type' => 'post', - ) - ); - - $query = new WP_Query( - array( - 'p' => $post_id_1, - ) - ); - - $this->assertEquals( 1, $query->found_posts ); - $this->assertEquals( 0, $query->posts[0]->post_author ); - - $post_id_2 = $this->factory()->post->create( - array( - 'post_author' => 0, - 'post_status' => 'publish', - 'post_type' => 'post', - ) - ); - - $second_post_query = new WP_Query( - array( - 'p' => $post_id_2, - ) - ); - - $this->assertEquals( 1, $second_post_query->found_posts ); - $this->assertEquals( 0, $second_post_query->posts[0]->post_author ); - - $random_username = 'random_user_' . rand( 90000, 100000 ); - $display_name = str_replace( '_', ' ', $random_username ); - - $guest_author_1_id = $this->_cap->guest_authors->create( - array( - 'user_login' => $random_username, - 'display_name' => $display_name, - ) - ); - $guest_author_1 = $this->_cap->get_coauthor_by( 'id', $guest_author_1_id ); - - $this->assertIsObject( $guest_author_1 ); - $this->assertThat( - $guest_author_1, - $this->logicalNot( - $this->isInstanceOf( WP_User::class ) - ) - ); - - $this->_cap->guest_authors->create_guest_author_from_user_id( $this->author3->ID ); - - $linked_author_1 = $this->_cap->get_coauthor_by( 'id', $this->author3->ID ); - $this->assertIsObject( $linked_author_1 ); - $this->assertThat( - $linked_author_1, - $this->logicalNot( - $this->isInstanceOf( WP_User::class ) - ) - ); - $this->assertTrue( property_exists( $linked_author_1, 'is_wp_user' ) ); - $this->assertTrue( $linked_author_1->is_wp_user ); - $this->assertTrue( property_exists( $linked_author_1, 'wp_user' ) ); - $this->assertInstanceOf( WP_User::class, $linked_author_1->wp_user ); - $this->assertEquals( $this->author3->ID, $linked_author_1->wp_user->ID ); - - $result = $this->_cap->add_coauthors( - $post_id_1, - array( - $guest_author_1->user_login, - ) - ); - - $this->assertFalse( $result ); - - $post_1_author_terms = wp_get_post_terms( $post_id_1, $this->_cap->coauthor_taxonomy ); - - $this->assertIsArray( $post_1_author_terms ); - $this->assertCount( 1, $post_1_author_terms ); - $this->assertEquals( 'cap-' . $guest_author_1->user_login, $post_1_author_terms[0]->slug ); - - $third_post_query = new WP_Query( - array( - 'p' => $post_id_1, - ) - ); - - $this->assertEquals( 1, $third_post_query->found_posts ); - $this->assertEquals( 0, $third_post_query->posts[0]->post_author ); - - $result = $this->_cap->add_coauthors( - $post_id_2, - array( - $guest_author_1->user_login, - ), - true - ); - - $this->assertTrue( $result ); - - $fourth_post_query = new WP_Query( - array( - 'p' => $post_id_2, - ) - ); - - $this->assertEquals( 1, $fourth_post_query->found_posts ); - $this->assertEquals( 0, $fourth_post_query->posts[0]->post_author ); - - $result = $this->_cap->add_coauthors( - $post_id_1, - array( - $this->author2->user_login, - $linked_author_1->user_login, - ) - ); - - $this->assertTrue( $result ); - - $fifth_post_query = new WP_Query( - array( - 'p' => $post_id_1, - ) - ); - - $this->assertEquals( 1, $fifth_post_query->found_posts ); - $this->assertEquals( $this->author2->ID, $fifth_post_query->posts[0]->post_author ); - - $post_author_terms = wp_get_post_terms( $post_id_1, $this->_cap->coauthor_taxonomy ); - - $this->assertIsArray( $post_author_terms ); - $this->assertCount( 2, $post_author_terms ); - - $author_slugs = array( - 'cap-' . $this->author2->user_login, - 'cap-' . $linked_author_1->user_login, - ); - - foreach ( $post_author_terms as $term ) { - $this->assertInstanceOf( WP_Term::class, $term ); - $this->assertContains( $term->slug, $author_slugs ); - } - - $result = $this->_cap->add_coauthors( - $post_id_2, - array( - $linked_author_1->user_login, - $this->author2->user_login, - ), - true - ); - - $this->assertTrue( $result ); - - $sixth_post_query = new WP_Query( - array( - 'p' => $post_id_2, - ) - ); - - $this->assertEquals( 1, $sixth_post_query->found_posts ); - $this->assertEquals( $this->author3->ID, $sixth_post_query->posts[0]->post_author ); - - $post_author_terms = wp_get_post_terms( $post_id_2, $this->_cap->coauthor_taxonomy ); - - $this->assertIsArray( $post_author_terms ); - $this->assertCount( 3, $post_author_terms ); - - $author_slugs = array( - 'cap-' . $guest_author_1->user_login, - 'cap-' . $this->author2->user_login, - 'cap-' . $linked_author_1->user_login, - ); - - foreach ( $post_author_terms as $term ) { - $this->assertInstanceOf( WP_Term::class, $term ); - $this->assertContains( $term->slug, $author_slugs ); - } - } - /** * @covers CoAuthors_Plus::is_block_editor() */