diff --git a/tests/phpunit/migration/Runner_Test.php b/tests/phpunit/migration/Runner_Test.php index 714c4431..1067ef8e 100644 --- a/tests/phpunit/migration/Runner_Test.php +++ b/tests/phpunit/migration/Runner_Test.php @@ -14,7 +14,7 @@ class Runner_Test extends ActionScheduler_UnitTestCase { public function setUp() { parent::setUp(); if ( ! taxonomy_exists( PostStore::GROUP_TAXONOMY ) ) { - // register the post type and taxonomy necessary for the store to work + // register the post type and taxonomy necessary for the store to work. $store = new PostStore(); $store->init(); } @@ -34,57 +34,60 @@ public function test_migrate_batches() { $runner = new Runner( $config ); - $due = []; - $future = []; - $complete = []; + $due = array(); + $future = array(); + $complete = array(); for ( $i = 0; $i < 5; $i ++ ) { $time = as_get_datetime_object( $i + 1 . ' minutes' ); $schedule = new ActionScheduler_SimpleSchedule( $time ); - $action = new ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, [], $schedule ); + $action = new ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule ); $future[] = $source_store->save_action( $action ); $time = as_get_datetime_object( $i + 1 . ' minutes ago' ); $schedule = new ActionScheduler_SimpleSchedule( $time ); - $action = new ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, [], $schedule ); + $action = new ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule ); $due[] = $source_store->save_action( $action ); $time = as_get_datetime_object( $i + 1 . ' minutes ago' ); $schedule = new ActionScheduler_SimpleSchedule( $time ); - $action = new ActionScheduler_FinishedAction( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, [], $schedule ); + $action = new ActionScheduler_FinishedAction( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule ); $complete[] = $source_store->save_action( $action ); } - $created = $source_store->query_actions( [ 'per_page' => 0 ] ); + $created = $source_store->query_actions( array( 'per_page' => 0 ) ); $this->assertCount( 15, $created ); $runner->run( 10 ); - // due actions should migrate in the first batch - $migrated = $destination_store->query_actions( [ 'per_page' => 0, 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ] ); + $args = array( + 'per_page' => 0, + 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, + ); + + // due actions should migrate in the first batch. + $migrated = $destination_store->query_actions( $args ); $this->assertCount( 5, $migrated ); - $remaining = $source_store->query_actions( [ 'per_page' => 0, 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ] ); + $remaining = $source_store->query_actions( $args ); $this->assertCount( 10, $remaining ); - $runner->run( 10 ); - // pending actions should migrate in the second batch - $migrated = $destination_store->query_actions( [ 'per_page' => 0, 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ] ); + // pending actions should migrate in the second batch. + $migrated = $destination_store->query_actions( $args ); $this->assertCount( 10, $migrated ); - $remaining = $source_store->query_actions( [ 'per_page' => 0, 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ] ); + $remaining = $source_store->query_actions( $args ); $this->assertCount( 5, $remaining ); - $runner->run( 10 ); - // completed actions should migrate in the third batch - $migrated = $destination_store->query_actions( [ 'per_page' => 0, 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ] ); + // completed actions should migrate in the third batch. + $migrated = $destination_store->query_actions( $args ); $this->assertCount( 15, $migrated ); - $remaining = $source_store->query_actions( [ 'per_page' => 0, 'hook' => ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ] ); + $remaining = $source_store->query_actions( $args ); $this->assertCount( 0, $remaining ); }