Changeset 30320 for trunk/tests/phpunit/tests/db.php

Timestamp:

11/13/2014 01:59:49 AM (10 years ago)

Author:

boonebgorges

Message:

Manually delete fixture in test_mysqli_flush_sync().

This test creates a dummy post and subsequently runs a query containing
DROP PROCEDURE. This latter query implies a COMMIT, which means that the
post is not cleaned up for later tests. Manually deleting the post with
wp_delete_post() solves this problem.

Fixes #28155.

File:

• trunk/tests/phpunit/tests/db.php (modified) (1 diff)

trunk/tests/phpunit/tests/db.php

@@ -483 +483 @@
         }
 
-        $this->factory->post->create();
+        $post_id = $this->factory->post->create();
 
         $this->assertNotEmpty( $wpdb->get_results( 'CALL `test_mysqli_flush_sync_procedure`' ) );
         $this->assertNotEmpty( $wpdb->get_results( "SELECT ID FROM `{$wpdb->posts}` LIMIT 1" ) );
 
+        // DROP PROCEDURE will cause a COMMIT, so we delete the post manually before that happens.
+        wp_delete_post( $post_id, true );
+
         $wpdb->query( 'DROP PROCEDURE IF EXISTS `test_mysqli_flush_sync_procedure`' );
         $wpdb->suppress_errors( $suppress );