diff --git a/phpcs.xml b/phpcs.xml index bfe78776..45f87d61 100644 --- a/phpcs.xml +++ b/phpcs.xml @@ -49,6 +49,7 @@ + tests/phpunit/logging/ActionScheduler_wpCommentLogger_Test.php classes/data-stores/ActionScheduler_wpPostStore_PostStatusRegistrar.php classes/data-stores/ActionScheduler_wpPostStore_TaxonomyRegistrar.php tests/phpunit/jobstore/ActionScheduler_wpPostStore_Test.php diff --git a/tests/phpunit/logging/ActionScheduler_wpCommentLogger_Test.php b/tests/phpunit/logging/ActionScheduler_wpCommentLogger_Test.php index d2f3fccc..b5d41db0 100644 --- a/tests/phpunit/logging/ActionScheduler_wpCommentLogger_Test.php +++ b/tests/phpunit/logging/ActionScheduler_wpCommentLogger_Test.php @@ -5,6 +5,8 @@ * @package test_cases\logging */ class ActionScheduler_wpCommentLogger_Test extends ActionScheduler_UnitTestCase { + + /** @var bool */ private $use_comment_logger; public function test_default_logger() { @@ -19,10 +21,10 @@ public function test_default_logger() { public function test_add_log_entry() { $action_id = as_schedule_single_action( time(), 'a hook' ); - $logger = ActionScheduler::logger(); - $message = 'Logging that something happened'; - $log_id = $logger->log( $action_id, $message ); - $entry = $logger->get_entry( $log_id ); + $logger = ActionScheduler::logger(); + $message = 'Logging that something happened'; + $log_id = $logger->log( $action_id, $message ); + $entry = $logger->get_entry( $log_id ); $this->assertEquals( $action_id, $entry->get_action_id() ); $this->assertEquals( $message, $entry->get_message() ); @@ -39,41 +41,53 @@ public function test_add_log_datetime() { $this->assertEquals( $action_id, $entry->get_action_id() ); $this->assertEquals( $message, $entry->get_message() ); - $date = new ActionScheduler_DateTime( 'now', new DateTimeZone( 'UTC' ) ); - $log_id = $logger->log( $action_id, $message, $date ); - $entry = $logger->get_entry( $log_id ); + $date = new ActionScheduler_DateTime( 'now', new DateTimeZone( 'UTC' ) ); + $log_id = $logger->log( $action_id, $message, $date ); + $entry = $logger->get_entry( $log_id ); $this->assertEquals( $action_id, $entry->get_action_id() ); $this->assertEquals( $message, $entry->get_message() ); } public function test_erroneous_entry_id() { - $comment = wp_insert_comment(array( - 'comment_post_ID' => 1, - 'comment_author' => 'test', - 'comment_content' => 'this is not a log entry', - )); + $comment = wp_insert_comment( + array( + 'comment_post_ID' => 1, + 'comment_author' => 'test', + 'comment_content' => 'this is not a log entry', + ) + ); + $logger = ActionScheduler::logger(); - $entry = $logger->get_entry( $comment ); + $entry = $logger->get_entry( $comment ); + $this->assertEquals( '', $entry->get_action_id() ); $this->assertEquals( '', $entry->get_message() ); } public function test_storage_comments() { $action_id = as_schedule_single_action( time(), 'a hook' ); - $logger = ActionScheduler::logger(); - $logs = $logger->get_logs( $action_id ); - $expected = new ActionScheduler_LogEntry( $action_id, 'action created' ); - $this->assertTrue( in_array( $this->log_entry_to_array( $expected ) , $this->log_entry_to_array( $logs ) ) ); + $logger = ActionScheduler::logger(); + $logs = $logger->get_logs( $action_id ); + $expected = new ActionScheduler_LogEntry( $action_id, 'action created' ); + + // phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict + $this->assertTrue( in_array( $this->log_entry_to_array( $expected ), $this->log_entry_to_array( $logs ) ) ); } protected function log_entry_to_array( $logs ) { if ( $logs instanceof ActionScheduler_LogEntry ) { - return array( 'action_id' => $logs->get_action_id(), 'message' => $logs->get_message() ); + return array( + 'action_id' => $logs->get_action_id(), + 'message' => $logs->get_message(), + ); } - foreach ( $logs as $id => $log) { - $logs[ $id ] = array( 'action_id' => $log->get_action_id(), 'message' => $log->get_message() ); + foreach ( $logs as $id => $log ) { + $logs[ $id ] = array( + 'action_id' => $log->get_action_id(), + 'message' => $log->get_message(), + ); } return $logs; @@ -81,67 +95,78 @@ protected function log_entry_to_array( $logs ) { public function test_execution_comments() { $action_id = as_schedule_single_action( time(), ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ); - $logger = ActionScheduler::logger(); - $started = new ActionScheduler_LogEntry( $action_id, 'action started via Unit Tests' ); - $finished = new ActionScheduler_LogEntry( $action_id, 'action complete via Unit Tests' ); + $logger = ActionScheduler::logger(); + $started = new ActionScheduler_LogEntry( $action_id, 'action started via Unit Tests' ); + $finished = new ActionScheduler_LogEntry( $action_id, 'action complete via Unit Tests' ); $runner = ActionScheduler_Mocker::get_queue_runner(); $runner->run( 'Unit Tests' ); $logs = $logger->get_logs( $action_id ); + + // phpcs:disable WordPress.PHP.StrictInArray.MissingTrueStrict $this->assertTrue( in_array( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) ) ); $this->assertTrue( in_array( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) ) ); + // phpcs:enable } public function test_failed_execution_comments() { - $hook = md5(rand()); - add_action( $hook, array( $this, '_a_hook_callback_that_throws_an_exception' ) ); + $hook = md5( wp_rand() ); + add_action( $hook, array( $this, 'a_hook_callback_that_throws_an_exception' ) ); + $action_id = as_schedule_single_action( time(), $hook ); - $logger = ActionScheduler::logger(); - $started = new ActionScheduler_LogEntry( $action_id, 'action started via Unit Tests' ); - $finished = new ActionScheduler_LogEntry( $action_id, 'action complete via Unit Tests' ); - $failed = new ActionScheduler_LogEntry( $action_id, 'action failed via Unit Tests: Execution failed' ); + $logger = ActionScheduler::logger(); + $started = new ActionScheduler_LogEntry( $action_id, 'action started via Unit Tests' ); + $finished = new ActionScheduler_LogEntry( $action_id, 'action complete via Unit Tests' ); + $failed = new ActionScheduler_LogEntry( $action_id, 'action failed via Unit Tests: Execution failed' ); $runner = ActionScheduler_Mocker::get_queue_runner(); $runner->run( 'Unit Tests' ); $logs = $logger->get_logs( $action_id ); + + // phpcs:disable WordPress.PHP.StrictInArray.MissingTrueStrict $this->assertTrue( in_array( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) ) ); $this->assertFalse( in_array( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) ) ); $this->assertTrue( in_array( $this->log_entry_to_array( $failed ), $this->log_entry_to_array( $logs ) ) ); + // phpcs:enable } public function test_failed_schedule_next_instance_comments() { - $action_id = rand(); + $action_id = wp_rand(); $logger = ActionScheduler::logger(); $log_entry = new ActionScheduler_LogEntry( $action_id, 'There was a failure scheduling the next instance of this action: Execution failed' ); try { - $this->_a_hook_callback_that_throws_an_exception(); + $this->a_hook_callback_that_throws_an_exception(); } catch ( Exception $e ) { do_action( 'action_scheduler_failed_to_schedule_next_instance', $action_id, $e, new ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ) ); } $logs = $logger->get_logs( $action_id ); + + // phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict $this->assertTrue( in_array( $this->log_entry_to_array( $log_entry ), $this->log_entry_to_array( $logs ) ) ); } public function test_fatal_error_comments() { - $hook = md5(rand()); + $hook = md5( wp_rand() ); $action_id = as_schedule_single_action( time(), $hook ); - $logger = ActionScheduler::logger(); - do_action( 'action_scheduler_unexpected_shutdown', $action_id, array( - 'type' => E_ERROR, + $logger = ActionScheduler::logger(); + $args = array( + 'type' => E_ERROR, 'message' => 'Test error', - 'file' => __FILE__, - 'line' => __LINE__, - )); + 'file' => __FILE__, + 'line' => __LINE__, + ); - $logs = $logger->get_logs( $action_id ); - $found_log = FALSE; + do_action( 'action_scheduler_unexpected_shutdown', $action_id, $args ); + + $logs = $logger->get_logs( $action_id ); + $found_log = false; foreach ( $logs as $l ) { if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) { - $found_log = TRUE; + $found_log = true; } } $this->assertTrue( $found_log, 'Unexpected shutdown log not found' ); @@ -150,14 +175,17 @@ public function test_fatal_error_comments() { public function test_canceled_action_comments() { $action_id = as_schedule_single_action( time(), 'a hook' ); as_unschedule_action( 'a hook' ); - $logger = ActionScheduler::logger(); - $logs = $logger->get_logs( $action_id ); + + $logger = ActionScheduler::logger(); + $logs = $logger->get_logs( $action_id ); $expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' ); + + // phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict $this->assertTrue( in_array( $this->log_entry_to_array( $expected ), $this->log_entry_to_array( $logs ) ) ); } - public function _a_hook_callback_that_throws_an_exception() { - throw new RuntimeException('Execution failed'); + public function a_hook_callback_that_throws_an_exception() { + throw new RuntimeException( 'Execution failed' ); } public function test_filtering_of_get_comments() { @@ -166,43 +194,44 @@ public function test_filtering_of_get_comments() { return; } - $post_id = $this->factory->post->create_object(array( - 'post_title' => __FUNCTION__, - )); - $comment_id = $this->factory->comment->create_object(array( - 'comment_post_ID' => $post_id, - 'comment_author' => __CLASS__, - 'comment_content' => __FUNCTION__, - )); + $post_id = $this->factory->post->create_object( array( 'post_title' => __FUNCTION__ ) ); + $comment_id = $this->factory->comment->create_object( + array( + 'comment_post_ID' => $post_id, + 'comment_author' => __CLASS__, + 'comment_content' => __FUNCTION__, + ) + ); - // Verify that we're getting the expected comment before we add logging comments + // Verify that we're getting the expected comment before we add logging comments. $comments = get_comments(); $this->assertCount( 1, $comments ); $this->assertEquals( $comment_id, $comments[0]->comment_ID ); - $action_id = as_schedule_single_action( time(), 'a hook' ); - $logger = ActionScheduler::logger(); - $message = 'Logging that something happened'; - $log_id = $logger->log( $action_id, $message ); - + $logger = ActionScheduler::logger(); + $message = 'Logging that something happened'; + $log_id = $logger->log( $action_id, $message ); - // Verify that logging comments are excluded from general comment queries + // Verify that logging comments are excluded from general comment queries. $comments = get_comments(); $this->assertCount( 1, $comments ); $this->assertEquals( $comment_id, $comments[0]->comment_ID ); - // Verify that logging comments are returned when asking for them specifically - $comments = get_comments(array( - 'type' => ActionScheduler_wpCommentLogger::TYPE, - )); - // Expecting two: one when the action is created, another when we added our custom log + // Verify that logging comments are returned when asking for them specifically. + $comments = get_comments( + array( + 'type' => ActionScheduler_wpCommentLogger::TYPE, + ) + ); + + // Expecting two: one when the action is created, another when we added our custom log. $this->assertCount( 2, $comments ); - $this->assertContains( $log_id, wp_list_pluck($comments, 'comment_ID')); + $this->assertContains( $log_id, wp_list_pluck( $comments, 'comment_ID' ) ); } private function using_comment_logger() { - if ( null === $this->use_comment_logger ) { + if ( is_null( $this->use_comment_logger ) ) { $this->use_comment_logger = ! ActionScheduler_DataController::dependencies_met(); }