insideout10 / wordlift-plugin

src/libraries/action-scheduler/tests/phpunit/jobstore/ActionScheduler_HybridStore_Test.php

Indentation +261 added lines, -261 removed lines

@@ -12,265 +12,265 @@
  */
 
 class ActionScheduler_HybridStore_Test extends ActionScheduler_UnitTestCase {
-	private $demarkation_id = 1000;
-
-	public function setUp() {
-		parent::setUp();
-		if ( ! taxonomy_exists( PostStore::GROUP_TAXONOMY ) ) {
-			// register the post type and taxonomy necessary for the store to work
-			$store = new PostStore();
-			$store->init();
-		}
-		update_option( ActionScheduler_HybridStore::DEMARKATION_OPTION, $this->demarkation_id );
-		$hybrid = new ActionScheduler_HybridStore();
-		$hybrid->set_autoincrement( '', ActionScheduler_StoreSchema::ACTIONS_TABLE );
-	}
-
-	public function tearDown() {
-		parent::tearDown();
-
-		// reset the autoincrement index
-		/** @var \wpdb $wpdb */
-		global $wpdb;
-		$wpdb->query( "TRUNCATE TABLE {$wpdb->actionscheduler_actions}" );
-		$wpdb->query( "TRUNCATE TABLE {$wpdb->actionscheduler_logs}" );
-		delete_option( ActionScheduler_HybridStore::DEMARKATION_OPTION );
-	}
-
-	public function test_actions_are_migrated_on_find() {
-		$source_store       = new PostStore();
-		$destination_store  = new ActionScheduler_DBStore();
-		$source_logger      = new CommentLogger();
-		$destination_logger = new ActionScheduler_DBLogger();
-
-		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
-
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
-
-		$time      = as_get_datetime_object( '10 minutes ago' );
-		$schedule  = new ActionScheduler_SimpleSchedule( $time );
-		$action    = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-		$source_id = $source_store->save_action( $action );
-
-		$found = $hybrid_store->find_action( __FUNCTION__, array() );
-
-		$this->assertNotEquals( $source_id, $found );
-		$this->assertGreaterThanOrEqual( $this->demarkation_id, $found );
-
-		$found_in_source = $source_store->fetch_action( $source_id );
-		$this->assertInstanceOf( NullAction::class, $found_in_source );
-	}
-
-	public function test_actions_are_migrated_on_query() {
-		$source_store       = new PostStore();
-		$destination_store  = new ActionScheduler_DBStore();
-		$source_logger      = new CommentLogger();
-		$destination_logger = new ActionScheduler_DBLogger();
-
-		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
-
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
-
-		$source_actions      = array();
-		$destination_actions = array();
-
-		for ( $i = 0; $i < 10; $i++ ) {
-			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$source_actions[] = $source_store->save_action( $action );
-
-			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$destination_actions[] = $destination_store->save_action( $action );
-		}
-
-		$found = $hybrid_store->query_actions(
-			array(
-				'hook'     => __FUNCTION__,
-				'per_page' => 6,
-			)
-		);
-
-		$this->assertCount( 6, $found );
-		foreach ( $found as $key => $action_id ) {
-			$this->assertNotContains( $action_id, $source_actions );
-			$this->assertGreaterThanOrEqual( $this->demarkation_id, $action_id );
-			if ( $key % 2 == 0 ) { // it should have been in the source store
-				$this->assertNotContains( $action_id, $destination_actions );
-			} else { // it should have already been in the destination store
-				$this->assertContains( $action_id, $destination_actions );
-			}
-		}
-
-		// six of the original 10 should have migrated to the new store
-		// even though only three were retrieve in the final query
-		$found_in_source = $source_store->query_actions(
-			array(
-				'hook'     => __FUNCTION__,
-				'per_page' => 10,
-			)
-		);
-		$this->assertCount( 4, $found_in_source );
-	}
-
-	public function test_actions_are_migrated_on_claim() {
-		$source_store       = new PostStore();
-		$destination_store  = new ActionScheduler_DBStore();
-		$source_logger      = new CommentLogger();
-		$destination_logger = new ActionScheduler_DBLogger();
-
-		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
-
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
-
-		$source_actions      = array();
-		$destination_actions = array();
-
-		for ( $i = 0; $i < 10; $i++ ) {
-			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$source_actions[] = $source_store->save_action( $action );
-
-			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$destination_actions[] = $destination_store->save_action( $action );
-		}
-
-		$claim = $hybrid_store->stake_claim( 6 );
-
-		$claimed_actions = $claim->get_actions();
-		$this->assertCount( 6, $claimed_actions );
-		$this->assertCount( 3, array_intersect( $destination_actions, $claimed_actions ) );
-
-		// six of the original 10 should have migrated to the new store
-		// even though only three were retrieve in the final claim
-		$found_in_source = $source_store->query_actions(
-			array(
-				'hook'     => __FUNCTION__,
-				'per_page' => 10,
-			)
-		);
-		$this->assertCount( 4, $found_in_source );
-
-		$this->assertSame( 0, $source_store->get_claim_count() );
-		$this->assertSame( 1, $destination_store->get_claim_count() );
-		$this->assertSame( 1, $hybrid_store->get_claim_count() );
-
-	}
-
-	public function test_fetch_respects_demarkation() {
-		$source_store       = new PostStore();
-		$destination_store  = new ActionScheduler_DBStore();
-		$source_logger      = new CommentLogger();
-		$destination_logger = new ActionScheduler_DBLogger();
-
-		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
-
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
-
-		$source_actions      = array();
-		$destination_actions = array();
-
-		for ( $i = 0; $i < 2; $i++ ) {
-			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$source_actions[] = $source_store->save_action( $action );
-
-			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$destination_actions[] = $destination_store->save_action( $action );
-		}
-
-		foreach ( $source_actions as $action_id ) {
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_Action::class, $action );
-			$this->assertNotInstanceOf( NullAction::class, $action );
-		}
-
-		foreach ( $destination_actions as $action_id ) {
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_Action::class, $action );
-			$this->assertNotInstanceOf( NullAction::class, $action );
-		}
-	}
-
-	public function test_mark_complete_respects_demarkation() {
-		$source_store       = new PostStore();
-		$destination_store  = new ActionScheduler_DBStore();
-		$source_logger      = new CommentLogger();
-		$destination_logger = new ActionScheduler_DBLogger();
-
-		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
-
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
-
-		$source_actions      = array();
-		$destination_actions = array();
-
-		for ( $i = 0; $i < 2; $i++ ) {
-			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$source_actions[] = $source_store->save_action( $action );
-
-			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-
-			$destination_actions[] = $destination_store->save_action( $action );
-		}
-
-		foreach ( $source_actions as $action_id ) {
-			$hybrid_store->mark_complete( $action_id );
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_FinishedAction::class, $action );
-		}
-
-		foreach ( $destination_actions as $action_id ) {
-			$hybrid_store->mark_complete( $action_id );
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_FinishedAction::class, $action );
-		}
-	}
+    private $demarkation_id = 1000;
+
+    public function setUp() {
+        parent::setUp();
+        if ( ! taxonomy_exists( PostStore::GROUP_TAXONOMY ) ) {
+            // register the post type and taxonomy necessary for the store to work
+            $store = new PostStore();
+            $store->init();
+        }
+        update_option( ActionScheduler_HybridStore::DEMARKATION_OPTION, $this->demarkation_id );
+        $hybrid = new ActionScheduler_HybridStore();
+        $hybrid->set_autoincrement( '', ActionScheduler_StoreSchema::ACTIONS_TABLE );
+    }
+
+    public function tearDown() {
+        parent::tearDown();
+
+        // reset the autoincrement index
+        /** @var \wpdb $wpdb */
+        global $wpdb;
+        $wpdb->query( "TRUNCATE TABLE {$wpdb->actionscheduler_actions}" );
+        $wpdb->query( "TRUNCATE TABLE {$wpdb->actionscheduler_logs}" );
+        delete_option( ActionScheduler_HybridStore::DEMARKATION_OPTION );
+    }
+
+    public function test_actions_are_migrated_on_find() {
+        $source_store       = new PostStore();
+        $destination_store  = new ActionScheduler_DBStore();
+        $source_logger      = new CommentLogger();
+        $destination_logger = new ActionScheduler_DBLogger();
+
+        $config = new Config();
+        $config->set_source_store( $source_store );
+        $config->set_source_logger( $source_logger );
+        $config->set_destination_store( $destination_store );
+        $config->set_destination_logger( $destination_logger );
+
+        $hybrid_store = new ActionScheduler_HybridStore( $config );
+
+        $time      = as_get_datetime_object( '10 minutes ago' );
+        $schedule  = new ActionScheduler_SimpleSchedule( $time );
+        $action    = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+        $source_id = $source_store->save_action( $action );
+
+        $found = $hybrid_store->find_action( __FUNCTION__, array() );
+
+        $this->assertNotEquals( $source_id, $found );
+        $this->assertGreaterThanOrEqual( $this->demarkation_id, $found );
+
+        $found_in_source = $source_store->fetch_action( $source_id );
+        $this->assertInstanceOf( NullAction::class, $found_in_source );
+    }
+
+    public function test_actions_are_migrated_on_query() {
+        $source_store       = new PostStore();
+        $destination_store  = new ActionScheduler_DBStore();
+        $source_logger      = new CommentLogger();
+        $destination_logger = new ActionScheduler_DBLogger();
+
+        $config = new Config();
+        $config->set_source_store( $source_store );
+        $config->set_source_logger( $source_logger );
+        $config->set_destination_store( $destination_store );
+        $config->set_destination_logger( $destination_logger );
+
+        $hybrid_store = new ActionScheduler_HybridStore( $config );
+
+        $source_actions      = array();
+        $destination_actions = array();
+
+        for ( $i = 0; $i < 10; $i++ ) {
+            // create in instance in the source store
+            $time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $source_actions[] = $source_store->save_action( $action );
+
+            // create an instance in the destination store
+            $time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $destination_actions[] = $destination_store->save_action( $action );
+        }
+
+        $found = $hybrid_store->query_actions(
+            array(
+                'hook'     => __FUNCTION__,
+                'per_page' => 6,
+            )
+        );
+
+        $this->assertCount( 6, $found );
+        foreach ( $found as $key => $action_id ) {
+            $this->assertNotContains( $action_id, $source_actions );
+            $this->assertGreaterThanOrEqual( $this->demarkation_id, $action_id );
+            if ( $key % 2 == 0 ) { // it should have been in the source store
+                $this->assertNotContains( $action_id, $destination_actions );
+            } else { // it should have already been in the destination store
+                $this->assertContains( $action_id, $destination_actions );
+            }
+        }
+
+        // six of the original 10 should have migrated to the new store
+        // even though only three were retrieve in the final query
+        $found_in_source = $source_store->query_actions(
+            array(
+                'hook'     => __FUNCTION__,
+                'per_page' => 10,
+            )
+        );
+        $this->assertCount( 4, $found_in_source );
+    }
+
+    public function test_actions_are_migrated_on_claim() {
+        $source_store       = new PostStore();
+        $destination_store  = new ActionScheduler_DBStore();
+        $source_logger      = new CommentLogger();
+        $destination_logger = new ActionScheduler_DBLogger();
+
+        $config = new Config();
+        $config->set_source_store( $source_store );
+        $config->set_source_logger( $source_logger );
+        $config->set_destination_store( $destination_store );
+        $config->set_destination_logger( $destination_logger );
+
+        $hybrid_store = new ActionScheduler_HybridStore( $config );
+
+        $source_actions      = array();
+        $destination_actions = array();
+
+        for ( $i = 0; $i < 10; $i++ ) {
+            // create in instance in the source store
+            $time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $source_actions[] = $source_store->save_action( $action );
+
+            // create an instance in the destination store
+            $time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $destination_actions[] = $destination_store->save_action( $action );
+        }
+
+        $claim = $hybrid_store->stake_claim( 6 );
+
+        $claimed_actions = $claim->get_actions();
+        $this->assertCount( 6, $claimed_actions );
+        $this->assertCount( 3, array_intersect( $destination_actions, $claimed_actions ) );
+
+        // six of the original 10 should have migrated to the new store
+        // even though only three were retrieve in the final claim
+        $found_in_source = $source_store->query_actions(
+            array(
+                'hook'     => __FUNCTION__,
+                'per_page' => 10,
+            )
+        );
+        $this->assertCount( 4, $found_in_source );
+
+        $this->assertSame( 0, $source_store->get_claim_count() );
+        $this->assertSame( 1, $destination_store->get_claim_count() );
+        $this->assertSame( 1, $hybrid_store->get_claim_count() );
+
+    }
+
+    public function test_fetch_respects_demarkation() {
+        $source_store       = new PostStore();
+        $destination_store  = new ActionScheduler_DBStore();
+        $source_logger      = new CommentLogger();
+        $destination_logger = new ActionScheduler_DBLogger();
+
+        $config = new Config();
+        $config->set_source_store( $source_store );
+        $config->set_source_logger( $source_logger );
+        $config->set_destination_store( $destination_store );
+        $config->set_destination_logger( $destination_logger );
+
+        $hybrid_store = new ActionScheduler_HybridStore( $config );
+
+        $source_actions      = array();
+        $destination_actions = array();
+
+        for ( $i = 0; $i < 2; $i++ ) {
+            // create in instance in the source store
+            $time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $source_actions[] = $source_store->save_action( $action );
+
+            // create an instance in the destination store
+            $time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $destination_actions[] = $destination_store->save_action( $action );
+        }
+
+        foreach ( $source_actions as $action_id ) {
+            $action = $hybrid_store->fetch_action( $action_id );
+            $this->assertInstanceOf( ActionScheduler_Action::class, $action );
+            $this->assertNotInstanceOf( NullAction::class, $action );
+        }
+
+        foreach ( $destination_actions as $action_id ) {
+            $action = $hybrid_store->fetch_action( $action_id );
+            $this->assertInstanceOf( ActionScheduler_Action::class, $action );
+            $this->assertNotInstanceOf( NullAction::class, $action );
+        }
+    }
+
+    public function test_mark_complete_respects_demarkation() {
+        $source_store       = new PostStore();
+        $destination_store  = new ActionScheduler_DBStore();
+        $source_logger      = new CommentLogger();
+        $destination_logger = new ActionScheduler_DBLogger();
+
+        $config = new Config();
+        $config->set_source_store( $source_store );
+        $config->set_source_logger( $source_logger );
+        $config->set_destination_store( $destination_store );
+        $config->set_destination_logger( $destination_logger );
+
+        $hybrid_store = new ActionScheduler_HybridStore( $config );
+
+        $source_actions      = array();
+        $destination_actions = array();
+
+        for ( $i = 0; $i < 2; $i++ ) {
+            // create in instance in the source store
+            $time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $source_actions[] = $source_store->save_action( $action );
+
+            // create an instance in the destination store
+            $time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
+            $schedule = new ActionScheduler_SimpleSchedule( $time );
+            $action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+
+            $destination_actions[] = $destination_store->save_action( $action );
+        }
+
+        foreach ( $source_actions as $action_id ) {
+            $hybrid_store->mark_complete( $action_id );
+            $action = $hybrid_store->fetch_action( $action_id );
+            $this->assertInstanceOf( ActionScheduler_FinishedAction::class, $action );
+        }
+
+        foreach ( $destination_actions as $action_id ) {
+            $hybrid_store->mark_complete( $action_id );
+            $action = $hybrid_store->fetch_action( $action_id );
+            $this->assertInstanceOf( ActionScheduler_FinishedAction::class, $action );
+        }
+    }
 }

Spacing +107 added lines, -107 removed lines

@@ -16,14 +16,14 @@ 
 
 	public function setUp() {
 		parent::setUp();
-		if ( ! taxonomy_exists( PostStore::GROUP_TAXONOMY ) ) {
+		if ( ! taxonomy_exists(PostStore::GROUP_TAXONOMY)) {
 			// register the post type and taxonomy necessary for the store to work
 			$store = new PostStore();
 			$store->init();
 		}
-		update_option( ActionScheduler_HybridStore::DEMARKATION_OPTION, $this->demarkation_id );
+		update_option(ActionScheduler_HybridStore::DEMARKATION_OPTION, $this->demarkation_id);
 		$hybrid = new ActionScheduler_HybridStore();
-		$hybrid->set_autoincrement( '', ActionScheduler_StoreSchema::ACTIONS_TABLE );
+		$hybrid->set_autoincrement('', ActionScheduler_StoreSchema::ACTIONS_TABLE);
 	}
 
 	public function tearDown() {
@@ -32,9 +32,9 @@ 
 		// reset the autoincrement index
 		/** @var \wpdb $wpdb */
 		global $wpdb;
-		$wpdb->query( "TRUNCATE TABLE {$wpdb->actionscheduler_actions}" );
-		$wpdb->query( "TRUNCATE TABLE {$wpdb->actionscheduler_logs}" );
-		delete_option( ActionScheduler_HybridStore::DEMARKATION_OPTION );
+		$wpdb->query("TRUNCATE TABLE {$wpdb->actionscheduler_actions}");
+		$wpdb->query("TRUNCATE TABLE {$wpdb->actionscheduler_logs}");
+		delete_option(ActionScheduler_HybridStore::DEMARKATION_OPTION);
 	}
 
 	public function test_actions_are_migrated_on_find() {
@@ -44,25 +44,25 @@ 
 		$destination_logger = new ActionScheduler_DBLogger();
 
 		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
+		$config->set_source_store($source_store);
+		$config->set_source_logger($source_logger);
+		$config->set_destination_store($destination_store);
+		$config->set_destination_logger($destination_logger);
 
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
+		$hybrid_store = new ActionScheduler_HybridStore($config);
 
-		$time      = as_get_datetime_object( '10 minutes ago' );
-		$schedule  = new ActionScheduler_SimpleSchedule( $time );
-		$action    = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
-		$source_id = $source_store->save_action( $action );
+		$time      = as_get_datetime_object('10 minutes ago');
+		$schedule  = new ActionScheduler_SimpleSchedule($time);
+		$action    = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
+		$source_id = $source_store->save_action($action);
 
-		$found = $hybrid_store->find_action( __FUNCTION__, array() );
+		$found = $hybrid_store->find_action(__FUNCTION__, array());
 
-		$this->assertNotEquals( $source_id, $found );
-		$this->assertGreaterThanOrEqual( $this->demarkation_id, $found );
+		$this->assertNotEquals($source_id, $found);
+		$this->assertGreaterThanOrEqual($this->demarkation_id, $found);
 
-		$found_in_source = $source_store->fetch_action( $source_id );
-		$this->assertInstanceOf( NullAction::class, $found_in_source );
+		$found_in_source = $source_store->fetch_action($source_id);
+		$this->assertInstanceOf(NullAction::class, $found_in_source);
 	}
 
 	public function test_actions_are_migrated_on_query() {
@@ -72,30 +72,30 @@ 
 		$destination_logger = new ActionScheduler_DBLogger();
 
 		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
+		$config->set_source_store($source_store);
+		$config->set_source_logger($source_logger);
+		$config->set_destination_store($destination_store);
+		$config->set_destination_logger($destination_logger);
 
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
+		$hybrid_store = new ActionScheduler_HybridStore($config);
 
 		$source_actions      = array();
 		$destination_actions = array();
 
-		for ( $i = 0; $i < 10; $i++ ) {
+		for ($i = 0; $i < 10; $i++) {
 			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 1).' minutes');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$source_actions[] = $source_store->save_action( $action );
+			$source_actions[] = $source_store->save_action($action);
 
 			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 5).' minutes');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$destination_actions[] = $destination_store->save_action( $action );
+			$destination_actions[] = $destination_store->save_action($action);
 		}
 
 		$found = $hybrid_store->query_actions(
@@ -105,14 +105,14 @@ 
 			)
 		);
 
-		$this->assertCount( 6, $found );
-		foreach ( $found as $key => $action_id ) {
-			$this->assertNotContains( $action_id, $source_actions );
-			$this->assertGreaterThanOrEqual( $this->demarkation_id, $action_id );
-			if ( $key % 2 == 0 ) { // it should have been in the source store
-				$this->assertNotContains( $action_id, $destination_actions );
+		$this->assertCount(6, $found);
+		foreach ($found as $key => $action_id) {
+			$this->assertNotContains($action_id, $source_actions);
+			$this->assertGreaterThanOrEqual($this->demarkation_id, $action_id);
+			if ($key % 2 == 0) { // it should have been in the source store
+				$this->assertNotContains($action_id, $destination_actions);
 			} else { // it should have already been in the destination store
-				$this->assertContains( $action_id, $destination_actions );
+				$this->assertContains($action_id, $destination_actions);
 			}
 		}
 
@@ -124,7 +124,7 @@ 
 				'per_page' => 10,
 			)
 		);
-		$this->assertCount( 4, $found_in_source );
+		$this->assertCount(4, $found_in_source);
 	}
 
 	public function test_actions_are_migrated_on_claim() {
@@ -134,37 +134,37 @@ 
 		$destination_logger = new ActionScheduler_DBLogger();
 
 		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
+		$config->set_source_store($source_store);
+		$config->set_source_logger($source_logger);
+		$config->set_destination_store($destination_store);
+		$config->set_destination_logger($destination_logger);
 
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
+		$hybrid_store = new ActionScheduler_HybridStore($config);
 
 		$source_actions      = array();
 		$destination_actions = array();
 
-		for ( $i = 0; $i < 10; $i++ ) {
+		for ($i = 0; $i < 10; $i++) {
 			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 1).' minutes ago');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$source_actions[] = $source_store->save_action( $action );
+			$source_actions[] = $source_store->save_action($action);
 
 			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 5).' minutes ago');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$destination_actions[] = $destination_store->save_action( $action );
+			$destination_actions[] = $destination_store->save_action($action);
 		}
 
-		$claim = $hybrid_store->stake_claim( 6 );
+		$claim = $hybrid_store->stake_claim(6);
 
 		$claimed_actions = $claim->get_actions();
-		$this->assertCount( 6, $claimed_actions );
-		$this->assertCount( 3, array_intersect( $destination_actions, $claimed_actions ) );
+		$this->assertCount(6, $claimed_actions);
+		$this->assertCount(3, array_intersect($destination_actions, $claimed_actions));
 
 		// six of the original 10 should have migrated to the new store
 		// even though only three were retrieve in the final claim
@@ -174,11 +174,11 @@ 
 				'per_page' => 10,
 			)
 		);
-		$this->assertCount( 4, $found_in_source );
+		$this->assertCount(4, $found_in_source);
 
-		$this->assertSame( 0, $source_store->get_claim_count() );
-		$this->assertSame( 1, $destination_store->get_claim_count() );
-		$this->assertSame( 1, $hybrid_store->get_claim_count() );
+		$this->assertSame(0, $source_store->get_claim_count());
+		$this->assertSame(1, $destination_store->get_claim_count());
+		$this->assertSame(1, $hybrid_store->get_claim_count());
 
 	}
 
@@ -189,42 +189,42 @@ 
 		$destination_logger = new ActionScheduler_DBLogger();
 
 		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
+		$config->set_source_store($source_store);
+		$config->set_source_logger($source_logger);
+		$config->set_destination_store($destination_store);
+		$config->set_destination_logger($destination_logger);
 
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
+		$hybrid_store = new ActionScheduler_HybridStore($config);
 
 		$source_actions      = array();
 		$destination_actions = array();
 
-		for ( $i = 0; $i < 2; $i++ ) {
+		for ($i = 0; $i < 2; $i++) {
 			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 1).' minutes ago');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$source_actions[] = $source_store->save_action( $action );
+			$source_actions[] = $source_store->save_action($action);
 
 			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 5).' minutes ago');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$destination_actions[] = $destination_store->save_action( $action );
+			$destination_actions[] = $destination_store->save_action($action);
 		}
 
-		foreach ( $source_actions as $action_id ) {
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_Action::class, $action );
-			$this->assertNotInstanceOf( NullAction::class, $action );
+		foreach ($source_actions as $action_id) {
+			$action = $hybrid_store->fetch_action($action_id);
+			$this->assertInstanceOf(ActionScheduler_Action::class, $action);
+			$this->assertNotInstanceOf(NullAction::class, $action);
 		}
 
-		foreach ( $destination_actions as $action_id ) {
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_Action::class, $action );
-			$this->assertNotInstanceOf( NullAction::class, $action );
+		foreach ($destination_actions as $action_id) {
+			$action = $hybrid_store->fetch_action($action_id);
+			$this->assertInstanceOf(ActionScheduler_Action::class, $action);
+			$this->assertNotInstanceOf(NullAction::class, $action);
 		}
 	}
 
@@ -235,42 +235,42 @@ 
 		$destination_logger = new ActionScheduler_DBLogger();
 
 		$config = new Config();
-		$config->set_source_store( $source_store );
-		$config->set_source_logger( $source_logger );
-		$config->set_destination_store( $destination_store );
-		$config->set_destination_logger( $destination_logger );
+		$config->set_source_store($source_store);
+		$config->set_source_logger($source_logger);
+		$config->set_destination_store($destination_store);
+		$config->set_destination_logger($destination_logger);
 
-		$hybrid_store = new ActionScheduler_HybridStore( $config );
+		$hybrid_store = new ActionScheduler_HybridStore($config);
 
 		$source_actions      = array();
 		$destination_actions = array();
 
-		for ( $i = 0; $i < 2; $i++ ) {
+		for ($i = 0; $i < 2; $i++) {
 			// create in instance in the source store
-			$time     = as_get_datetime_object( ( $i * 10 + 1 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 1).' minutes ago');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$source_actions[] = $source_store->save_action( $action );
+			$source_actions[] = $source_store->save_action($action);
 
 			// create an instance in the destination store
-			$time     = as_get_datetime_object( ( $i * 10 + 5 ) . ' minutes ago' );
-			$schedule = new ActionScheduler_SimpleSchedule( $time );
-			$action   = new ActionScheduler_Action( __FUNCTION__, array(), $schedule );
+			$time     = as_get_datetime_object(($i * 10 + 5).' minutes ago');
+			$schedule = new ActionScheduler_SimpleSchedule($time);
+			$action   = new ActionScheduler_Action(__FUNCTION__, array(), $schedule);
 
-			$destination_actions[] = $destination_store->save_action( $action );
+			$destination_actions[] = $destination_store->save_action($action);
 		}
 
-		foreach ( $source_actions as $action_id ) {
-			$hybrid_store->mark_complete( $action_id );
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_FinishedAction::class, $action );
+		foreach ($source_actions as $action_id) {
+			$hybrid_store->mark_complete($action_id);
+			$action = $hybrid_store->fetch_action($action_id);
+			$this->assertInstanceOf(ActionScheduler_FinishedAction::class, $action);
 		}
 
-		foreach ( $destination_actions as $action_id ) {
-			$hybrid_store->mark_complete( $action_id );
-			$action = $hybrid_store->fetch_action( $action_id );
-			$this->assertInstanceOf( ActionScheduler_FinishedAction::class, $action );
+		foreach ($destination_actions as $action_id) {
+			$hybrid_store->mark_complete($action_id);
+			$action = $hybrid_store->fetch_action($action_id);
+			$this->assertInstanceOf(ActionScheduler_FinishedAction::class, $action);
 		}
 	}
 }

src/libraries/action-scheduler/tests/phpunit/procedural_api/wc_get_scheduled_actions_Test.php

Indentation +106 added lines, -106 removed lines

@@ -4,124 +4,124 @@
  * Class as_get_scheduled_actions_Test
  */
 class as_get_scheduled_actions_Test extends ActionScheduler_UnitTestCase {
-	private $hooks  = array();
-	private $args   = array();
-	private $groups = array();
+    private $hooks  = array();
+    private $args   = array();
+    private $groups = array();
 
-	public function setUp() {
-		parent::setUp();
+    public function setUp() {
+        parent::setUp();
 
-		$store = ActionScheduler::store();
+        $store = ActionScheduler::store();
 
-		for ( $i = 0; $i < 10; $i++ ) {
-			$this->hooks[ $i ]  = md5( rand() );
-			$this->args[ $i ]   = md5( rand() );
-			$this->groups[ $i ] = md5( rand() );
-		}
+        for ( $i = 0; $i < 10; $i++ ) {
+            $this->hooks[ $i ]  = md5( rand() );
+            $this->args[ $i ]   = md5( rand() );
+            $this->groups[ $i ] = md5( rand() );
+        }
 
-		for ( $i = 0; $i < 10; $i++ ) {
-			for ( $j = 0; $j < 10; $j++ ) {
-				$schedule = new ActionScheduler_SimpleSchedule( as_get_datetime_object( $j - 3 . 'days' ) );
-				$group    = $this->groups[ ( $i + $j ) % 10 ];
-				$action   = new ActionScheduler_Action( $this->hooks[ $i ], array( $this->args[ $j ] ), $schedule, $group );
-				$store->save_action( $action );
-			}
-		}
-	}
+        for ( $i = 0; $i < 10; $i++ ) {
+            for ( $j = 0; $j < 10; $j++ ) {
+                $schedule = new ActionScheduler_SimpleSchedule( as_get_datetime_object( $j - 3 . 'days' ) );
+                $group    = $this->groups[ ( $i + $j ) % 10 ];
+                $action   = new ActionScheduler_Action( $this->hooks[ $i ], array( $this->args[ $j ] ), $schedule, $group );
+                $store->save_action( $action );
+            }
+        }
+    }
 
-	public function test_date_queries() {
-		$actions = as_get_scheduled_actions(
-			array(
-				'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 30, $actions );
+    public function test_date_queries() {
+        $actions = as_get_scheduled_actions(
+            array(
+                'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 30, $actions );
 
-		$actions = as_get_scheduled_actions(
-			array(
-				'date'         => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
-				'date_compare' => '>=',
-				'per_page'     => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 70, $actions );
-	}
+        $actions = as_get_scheduled_actions(
+            array(
+                'date'         => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+                'date_compare' => '>=',
+                'per_page'     => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 70, $actions );
+    }
 
-	public function test_hook_queries() {
-		$actions = as_get_scheduled_actions(
-			array(
-				'hook'     => $this->hooks[2],
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 10, $actions );
+    public function test_hook_queries() {
+        $actions = as_get_scheduled_actions(
+            array(
+                'hook'     => $this->hooks[2],
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 10, $actions );
 
-		$actions = as_get_scheduled_actions(
-			array(
-				'hook'     => $this->hooks[2],
-				'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 3, $actions );
-	}
+        $actions = as_get_scheduled_actions(
+            array(
+                'hook'     => $this->hooks[2],
+                'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 3, $actions );
+    }
 
-	public function test_args_queries() {
-		$actions = as_get_scheduled_actions(
-			array(
-				'args'     => array( $this->args[5] ),
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 10, $actions );
+    public function test_args_queries() {
+        $actions = as_get_scheduled_actions(
+            array(
+                'args'     => array( $this->args[5] ),
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 10, $actions );
 
-		$actions = as_get_scheduled_actions(
-			array(
-				'args'     => array( $this->args[5] ),
-				'hook'     => $this->hooks[3],
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 1, $actions );
+        $actions = as_get_scheduled_actions(
+            array(
+                'args'     => array( $this->args[5] ),
+                'hook'     => $this->hooks[3],
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 1, $actions );
 
-		$actions = as_get_scheduled_actions(
-			array(
-				'args'     => array( $this->args[5] ),
-				'hook'     => $this->hooks[3],
-				'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 0, $actions );
-	}
+        $actions = as_get_scheduled_actions(
+            array(
+                'args'     => array( $this->args[5] ),
+                'hook'     => $this->hooks[3],
+                'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 0, $actions );
+    }
 
-	public function test_group_queries() {
-		$actions = as_get_scheduled_actions(
-			array(
-				'group'    => $this->groups[1],
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 10, $actions );
+    public function test_group_queries() {
+        $actions = as_get_scheduled_actions(
+            array(
+                'group'    => $this->groups[1],
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 10, $actions );
 
-		$actions = as_get_scheduled_actions(
-			array(
-				'group'    => $this->groups[1],
-				'hook'     => $this->hooks[9],
-				'per_page' => -1,
-			),
-			'ids'
-		);
-		$this->assertCount( 1, $actions );
-	}
+        $actions = as_get_scheduled_actions(
+            array(
+                'group'    => $this->groups[1],
+                'hook'     => $this->hooks[9],
+                'per_page' => -1,
+            ),
+            'ids'
+        );
+        $this->assertCount( 1, $actions );
+    }
 }
 

Spacing +26 added lines, -26 removed lines

@@ -13,18 +13,18 @@ 
 
 		$store = ActionScheduler::store();
 
-		for ( $i = 0; $i < 10; $i++ ) {
-			$this->hooks[ $i ]  = md5( rand() );
-			$this->args[ $i ]   = md5( rand() );
-			$this->groups[ $i ] = md5( rand() );
+		for ($i = 0; $i < 10; $i++) {
+			$this->hooks[$i]  = md5(rand());
+			$this->args[$i]   = md5(rand());
+			$this->groups[$i] = md5(rand());
 		}
 
-		for ( $i = 0; $i < 10; $i++ ) {
-			for ( $j = 0; $j < 10; $j++ ) {
-				$schedule = new ActionScheduler_SimpleSchedule( as_get_datetime_object( $j - 3 . 'days' ) );
-				$group    = $this->groups[ ( $i + $j ) % 10 ];
-				$action   = new ActionScheduler_Action( $this->hooks[ $i ], array( $this->args[ $j ] ), $schedule, $group );
-				$store->save_action( $action );
+		for ($i = 0; $i < 10; $i++) {
+			for ($j = 0; $j < 10; $j++) {
+				$schedule = new ActionScheduler_SimpleSchedule(as_get_datetime_object($j - 3.'days'));
+				$group    = $this->groups[($i + $j) % 10];
+				$action   = new ActionScheduler_Action($this->hooks[$i], array($this->args[$j]), $schedule, $group);
+				$store->save_action($action);
 			}
 		}
 	}
@@ -32,22 +32,22 @@ 
 	public function test_date_queries() {
 		$actions = as_get_scheduled_actions(
 			array(
-				'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+				'date'     => as_get_datetime_object(gmdate('Y-m-d 00:00:00')),
 				'per_page' => -1,
 			),
 			'ids'
 		);
-		$this->assertCount( 30, $actions );
+		$this->assertCount(30, $actions);
 
 		$actions = as_get_scheduled_actions(
 			array(
-				'date'         => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+				'date'         => as_get_datetime_object(gmdate('Y-m-d 00:00:00')),
 				'date_compare' => '>=',
 				'per_page'     => -1,
 			),
 			'ids'
 		);
-		$this->assertCount( 70, $actions );
+		$this->assertCount(70, $actions);
 	}
 
 	public function test_hook_queries() {
@@ -58,49 +58,49 @@ 
 			),
 			'ids'
 		);
-		$this->assertCount( 10, $actions );
+		$this->assertCount(10, $actions);
 
 		$actions = as_get_scheduled_actions(
 			array(
 				'hook'     => $this->hooks[2],
-				'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+				'date'     => as_get_datetime_object(gmdate('Y-m-d 00:00:00')),
 				'per_page' => -1,
 			),
 			'ids'
 		);
-		$this->assertCount( 3, $actions );
+		$this->assertCount(3, $actions);
 	}
 
 	public function test_args_queries() {
 		$actions = as_get_scheduled_actions(
 			array(
-				'args'     => array( $this->args[5] ),
+				'args'     => array($this->args[5]),
 				'per_page' => -1,
 			),
 			'ids'
 		);
-		$this->assertCount( 10, $actions );
+		$this->assertCount(10, $actions);
 
 		$actions = as_get_scheduled_actions(
 			array(
-				'args'     => array( $this->args[5] ),
+				'args'     => array($this->args[5]),
 				'hook'     => $this->hooks[3],
 				'per_page' => -1,
 			),
 			'ids'
 		);
-		$this->assertCount( 1, $actions );
+		$this->assertCount(1, $actions);
 
 		$actions = as_get_scheduled_actions(
 			array(
-				'args'     => array( $this->args[5] ),
+				'args'     => array($this->args[5]),
 				'hook'     => $this->hooks[3],
-				'date'     => as_get_datetime_object( gmdate( 'Y-m-d 00:00:00' ) ),
+				'date'     => as_get_datetime_object(gmdate('Y-m-d 00:00:00')),
 				'per_page' => -1,
 			),
 			'ids'
 		);
-		$this->assertCount( 0, $actions );
+		$this->assertCount(0, $actions);
 	}
 
 	public function test_group_queries() {
@@ -111,7 +111,7 @@ 
 			),
 			'ids'
 		);
-		$this->assertCount( 10, $actions );
+		$this->assertCount(10, $actions);
 
 		$actions = as_get_scheduled_actions(
 			array(
@@ -121,7 +121,7 @@ 
 			),
 			'ids'
 		);
-		$this->assertCount( 1, $actions );
+		$this->assertCount(1, $actions);
 	}
 }
 

src/libraries/action-scheduler/tests/phpunit/procedural_api/procedural_api_Test.php

Indentation +312 added lines, -312 removed lines

@@ -5,373 +5,373 @@
  */
 class Procedural_API_Test extends ActionScheduler_UnitTestCase {
 
-	// phpcs:disable Squiz.Commenting.FunctionComment.Missing, Squiz.Commenting.FunctionComment.MissingParamTag
-	public function test_schedule_action() {
-		$time      = time();
-		$hook      = md5( rand() );
-		$action_id = as_schedule_single_action( $time, $hook );
-
-		$store  = ActionScheduler::store();
-		$action = $store->fetch_action( $action_id );
-		$this->assertEquals( $time, $action->get_schedule()->get_date()->getTimestamp() );
-		$this->assertEquals( $hook, $action->get_hook() );
-	}
-
-	public function test_recurring_action() {
-		$time      = time();
-		$hook      = md5( rand() );
-		$action_id = as_schedule_recurring_action( $time, HOUR_IN_SECONDS, $hook );
-
-		$store  = ActionScheduler::store();
-		$action = $store->fetch_action( $action_id );
-		$this->assertEquals( $time, $action->get_schedule()->get_date()->getTimestamp() );
-		$this->assertEquals( $time + HOUR_IN_SECONDS + 2, $action->get_schedule()->get_next( as_get_datetime_object( $time + 2 ) )->getTimestamp() );
-		$this->assertEquals( $hook, $action->get_hook() );
-	}
-
-	public function test_cron_schedule() {
-		$time      = as_get_datetime_object( '2014-01-01' );
-		$hook      = md5( rand() );
-		$action_id = as_schedule_cron_action( $time->getTimestamp(), '0 0 10 10 *', $hook );
-
-		$store         = ActionScheduler::store();
-		$action        = $store->fetch_action( $action_id );
-		$expected_date = as_get_datetime_object( '2014-10-10' );
-		$this->assertEquals( $expected_date->getTimestamp(), $action->get_schedule()->get_date()->getTimestamp() );
-		$this->assertEquals( $hook, $action->get_hook() );
-
-		$expected_date = as_get_datetime_object( '2015-10-10' );
-		$this->assertEquals( $expected_date->getTimestamp(), $action->get_schedule()->get_next( as_get_datetime_object( '2015-01-02' ) )->getTimestamp() );
-	}
-
-	public function test_get_next() {
-		$time = as_get_datetime_object( 'tomorrow' );
-		$hook = md5( rand() );
-		as_schedule_recurring_action( $time->getTimestamp(), HOUR_IN_SECONDS, $hook );
-
-		$next = as_next_scheduled_action( $hook );
-
-		$this->assertEquals( $time->getTimestamp(), $next );
-	}
-
-	public function test_get_next_async() {
-		$hook      = md5( rand() );
-		$action_id = as_enqueue_async_action( $hook );
-
-		$next = as_next_scheduled_action( $hook );
-
-		$this->assertTrue( $next );
-
-		$store = ActionScheduler::store();
-
-		// Completed async actions should still return false.
-		$store->mark_complete( $action_id );
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
-
-		// Failed async actions should still return false.
-		$store->mark_failure( $action_id );
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
-
-		// Cancelled async actions should still return false.
-		$store->cancel_action( $action_id );
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
-	}
-
-	public function provider_time_hook_args_group() {
-		$time  = time() + 60 * 2;
-		$hook  = md5( rand() );
-		$args  = array( rand(), rand() );
-		$group = 'test_group';
-
-		return array(
-
-			// Test with no args or group.
-			array(
-				'time'  => $time,
-				'hook'  => $hook,
-				'args'  => array(),
-				'group' => '',
-			),
-
-			// Test with args but no group.
-			array(
-				'time'  => $time,
-				'hook'  => $hook,
-				'args'  => $args,
-				'group' => '',
-			),
-
-			// Test with group but no args.
-			array(
-				'time'  => $time,
-				'hook'  => $hook,
-				'args'  => array(),
-				'group' => $group,
-			),
-
-			// Test with args & group.
-			array(
-				'time'  => $time,
-				'hook'  => $hook,
-				'args'  => $args,
-				'group' => $group,
-			),
-		);
-	}
+    // phpcs:disable Squiz.Commenting.FunctionComment.Missing, Squiz.Commenting.FunctionComment.MissingParamTag
+    public function test_schedule_action() {
+        $time      = time();
+        $hook      = md5( rand() );
+        $action_id = as_schedule_single_action( $time, $hook );
+
+        $store  = ActionScheduler::store();
+        $action = $store->fetch_action( $action_id );
+        $this->assertEquals( $time, $action->get_schedule()->get_date()->getTimestamp() );
+        $this->assertEquals( $hook, $action->get_hook() );
+    }
+
+    public function test_recurring_action() {
+        $time      = time();
+        $hook      = md5( rand() );
+        $action_id = as_schedule_recurring_action( $time, HOUR_IN_SECONDS, $hook );
+
+        $store  = ActionScheduler::store();
+        $action = $store->fetch_action( $action_id );
+        $this->assertEquals( $time, $action->get_schedule()->get_date()->getTimestamp() );
+        $this->assertEquals( $time + HOUR_IN_SECONDS + 2, $action->get_schedule()->get_next( as_get_datetime_object( $time + 2 ) )->getTimestamp() );
+        $this->assertEquals( $hook, $action->get_hook() );
+    }
+
+    public function test_cron_schedule() {
+        $time      = as_get_datetime_object( '2014-01-01' );
+        $hook      = md5( rand() );
+        $action_id = as_schedule_cron_action( $time->getTimestamp(), '0 0 10 10 *', $hook );
+
+        $store         = ActionScheduler::store();
+        $action        = $store->fetch_action( $action_id );
+        $expected_date = as_get_datetime_object( '2014-10-10' );
+        $this->assertEquals( $expected_date->getTimestamp(), $action->get_schedule()->get_date()->getTimestamp() );
+        $this->assertEquals( $hook, $action->get_hook() );
+
+        $expected_date = as_get_datetime_object( '2015-10-10' );
+        $this->assertEquals( $expected_date->getTimestamp(), $action->get_schedule()->get_next( as_get_datetime_object( '2015-01-02' ) )->getTimestamp() );
+    }
+
+    public function test_get_next() {
+        $time = as_get_datetime_object( 'tomorrow' );
+        $hook = md5( rand() );
+        as_schedule_recurring_action( $time->getTimestamp(), HOUR_IN_SECONDS, $hook );
+
+        $next = as_next_scheduled_action( $hook );
+
+        $this->assertEquals( $time->getTimestamp(), $next );
+    }
+
+    public function test_get_next_async() {
+        $hook      = md5( rand() );
+        $action_id = as_enqueue_async_action( $hook );
+
+        $next = as_next_scheduled_action( $hook );
+
+        $this->assertTrue( $next );
+
+        $store = ActionScheduler::store();
+
+        // Completed async actions should still return false.
+        $store->mark_complete( $action_id );
+        $next = as_next_scheduled_action( $hook );
+        $this->assertFalse( $next );
+
+        // Failed async actions should still return false.
+        $store->mark_failure( $action_id );
+        $next = as_next_scheduled_action( $hook );
+        $this->assertFalse( $next );
+
+        // Cancelled async actions should still return false.
+        $store->cancel_action( $action_id );
+        $next = as_next_scheduled_action( $hook );
+        $this->assertFalse( $next );
+    }
+
+    public function provider_time_hook_args_group() {
+        $time  = time() + 60 * 2;
+        $hook  = md5( rand() );
+        $args  = array( rand(), rand() );
+        $group = 'test_group';
+
+        return array(
+
+            // Test with no args or group.
+            array(
+                'time'  => $time,
+                'hook'  => $hook,
+                'args'  => array(),
+                'group' => '',
+            ),
+
+            // Test with args but no group.
+            array(
+                'time'  => $time,
+                'hook'  => $hook,
+                'args'  => $args,
+                'group' => '',
+            ),
+
+            // Test with group but no args.
+            array(
+                'time'  => $time,
+                'hook'  => $hook,
+                'args'  => array(),
+                'group' => $group,
+            ),
+
+            // Test with args & group.
+            array(
+                'time'  => $time,
+                'hook'  => $hook,
+                'args'  => $args,
+                'group' => $group,
+            ),
+        );
+    }
 
-	/**
-	 * @dataProvider provider_time_hook_args_group
-	 */
-	public function test_unschedule( $time, $hook, $args, $group ) {
+    /**
+     * @dataProvider provider_time_hook_args_group
+     */
+    public function test_unschedule( $time, $hook, $args, $group ) {
 
-		$action_id_unscheduled = as_schedule_single_action( $time, $hook, $args, $group );
-		$action_scheduled_time = $time + 1;
-		$action_id_scheduled   = as_schedule_single_action( $action_scheduled_time, $hook, $args, $group );
+        $action_id_unscheduled = as_schedule_single_action( $time, $hook, $args, $group );
+        $action_scheduled_time = $time + 1;
+        $action_id_scheduled   = as_schedule_single_action( $action_scheduled_time, $hook, $args, $group );
 
-		as_unschedule_action( $hook, $args, $group );
+        as_unschedule_action( $hook, $args, $group );
 
-		$next = as_next_scheduled_action( $hook, $args, $group );
-		$this->assertEquals( $action_scheduled_time, $next );
+        $next = as_next_scheduled_action( $hook, $args, $group );
+        $this->assertEquals( $action_scheduled_time, $next );
 
-		$store              = ActionScheduler::store();
-		$unscheduled_action = $store->fetch_action( $action_id_unscheduled );
+        $store              = ActionScheduler::store();
+        $unscheduled_action = $store->fetch_action( $action_id_unscheduled );
 
-		// Make sure the next scheduled action is unscheduled.
-		$this->assertEquals( $hook, $unscheduled_action->get_hook() );
-		$this->assertEquals( as_get_datetime_object( $time ), $unscheduled_action->get_schedule()->get_date() );
-		$this->assertEquals( ActionScheduler_Store::STATUS_CANCELED, $store->get_status( $action_id_unscheduled ) );
-		$this->assertNull( $unscheduled_action->get_schedule()->get_next( as_get_datetime_object() ) );
+        // Make sure the next scheduled action is unscheduled.
+        $this->assertEquals( $hook, $unscheduled_action->get_hook() );
+        $this->assertEquals( as_get_datetime_object( $time ), $unscheduled_action->get_schedule()->get_date() );
+        $this->assertEquals( ActionScheduler_Store::STATUS_CANCELED, $store->get_status( $action_id_unscheduled ) );
+        $this->assertNull( $unscheduled_action->get_schedule()->get_next( as_get_datetime_object() ) );
 
-		// Make sure other scheduled actions are not unscheduled.
-		$this->assertEquals( ActionScheduler_Store::STATUS_PENDING, $store->get_status( $action_id_scheduled ) );
-		$scheduled_action = $store->fetch_action( $action_id_scheduled );
+        // Make sure other scheduled actions are not unscheduled.
+        $this->assertEquals( ActionScheduler_Store::STATUS_PENDING, $store->get_status( $action_id_scheduled ) );
+        $scheduled_action = $store->fetch_action( $action_id_scheduled );
 
-		$this->assertEquals( $hook, $scheduled_action->get_hook() );
-		$this->assertEquals( $action_scheduled_time, $scheduled_action->get_schedule()->get_date()->getTimestamp() );
-	}
+        $this->assertEquals( $hook, $scheduled_action->get_hook() );
+        $this->assertEquals( $action_scheduled_time, $scheduled_action->get_schedule()->get_date()->getTimestamp() );
+    }
 
-	/**
-	 * @dataProvider provider_time_hook_args_group
-	 */
-	public function test_unschedule_all( $time, $hook, $args, $group ) {
+    /**
+     * @dataProvider provider_time_hook_args_group
+     */
+    public function test_unschedule_all( $time, $hook, $args, $group ) {
 
-		$hook       = md5( $hook );
-		$action_ids = array();
+        $hook       = md5( $hook );
+        $action_ids = array();
 
-		for ( $i = 0; $i < 3; $i++ ) {
-			$action_ids[] = as_schedule_single_action( $time, $hook, $args, $group );
-		}
+        for ( $i = 0; $i < 3; $i++ ) {
+            $action_ids[] = as_schedule_single_action( $time, $hook, $args, $group );
+        }
 
-		as_unschedule_all_actions( $hook, $args, $group );
+        as_unschedule_all_actions( $hook, $args, $group );
 
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
+        $next = as_next_scheduled_action( $hook );
+        $this->assertFalse( $next );
 
-		$after = as_get_datetime_object( $time );
-		$after->modify( '+1 minute' );
+        $after = as_get_datetime_object( $time );
+        $after->modify( '+1 minute' );
 
-		$store = ActionScheduler::store();
+        $store = ActionScheduler::store();
 
-		foreach ( $action_ids as $action_id ) {
-			$action = $store->fetch_action( $action_id );
+        foreach ( $action_ids as $action_id ) {
+            $action = $store->fetch_action( $action_id );
 
-			$this->assertEquals( $hook, $action->get_hook() );
-			$this->assertEquals( as_get_datetime_object( $time ), $action->get_schedule()->get_date() );
-			$this->assertEquals( ActionScheduler_Store::STATUS_CANCELED, $store->get_status( $action_id ) );
-			$this->assertNull( $action->get_schedule()->get_next( $after ) );
-		}
-	}
+            $this->assertEquals( $hook, $action->get_hook() );
+            $this->assertEquals( as_get_datetime_object( $time ), $action->get_schedule()->get_date() );
+            $this->assertEquals( ActionScheduler_Store::STATUS_CANCELED, $store->get_status( $action_id ) );
+            $this->assertNull( $action->get_schedule()->get_next( $after ) );
+        }
+    }
 
-	public function test_as_get_datetime_object_default() {
+    public function test_as_get_datetime_object_default() {
 
-		$utc_now = new ActionScheduler_DateTime( null, new DateTimeZone( 'UTC' ) );
-		$as_now  = as_get_datetime_object();
+        $utc_now = new ActionScheduler_DateTime( null, new DateTimeZone( 'UTC' ) );
+        $as_now  = as_get_datetime_object();
 
-		// Don't want to use 'U' as timestamps will always be in UTC.
-		$this->assertEquals( $utc_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
-	}
+        // Don't want to use 'U' as timestamps will always be in UTC.
+        $this->assertEquals( $utc_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
+    }
 
-	public function test_as_get_datetime_object_relative() {
+    public function test_as_get_datetime_object_relative() {
 
-		$utc_tomorrow = new ActionScheduler_DateTime( 'tomorrow', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( 'tomorrow' );
+        $utc_tomorrow = new ActionScheduler_DateTime( 'tomorrow', new DateTimeZone( 'UTC' ) );
+        $as_tomorrow  = as_get_datetime_object( 'tomorrow' );
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+        $this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
 
-		$utc_tomorrow = new ActionScheduler_DateTime( 'yesterday', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( 'yesterday' );
+        $utc_tomorrow = new ActionScheduler_DateTime( 'yesterday', new DateTimeZone( 'UTC' ) );
+        $as_tomorrow  = as_get_datetime_object( 'yesterday' );
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
-	}
+        $this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+    }
 
-	public function test_as_get_datetime_object_fixed() {
+    public function test_as_get_datetime_object_fixed() {
 
-		$utc_tomorrow = new ActionScheduler_DateTime( '29 February 2016', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( '29 February 2016' );
+        $utc_tomorrow = new ActionScheduler_DateTime( '29 February 2016', new DateTimeZone( 'UTC' ) );
+        $as_tomorrow  = as_get_datetime_object( '29 February 2016' );
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+        $this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
 
-		$utc_tomorrow = new ActionScheduler_DateTime( '1st January 2024', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( '1st January 2024' );
+        $utc_tomorrow = new ActionScheduler_DateTime( '1st January 2024', new DateTimeZone( 'UTC' ) );
+        $as_tomorrow  = as_get_datetime_object( '1st January 2024' );
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
-	}
+        $this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+    }
 
-	public function test_as_get_datetime_object_timezone() {
+    public function test_as_get_datetime_object_timezone() {
 
-		$timezone_au      = 'Australia/Brisbane';
-		$timezone_default = date_default_timezone_get();
+        $timezone_au      = 'Australia/Brisbane';
+        $timezone_default = date_default_timezone_get();
 
-		// phpcs:ignore
-		date_default_timezone_set( $timezone_au );
+        // phpcs:ignore
+        date_default_timezone_set( $timezone_au );
 
-		$au_now = new ActionScheduler_DateTime( null );
-		$as_now = as_get_datetime_object();
+        $au_now = new ActionScheduler_DateTime( null );
+        $as_now = as_get_datetime_object();
 
-		// Make sure they're for the same time.
-		$this->assertEquals( $au_now->getTimestamp(), $as_now->getTimestamp() );
+        // Make sure they're for the same time.
+        $this->assertEquals( $au_now->getTimestamp(), $as_now->getTimestamp() );
 
-		// But not in the same timezone, as $as_now should be using UTC.
-		$this->assertNotEquals( $au_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
+        // But not in the same timezone, as $as_now should be using UTC.
+        $this->assertNotEquals( $au_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
 
-		$au_now    = new ActionScheduler_DateTime( null );
-		$as_au_now = as_get_datetime_object();
+        $au_now    = new ActionScheduler_DateTime( null );
+        $as_au_now = as_get_datetime_object();
 
-		$this->assertEquals( $au_now->getTimestamp(), $as_now->getTimestamp(), '', 2 );
+        $this->assertEquals( $au_now->getTimestamp(), $as_now->getTimestamp(), '', 2 );
 
-		// But not in the same timezone, as $as_now should be using UTC.
-		$this->assertNotEquals( $au_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
+        // But not in the same timezone, as $as_now should be using UTC.
+        $this->assertNotEquals( $au_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
 
-		// phpcs:ignore
-		date_default_timezone_set( $timezone_default );
-	}
+        // phpcs:ignore
+        date_default_timezone_set( $timezone_default );
+    }
 
-	public function test_as_get_datetime_object_type() {
-		$f   = 'Y-m-d H:i:s';
-		$now = as_get_datetime_object();
-		$this->assertInstanceOf( 'ActionScheduler_DateTime', $now );
+    public function test_as_get_datetime_object_type() {
+        $f   = 'Y-m-d H:i:s';
+        $now = as_get_datetime_object();
+        $this->assertInstanceOf( 'ActionScheduler_DateTime', $now );
 
-		$datetime    = new DateTime( 'now', new DateTimeZone( 'UTC' ) );
-		$as_datetime = as_get_datetime_object( $datetime );
-		$this->assertEquals( $datetime->format( $f ), $as_datetime->format( $f ) );
-	}
+        $datetime    = new DateTime( 'now', new DateTimeZone( 'UTC' ) );
+        $as_datetime = as_get_datetime_object( $datetime );
+        $this->assertEquals( $datetime->format( $f ), $as_datetime->format( $f ) );
+    }
 
-	public function test_as_has_scheduled_action() {
-		$store = ActionScheduler::store();
+    public function test_as_has_scheduled_action() {
+        $store = ActionScheduler::store();
 
-		$time      = as_get_datetime_object( 'tomorrow' );
-		$action_id = as_schedule_single_action( $time->getTimestamp(), 'hook_1' );
+        $time      = as_get_datetime_object( 'tomorrow' );
+        $action_id = as_schedule_single_action( $time->getTimestamp(), 'hook_1' );
 
-		$this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
-		$this->assertFalse( as_has_scheduled_action( 'hook_2' ) );
+        $this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
+        $this->assertFalse( as_has_scheduled_action( 'hook_2' ) );
 
-		// Go to in-progress.
-		$store->log_execution( $action_id );
-		$this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
+        // Go to in-progress.
+        $store->log_execution( $action_id );
+        $this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
 
-		// Go to complete.
-		$store->mark_complete( $action_id );
-		$this->assertFalse( as_has_scheduled_action( 'hook_1' ) );
+        // Go to complete.
+        $store->mark_complete( $action_id );
+        $this->assertFalse( as_has_scheduled_action( 'hook_1' ) );
 
-		// Go to failed.
-		$store->mark_failure( $action_id );
-		$this->assertFalse( as_has_scheduled_action( 'hook_1' ) );
-	}
+        // Go to failed.
+        $store->mark_failure( $action_id );
+        $this->assertFalse( as_has_scheduled_action( 'hook_1' ) );
+    }
 
-	public function test_as_has_scheduled_action_with_args() {
-		as_schedule_single_action( time(), 'hook_1', array( 'a' ) );
+    public function test_as_has_scheduled_action_with_args() {
+        as_schedule_single_action( time(), 'hook_1', array( 'a' ) );
 
-		$this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
-		$this->assertFalse( as_has_scheduled_action( 'hook_1', array( 'b' ) ) );
+        $this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
+        $this->assertFalse( as_has_scheduled_action( 'hook_1', array( 'b' ) ) );
 
-		// Test for any args.
-		$this->assertTrue( as_has_scheduled_action( 'hook_1', array( 'a' ) ) );
-	}
+        // Test for any args.
+        $this->assertTrue( as_has_scheduled_action( 'hook_1', array( 'a' ) ) );
+    }
 
-	public function test_as_has_scheduled_action_with_group() {
-		as_schedule_single_action( time(), 'hook_1', array(), 'group_1' );
+    public function test_as_has_scheduled_action_with_group() {
+        as_schedule_single_action( time(), 'hook_1', array(), 'group_1' );
 
-		$this->assertTrue( as_has_scheduled_action( 'hook_1', null, 'group_1' ) );
-		$this->assertTrue( as_has_scheduled_action( 'hook_1', array(), 'group_1' ) );
-	}
-	// phpcs:enable
+        $this->assertTrue( as_has_scheduled_action( 'hook_1', null, 'group_1' ) );
+        $this->assertTrue( as_has_scheduled_action( 'hook_1', array(), 'group_1' ) );
+    }
+    // phpcs:enable
 
-	/**
-	 * Test as_enqueue_async_action with unique param.
-	 */
-	public function test_as_enqueue_async_action_unique() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+    /**
+     * Test as_enqueue_async_action with unique param.
+     */
+    public function test_as_enqueue_async_action_unique() {
+        $this->set_action_scheduler_store( new ActionScheduler_DBStore() );
 
-		$action_id = as_enqueue_async_action( 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
+        $action_id = as_enqueue_async_action( 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertValidAction( $action_id );
 
-		$action_id_duplicate = as_enqueue_async_action( 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
-	}
+        $action_id_duplicate = as_enqueue_async_action( 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertSame( 0, $action_id_duplicate );
+    }
 
-	/**
-	 * Test as_schedule_single_action with unique param.
-	 */
-	public function test_as_schedule_single_action_unique() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
-
-		$action_id = as_schedule_single_action( time(), 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
-
-		$action_id_duplicate = as_schedule_single_action( time(), 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
-	}
-
-	/**
-	 * Test as_schedule_recurring_action with unique param.
-	 */
-	public function test_as_schedule_recurring_action_unique() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
-
-		$action_id = as_schedule_recurring_action( time(), MINUTE_IN_SECONDS, 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
-
-		$action_id_duplicate = as_schedule_recurring_action( time(), MINUTE_IN_SECONDS, 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
-	}
-
-	/**
-	 * Test as_schedule_cron with unique param.
-	 */
-	public function test_as_schedule_cron_action() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
-
-		$action_id = as_schedule_cron_action( time(), '0 0 * * *', 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
-
-		$action_id_duplicate = as_schedule_cron_action( time(), '0 0 * * *', 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
-	}
-
-	/**
-	 * Helper method to set actions scheduler store.
-	 *
-	 * @param ActionScheduler_Store $store Store instance to set.
-	 */
-	private function set_action_scheduler_store( $store ) {
-		$store_factory_setter        = function () use ( $store ) {
-			self::$store = $store;
-		};
-		$binded_store_factory_setter = Closure::bind( $store_factory_setter, null, ActionScheduler_Store::class );
-		$binded_store_factory_setter();
-	}
-
-	/**
-	 * Helper method to assert valid action.
-	 *
-	 * @param int $action_id Action ID to assert.
-	 */
-	private function assertValidAction( $action_id ) {
-		$this->assertNotEquals( 0, $action_id );
-		$action = ActionScheduler::store()->fetch_action( $action_id );
-		$this->assertInstanceOf( 'ActionScheduler_Action', $action );
-	}
+    /**
+     * Test as_schedule_single_action with unique param.
+     */
+    public function test_as_schedule_single_action_unique() {
+        $this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+
+        $action_id = as_schedule_single_action( time(), 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertValidAction( $action_id );
+
+        $action_id_duplicate = as_schedule_single_action( time(), 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertSame( 0, $action_id_duplicate );
+    }
+
+    /**
+     * Test as_schedule_recurring_action with unique param.
+     */
+    public function test_as_schedule_recurring_action_unique() {
+        $this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+
+        $action_id = as_schedule_recurring_action( time(), MINUTE_IN_SECONDS, 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertValidAction( $action_id );
+
+        $action_id_duplicate = as_schedule_recurring_action( time(), MINUTE_IN_SECONDS, 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertSame( 0, $action_id_duplicate );
+    }
+
+    /**
+     * Test as_schedule_cron with unique param.
+     */
+    public function test_as_schedule_cron_action() {
+        $this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+
+        $action_id = as_schedule_cron_action( time(), '0 0 * * *', 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertValidAction( $action_id );
+
+        $action_id_duplicate = as_schedule_cron_action( time(), '0 0 * * *', 'hook_1', array( 'a' ), 'dummy', true );
+        $this->assertSame( 0, $action_id_duplicate );
+    }
+
+    /**
+     * Helper method to set actions scheduler store.
+     *
+     * @param ActionScheduler_Store $store Store instance to set.
+     */
+    private function set_action_scheduler_store( $store ) {
+        $store_factory_setter        = function () use ( $store ) {
+            self::$store = $store;
+        };
+        $binded_store_factory_setter = Closure::bind( $store_factory_setter, null, ActionScheduler_Store::class );
+        $binded_store_factory_setter();
+    }
+
+    /**
+     * Helper method to assert valid action.
+     *
+     * @param int $action_id Action ID to assert.
+     */
+    private function assertValidAction( $action_id ) {
+        $this->assertNotEquals( 0, $action_id );
+        $action = ActionScheduler::store()->fetch_action( $action_id );
+        $this->assertInstanceOf( 'ActionScheduler_Action', $action );
+    }
 }

Spacing +140 added lines, -140 removed lines

@@ -8,82 +8,82 @@ 
 	// phpcs:disable Squiz.Commenting.FunctionComment.Missing, Squiz.Commenting.FunctionComment.MissingParamTag
 	public function test_schedule_action() {
 		$time      = time();
-		$hook      = md5( rand() );
-		$action_id = as_schedule_single_action( $time, $hook );
+		$hook      = md5(rand());
+		$action_id = as_schedule_single_action($time, $hook);
 
 		$store  = ActionScheduler::store();
-		$action = $store->fetch_action( $action_id );
-		$this->assertEquals( $time, $action->get_schedule()->get_date()->getTimestamp() );
-		$this->assertEquals( $hook, $action->get_hook() );
+		$action = $store->fetch_action($action_id);
+		$this->assertEquals($time, $action->get_schedule()->get_date()->getTimestamp());
+		$this->assertEquals($hook, $action->get_hook());
 	}
 
 	public function test_recurring_action() {
 		$time      = time();
-		$hook      = md5( rand() );
-		$action_id = as_schedule_recurring_action( $time, HOUR_IN_SECONDS, $hook );
+		$hook      = md5(rand());
+		$action_id = as_schedule_recurring_action($time, HOUR_IN_SECONDS, $hook);
 
 		$store  = ActionScheduler::store();
-		$action = $store->fetch_action( $action_id );
-		$this->assertEquals( $time, $action->get_schedule()->get_date()->getTimestamp() );
-		$this->assertEquals( $time + HOUR_IN_SECONDS + 2, $action->get_schedule()->get_next( as_get_datetime_object( $time + 2 ) )->getTimestamp() );
-		$this->assertEquals( $hook, $action->get_hook() );
+		$action = $store->fetch_action($action_id);
+		$this->assertEquals($time, $action->get_schedule()->get_date()->getTimestamp());
+		$this->assertEquals($time + HOUR_IN_SECONDS + 2, $action->get_schedule()->get_next(as_get_datetime_object($time + 2))->getTimestamp());
+		$this->assertEquals($hook, $action->get_hook());
 	}
 
 	public function test_cron_schedule() {
-		$time      = as_get_datetime_object( '2014-01-01' );
-		$hook      = md5( rand() );
-		$action_id = as_schedule_cron_action( $time->getTimestamp(), '0 0 10 10 *', $hook );
+		$time      = as_get_datetime_object('2014-01-01');
+		$hook      = md5(rand());
+		$action_id = as_schedule_cron_action($time->getTimestamp(), '0 0 10 10 *', $hook);
 
 		$store         = ActionScheduler::store();
-		$action        = $store->fetch_action( $action_id );
-		$expected_date = as_get_datetime_object( '2014-10-10' );
-		$this->assertEquals( $expected_date->getTimestamp(), $action->get_schedule()->get_date()->getTimestamp() );
-		$this->assertEquals( $hook, $action->get_hook() );
+		$action        = $store->fetch_action($action_id);
+		$expected_date = as_get_datetime_object('2014-10-10');
+		$this->assertEquals($expected_date->getTimestamp(), $action->get_schedule()->get_date()->getTimestamp());
+		$this->assertEquals($hook, $action->get_hook());
 
-		$expected_date = as_get_datetime_object( '2015-10-10' );
-		$this->assertEquals( $expected_date->getTimestamp(), $action->get_schedule()->get_next( as_get_datetime_object( '2015-01-02' ) )->getTimestamp() );
+		$expected_date = as_get_datetime_object('2015-10-10');
+		$this->assertEquals($expected_date->getTimestamp(), $action->get_schedule()->get_next(as_get_datetime_object('2015-01-02'))->getTimestamp());
 	}
 
 	public function test_get_next() {
-		$time = as_get_datetime_object( 'tomorrow' );
-		$hook = md5( rand() );
-		as_schedule_recurring_action( $time->getTimestamp(), HOUR_IN_SECONDS, $hook );
+		$time = as_get_datetime_object('tomorrow');
+		$hook = md5(rand());
+		as_schedule_recurring_action($time->getTimestamp(), HOUR_IN_SECONDS, $hook);
 
-		$next = as_next_scheduled_action( $hook );
+		$next = as_next_scheduled_action($hook);
 
-		$this->assertEquals( $time->getTimestamp(), $next );
+		$this->assertEquals($time->getTimestamp(), $next);
 	}
 
 	public function test_get_next_async() {
-		$hook      = md5( rand() );
-		$action_id = as_enqueue_async_action( $hook );
+		$hook      = md5(rand());
+		$action_id = as_enqueue_async_action($hook);
 
-		$next = as_next_scheduled_action( $hook );
+		$next = as_next_scheduled_action($hook);
 
-		$this->assertTrue( $next );
+		$this->assertTrue($next);
 
 		$store = ActionScheduler::store();
 
 		// Completed async actions should still return false.
-		$store->mark_complete( $action_id );
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
+		$store->mark_complete($action_id);
+		$next = as_next_scheduled_action($hook);
+		$this->assertFalse($next);
 
 		// Failed async actions should still return false.
-		$store->mark_failure( $action_id );
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
+		$store->mark_failure($action_id);
+		$next = as_next_scheduled_action($hook);
+		$this->assertFalse($next);
 
 		// Cancelled async actions should still return false.
-		$store->cancel_action( $action_id );
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
+		$store->cancel_action($action_id);
+		$next = as_next_scheduled_action($hook);
+		$this->assertFalse($next);
 	}
 
 	public function provider_time_hook_args_group() {
 		$time  = time() + 60 * 2;
-		$hook  = md5( rand() );
-		$args  = array( rand(), rand() );
+		$hook  = md5(rand());
+		$args  = array(rand(), rand());
 		$group = 'test_group';
 
 		return array(
@@ -125,99 +125,99 @@ 
 	/**
 	 * @dataProvider provider_time_hook_args_group
 	 */
-	public function test_unschedule( $time, $hook, $args, $group ) {
+	public function test_unschedule($time, $hook, $args, $group) {
 
-		$action_id_unscheduled = as_schedule_single_action( $time, $hook, $args, $group );
+		$action_id_unscheduled = as_schedule_single_action($time, $hook, $args, $group);
 		$action_scheduled_time = $time + 1;
-		$action_id_scheduled   = as_schedule_single_action( $action_scheduled_time, $hook, $args, $group );
+		$action_id_scheduled   = as_schedule_single_action($action_scheduled_time, $hook, $args, $group);
 
-		as_unschedule_action( $hook, $args, $group );
+		as_unschedule_action($hook, $args, $group);
 
-		$next = as_next_scheduled_action( $hook, $args, $group );
-		$this->assertEquals( $action_scheduled_time, $next );
+		$next = as_next_scheduled_action($hook, $args, $group);
+		$this->assertEquals($action_scheduled_time, $next);
 
 		$store              = ActionScheduler::store();
-		$unscheduled_action = $store->fetch_action( $action_id_unscheduled );
+		$unscheduled_action = $store->fetch_action($action_id_unscheduled);
 
 		// Make sure the next scheduled action is unscheduled.
-		$this->assertEquals( $hook, $unscheduled_action->get_hook() );
-		$this->assertEquals( as_get_datetime_object( $time ), $unscheduled_action->get_schedule()->get_date() );
-		$this->assertEquals( ActionScheduler_Store::STATUS_CANCELED, $store->get_status( $action_id_unscheduled ) );
-		$this->assertNull( $unscheduled_action->get_schedule()->get_next( as_get_datetime_object() ) );
+		$this->assertEquals($hook, $unscheduled_action->get_hook());
+		$this->assertEquals(as_get_datetime_object($time), $unscheduled_action->get_schedule()->get_date());
+		$this->assertEquals(ActionScheduler_Store::STATUS_CANCELED, $store->get_status($action_id_unscheduled));
+		$this->assertNull($unscheduled_action->get_schedule()->get_next(as_get_datetime_object()));
 
 		// Make sure other scheduled actions are not unscheduled.
-		$this->assertEquals( ActionScheduler_Store::STATUS_PENDING, $store->get_status( $action_id_scheduled ) );
-		$scheduled_action = $store->fetch_action( $action_id_scheduled );
+		$this->assertEquals(ActionScheduler_Store::STATUS_PENDING, $store->get_status($action_id_scheduled));
+		$scheduled_action = $store->fetch_action($action_id_scheduled);
 
-		$this->assertEquals( $hook, $scheduled_action->get_hook() );
-		$this->assertEquals( $action_scheduled_time, $scheduled_action->get_schedule()->get_date()->getTimestamp() );
+		$this->assertEquals($hook, $scheduled_action->get_hook());
+		$this->assertEquals($action_scheduled_time, $scheduled_action->get_schedule()->get_date()->getTimestamp());
 	}
 
 	/**
 	 * @dataProvider provider_time_hook_args_group
 	 */
-	public function test_unschedule_all( $time, $hook, $args, $group ) {
+	public function test_unschedule_all($time, $hook, $args, $group) {
 
-		$hook       = md5( $hook );
+		$hook       = md5($hook);
 		$action_ids = array();
 
-		for ( $i = 0; $i < 3; $i++ ) {
-			$action_ids[] = as_schedule_single_action( $time, $hook, $args, $group );
+		for ($i = 0; $i < 3; $i++) {
+			$action_ids[] = as_schedule_single_action($time, $hook, $args, $group);
 		}
 
-		as_unschedule_all_actions( $hook, $args, $group );
+		as_unschedule_all_actions($hook, $args, $group);
 
-		$next = as_next_scheduled_action( $hook );
-		$this->assertFalse( $next );
+		$next = as_next_scheduled_action($hook);
+		$this->assertFalse($next);
 
-		$after = as_get_datetime_object( $time );
-		$after->modify( '+1 minute' );
+		$after = as_get_datetime_object($time);
+		$after->modify('+1 minute');
 
 		$store = ActionScheduler::store();
 
-		foreach ( $action_ids as $action_id ) {
-			$action = $store->fetch_action( $action_id );
+		foreach ($action_ids as $action_id) {
+			$action = $store->fetch_action($action_id);
 
-			$this->assertEquals( $hook, $action->get_hook() );
-			$this->assertEquals( as_get_datetime_object( $time ), $action->get_schedule()->get_date() );
-			$this->assertEquals( ActionScheduler_Store::STATUS_CANCELED, $store->get_status( $action_id ) );
-			$this->assertNull( $action->get_schedule()->get_next( $after ) );
+			$this->assertEquals($hook, $action->get_hook());
+			$this->assertEquals(as_get_datetime_object($time), $action->get_schedule()->get_date());
+			$this->assertEquals(ActionScheduler_Store::STATUS_CANCELED, $store->get_status($action_id));
+			$this->assertNull($action->get_schedule()->get_next($after));
 		}
 	}
 
 	public function test_as_get_datetime_object_default() {
 
-		$utc_now = new ActionScheduler_DateTime( null, new DateTimeZone( 'UTC' ) );
+		$utc_now = new ActionScheduler_DateTime(null, new DateTimeZone('UTC'));
 		$as_now  = as_get_datetime_object();
 
 		// Don't want to use 'U' as timestamps will always be in UTC.
-		$this->assertEquals( $utc_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
+		$this->assertEquals($utc_now->format('Y-m-d H:i:s'), $as_now->format('Y-m-d H:i:s'));
 	}
 
 	public function test_as_get_datetime_object_relative() {
 
-		$utc_tomorrow = new ActionScheduler_DateTime( 'tomorrow', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( 'tomorrow' );
+		$utc_tomorrow = new ActionScheduler_DateTime('tomorrow', new DateTimeZone('UTC'));
+		$as_tomorrow  = as_get_datetime_object('tomorrow');
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+		$this->assertEquals($utc_tomorrow->format('Y-m-d H:i:s'), $as_tomorrow->format('Y-m-d H:i:s'));
 
-		$utc_tomorrow = new ActionScheduler_DateTime( 'yesterday', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( 'yesterday' );
+		$utc_tomorrow = new ActionScheduler_DateTime('yesterday', new DateTimeZone('UTC'));
+		$as_tomorrow  = as_get_datetime_object('yesterday');
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+		$this->assertEquals($utc_tomorrow->format('Y-m-d H:i:s'), $as_tomorrow->format('Y-m-d H:i:s'));
 	}
 
 	public function test_as_get_datetime_object_fixed() {
 
-		$utc_tomorrow = new ActionScheduler_DateTime( '29 February 2016', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( '29 February 2016' );
+		$utc_tomorrow = new ActionScheduler_DateTime('29 February 2016', new DateTimeZone('UTC'));
+		$as_tomorrow  = as_get_datetime_object('29 February 2016');
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+		$this->assertEquals($utc_tomorrow->format('Y-m-d H:i:s'), $as_tomorrow->format('Y-m-d H:i:s'));
 
-		$utc_tomorrow = new ActionScheduler_DateTime( '1st January 2024', new DateTimeZone( 'UTC' ) );
-		$as_tomorrow  = as_get_datetime_object( '1st January 2024' );
+		$utc_tomorrow = new ActionScheduler_DateTime('1st January 2024', new DateTimeZone('UTC'));
+		$as_tomorrow  = as_get_datetime_object('1st January 2024');
 
-		$this->assertEquals( $utc_tomorrow->format( 'Y-m-d H:i:s' ), $as_tomorrow->format( 'Y-m-d H:i:s' ) );
+		$this->assertEquals($utc_tomorrow->format('Y-m-d H:i:s'), $as_tomorrow->format('Y-m-d H:i:s'));
 	}
 
 	public function test_as_get_datetime_object_timezone() {
@@ -226,76 +226,76 @@ 
 		$timezone_default = date_default_timezone_get();
 
 		// phpcs:ignore
-		date_default_timezone_set( $timezone_au );
+		date_default_timezone_set($timezone_au);
 
-		$au_now = new ActionScheduler_DateTime( null );
+		$au_now = new ActionScheduler_DateTime(null);
 		$as_now = as_get_datetime_object();
 
 		// Make sure they're for the same time.
-		$this->assertEquals( $au_now->getTimestamp(), $as_now->getTimestamp() );
+		$this->assertEquals($au_now->getTimestamp(), $as_now->getTimestamp());
 
 		// But not in the same timezone, as $as_now should be using UTC.
-		$this->assertNotEquals( $au_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
+		$this->assertNotEquals($au_now->format('Y-m-d H:i:s'), $as_now->format('Y-m-d H:i:s'));
 
-		$au_now    = new ActionScheduler_DateTime( null );
+		$au_now    = new ActionScheduler_DateTime(null);
 		$as_au_now = as_get_datetime_object();
 
-		$this->assertEquals( $au_now->getTimestamp(), $as_now->getTimestamp(), '', 2 );
+		$this->assertEquals($au_now->getTimestamp(), $as_now->getTimestamp(), '', 2);
 
 		// But not in the same timezone, as $as_now should be using UTC.
-		$this->assertNotEquals( $au_now->format( 'Y-m-d H:i:s' ), $as_now->format( 'Y-m-d H:i:s' ) );
+		$this->assertNotEquals($au_now->format('Y-m-d H:i:s'), $as_now->format('Y-m-d H:i:s'));
 
 		// phpcs:ignore
-		date_default_timezone_set( $timezone_default );
+		date_default_timezone_set($timezone_default);
 	}
 
 	public function test_as_get_datetime_object_type() {
 		$f   = 'Y-m-d H:i:s';
 		$now = as_get_datetime_object();
-		$this->assertInstanceOf( 'ActionScheduler_DateTime', $now );
+		$this->assertInstanceOf('ActionScheduler_DateTime', $now);
 
-		$datetime    = new DateTime( 'now', new DateTimeZone( 'UTC' ) );
-		$as_datetime = as_get_datetime_object( $datetime );
-		$this->assertEquals( $datetime->format( $f ), $as_datetime->format( $f ) );
+		$datetime    = new DateTime('now', new DateTimeZone('UTC'));
+		$as_datetime = as_get_datetime_object($datetime);
+		$this->assertEquals($datetime->format($f), $as_datetime->format($f));
 	}
 
 	public function test_as_has_scheduled_action() {
 		$store = ActionScheduler::store();
 
-		$time      = as_get_datetime_object( 'tomorrow' );
-		$action_id = as_schedule_single_action( $time->getTimestamp(), 'hook_1' );
+		$time      = as_get_datetime_object('tomorrow');
+		$action_id = as_schedule_single_action($time->getTimestamp(), 'hook_1');
 
-		$this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
-		$this->assertFalse( as_has_scheduled_action( 'hook_2' ) );
+		$this->assertTrue(as_has_scheduled_action('hook_1'));
+		$this->assertFalse(as_has_scheduled_action('hook_2'));
 
 		// Go to in-progress.
-		$store->log_execution( $action_id );
-		$this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
+		$store->log_execution($action_id);
+		$this->assertTrue(as_has_scheduled_action('hook_1'));
 
 		// Go to complete.
-		$store->mark_complete( $action_id );
-		$this->assertFalse( as_has_scheduled_action( 'hook_1' ) );
+		$store->mark_complete($action_id);
+		$this->assertFalse(as_has_scheduled_action('hook_1'));
 
 		// Go to failed.
-		$store->mark_failure( $action_id );
-		$this->assertFalse( as_has_scheduled_action( 'hook_1' ) );
+		$store->mark_failure($action_id);
+		$this->assertFalse(as_has_scheduled_action('hook_1'));
 	}
 
 	public function test_as_has_scheduled_action_with_args() {
-		as_schedule_single_action( time(), 'hook_1', array( 'a' ) );
+		as_schedule_single_action(time(), 'hook_1', array('a'));
 
-		$this->assertTrue( as_has_scheduled_action( 'hook_1' ) );
-		$this->assertFalse( as_has_scheduled_action( 'hook_1', array( 'b' ) ) );
+		$this->assertTrue(as_has_scheduled_action('hook_1'));
+		$this->assertFalse(as_has_scheduled_action('hook_1', array('b')));
 
 		// Test for any args.
-		$this->assertTrue( as_has_scheduled_action( 'hook_1', array( 'a' ) ) );
+		$this->assertTrue(as_has_scheduled_action('hook_1', array('a')));
 	}
 
 	public function test_as_has_scheduled_action_with_group() {
-		as_schedule_single_action( time(), 'hook_1', array(), 'group_1' );
+		as_schedule_single_action(time(), 'hook_1', array(), 'group_1');
 
-		$this->assertTrue( as_has_scheduled_action( 'hook_1', null, 'group_1' ) );
-		$this->assertTrue( as_has_scheduled_action( 'hook_1', array(), 'group_1' ) );
+		$this->assertTrue(as_has_scheduled_action('hook_1', null, 'group_1'));
+		$this->assertTrue(as_has_scheduled_action('hook_1', array(), 'group_1'));
 	}
 	// phpcs:enable
 
@@ -303,52 +303,52 @@ 
 	 * Test as_enqueue_async_action with unique param.
 	 */
 	public function test_as_enqueue_async_action_unique() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+		$this->set_action_scheduler_store(new ActionScheduler_DBStore());
 
-		$action_id = as_enqueue_async_action( 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
+		$action_id = as_enqueue_async_action('hook_1', array('a'), 'dummy', true);
+		$this->assertValidAction($action_id);
 
-		$action_id_duplicate = as_enqueue_async_action( 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
+		$action_id_duplicate = as_enqueue_async_action('hook_1', array('a'), 'dummy', true);
+		$this->assertSame(0, $action_id_duplicate);
 	}
 
 	/**
 	 * Test as_schedule_single_action with unique param.
 	 */
 	public function test_as_schedule_single_action_unique() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+		$this->set_action_scheduler_store(new ActionScheduler_DBStore());
 
-		$action_id = as_schedule_single_action( time(), 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
+		$action_id = as_schedule_single_action(time(), 'hook_1', array('a'), 'dummy', true);
+		$this->assertValidAction($action_id);
 
-		$action_id_duplicate = as_schedule_single_action( time(), 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
+		$action_id_duplicate = as_schedule_single_action(time(), 'hook_1', array('a'), 'dummy', true);
+		$this->assertSame(0, $action_id_duplicate);
 	}
 
 	/**
 	 * Test as_schedule_recurring_action with unique param.
 	 */
 	public function test_as_schedule_recurring_action_unique() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+		$this->set_action_scheduler_store(new ActionScheduler_DBStore());
 
-		$action_id = as_schedule_recurring_action( time(), MINUTE_IN_SECONDS, 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
+		$action_id = as_schedule_recurring_action(time(), MINUTE_IN_SECONDS, 'hook_1', array('a'), 'dummy', true);
+		$this->assertValidAction($action_id);
 
-		$action_id_duplicate = as_schedule_recurring_action( time(), MINUTE_IN_SECONDS, 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
+		$action_id_duplicate = as_schedule_recurring_action(time(), MINUTE_IN_SECONDS, 'hook_1', array('a'), 'dummy', true);
+		$this->assertSame(0, $action_id_duplicate);
 	}
 
 	/**
 	 * Test as_schedule_cron with unique param.
 	 */
 	public function test_as_schedule_cron_action() {
-		$this->set_action_scheduler_store( new ActionScheduler_DBStore() );
+		$this->set_action_scheduler_store(new ActionScheduler_DBStore());
 
-		$action_id = as_schedule_cron_action( time(), '0 0 * * *', 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertValidAction( $action_id );
+		$action_id = as_schedule_cron_action(time(), '0 0 * * *', 'hook_1', array('a'), 'dummy', true);
+		$this->assertValidAction($action_id);
 
-		$action_id_duplicate = as_schedule_cron_action( time(), '0 0 * * *', 'hook_1', array( 'a' ), 'dummy', true );
-		$this->assertSame( 0, $action_id_duplicate );
+		$action_id_duplicate = as_schedule_cron_action(time(), '0 0 * * *', 'hook_1', array('a'), 'dummy', true);
+		$this->assertSame(0, $action_id_duplicate);
 	}
 
 	/**
@@ -356,11 +356,11 @@ 
 	 *
 	 * @param ActionScheduler_Store $store Store instance to set.
 	 */
-	private function set_action_scheduler_store( $store ) {
-		$store_factory_setter        = function () use ( $store ) {
+	private function set_action_scheduler_store($store) {
+		$store_factory_setter        = function() use ($store) {
 			self::$store = $store;
 		};
-		$binded_store_factory_setter = Closure::bind( $store_factory_setter, null, ActionScheduler_Store::class );
+		$binded_store_factory_setter = Closure::bind($store_factory_setter, null, ActionScheduler_Store::class);
 		$binded_store_factory_setter();
 	}
 
@@ -369,9 +369,9 @@ 
 	 *
 	 * @param int $action_id Action ID to assert.
 	 */
-	private function assertValidAction( $action_id ) {
-		$this->assertNotEquals( 0, $action_id );
-		$action = ActionScheduler::store()->fetch_action( $action_id );
-		$this->assertInstanceOf( 'ActionScheduler_Action', $action );
+	private function assertValidAction($action_id) {
+		$this->assertNotEquals(0, $action_id);
+		$action = ActionScheduler::store()->fetch_action($action_id);
+		$this->assertInstanceOf('ActionScheduler_Action', $action);
 	}
 }

src/libraries/action-scheduler/tests/phpunit/versioning/ActionScheduler_Versions_Test.php

Indentation +27 added lines, -27 removed lines

@@ -4,40 +4,40 @@
  * Class ActionScheduler_Versions_Test
  */
 class ActionScheduler_Versions_Test extends ActionScheduler_UnitTestCase {
-	public function test_register_version() {
-		$versions = new ActionScheduler_Versions();
-		$versions->register( '1.0-dev', 'callback_1_dot_0_dev' );
-		$versions->register( '1.0', 'callback_1_dot_0' );
+    public function test_register_version() {
+        $versions = new ActionScheduler_Versions();
+        $versions->register( '1.0-dev', 'callback_1_dot_0_dev' );
+        $versions->register( '1.0', 'callback_1_dot_0' );
 
-		$registered = $versions->get_versions();
+        $registered = $versions->get_versions();
 
-		$this->assertArrayHasKey( '1.0-dev', $registered );
-		$this->assertArrayHasKey( '1.0', $registered );
-		$this->assertCount( 2, $registered );
+        $this->assertArrayHasKey( '1.0-dev', $registered );
+        $this->assertArrayHasKey( '1.0', $registered );
+        $this->assertCount( 2, $registered );
 
-		$this->assertEquals( 'callback_1_dot_0_dev', $registered['1.0-dev'] );
-	}
+        $this->assertEquals( 'callback_1_dot_0_dev', $registered['1.0-dev'] );
+    }
 
-	public function test_duplicate_version() {
-		$versions = new ActionScheduler_Versions();
-		$versions->register( '1.0', 'callback_1_dot_0_a' );
-		$versions->register( '1.0', 'callback_1_dot_0_b' );
+    public function test_duplicate_version() {
+        $versions = new ActionScheduler_Versions();
+        $versions->register( '1.0', 'callback_1_dot_0_a' );
+        $versions->register( '1.0', 'callback_1_dot_0_b' );
 
-		$registered = $versions->get_versions();
+        $registered = $versions->get_versions();
 
-		$this->assertArrayHasKey( '1.0', $registered );
-		$this->assertCount( 1, $registered );
-	}
+        $this->assertArrayHasKey( '1.0', $registered );
+        $this->assertCount( 1, $registered );
+    }
 
-	public function test_latest_version() {
-		$versions = new ActionScheduler_Versions();
-		$this->assertEquals( '__return_null', $versions->latest_version_callback() );
-		$versions->register( '1.2', 'callback_1_dot_2' );
-		$versions->register( '1.3', 'callback_1_dot_3' );
-		$versions->register( '1.0', 'callback_1_dot_0' );
+    public function test_latest_version() {
+        $versions = new ActionScheduler_Versions();
+        $this->assertEquals( '__return_null', $versions->latest_version_callback() );
+        $versions->register( '1.2', 'callback_1_dot_2' );
+        $versions->register( '1.3', 'callback_1_dot_3' );
+        $versions->register( '1.0', 'callback_1_dot_0' );
 
-		$this->assertSame( '1.3', $versions->latest_version() );
-		$this->assertEquals( 'callback_1_dot_3', $versions->latest_version_callback() );
-	}
+        $this->assertSame( '1.3', $versions->latest_version() );
+        $this->assertEquals( 'callback_1_dot_3', $versions->latest_version_callback() );
+    }
 }
 

Spacing +16 added lines, -16 removed lines

@@ -6,38 +6,38 @@
 class ActionScheduler_Versions_Test extends ActionScheduler_UnitTestCase {
 	public function test_register_version() {
 		$versions = new ActionScheduler_Versions();
-		$versions->register( '1.0-dev', 'callback_1_dot_0_dev' );
-		$versions->register( '1.0', 'callback_1_dot_0' );
+		$versions->register('1.0-dev', 'callback_1_dot_0_dev');
+		$versions->register('1.0', 'callback_1_dot_0');
 
 		$registered = $versions->get_versions();
 
-		$this->assertArrayHasKey( '1.0-dev', $registered );
-		$this->assertArrayHasKey( '1.0', $registered );
-		$this->assertCount( 2, $registered );
+		$this->assertArrayHasKey('1.0-dev', $registered);
+		$this->assertArrayHasKey('1.0', $registered);
+		$this->assertCount(2, $registered);
 
-		$this->assertEquals( 'callback_1_dot_0_dev', $registered['1.0-dev'] );
+		$this->assertEquals('callback_1_dot_0_dev', $registered['1.0-dev']);
 	}
 
 	public function test_duplicate_version() {
 		$versions = new ActionScheduler_Versions();
-		$versions->register( '1.0', 'callback_1_dot_0_a' );
-		$versions->register( '1.0', 'callback_1_dot_0_b' );
+		$versions->register('1.0', 'callback_1_dot_0_a');
+		$versions->register('1.0', 'callback_1_dot_0_b');
 
 		$registered = $versions->get_versions();
 
-		$this->assertArrayHasKey( '1.0', $registered );
-		$this->assertCount( 1, $registered );
+		$this->assertArrayHasKey('1.0', $registered);
+		$this->assertCount(1, $registered);
 	}
 
 	public function test_latest_version() {
 		$versions = new ActionScheduler_Versions();
-		$this->assertEquals( '__return_null', $versions->latest_version_callback() );
-		$versions->register( '1.2', 'callback_1_dot_2' );
-		$versions->register( '1.3', 'callback_1_dot_3' );
-		$versions->register( '1.0', 'callback_1_dot_0' );
+		$this->assertEquals('__return_null', $versions->latest_version_callback());
+		$versions->register('1.2', 'callback_1_dot_2');
+		$versions->register('1.3', 'callback_1_dot_3');
+		$versions->register('1.0', 'callback_1_dot_0');
 
-		$this->assertSame( '1.3', $versions->latest_version() );
-		$this->assertEquals( 'callback_1_dot_3', $versions->latest_version_callback() );
+		$this->assertSame('1.3', $versions->latest_version());
+		$this->assertEquals('callback_1_dot_3', $versions->latest_version_callback());
 	}
 }
 

src/libraries/action-scheduler/tests/phpunit/logging/ActionScheduler_DBLogger_Test.php

Indentation +127 added lines, -127 removed lines

@@ -7,131 +7,131 @@
  * @group tables
  */
 class ActionScheduler_DBLogger_Test extends ActionScheduler_UnitTestCase {
-	public function test_default_logger() {
-		$logger = ActionScheduler::logger();
-		$this->assertInstanceOf( 'ActionScheduler_Logger', $logger );
-		$this->assertInstanceOf( ActionScheduler_DBLogger::class, $logger );
-	}
-
-	public function test_add_log_entry() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
-		$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() );
-	}
-
-	public function test_storage_logs() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
-		$logger    = ActionScheduler::logger();
-		$logs      = $logger->get_logs( $action_id );
-		$expected  = new ActionScheduler_LogEntry( $action_id, 'action created' );
-		$this->assertCount( 1, $logs );
-		$this->assertEquals( $expected->get_action_id(), $logs[0]->get_action_id() );
-		$this->assertEquals( $expected->get_message(), $logs[0]->get_message() );
-	}
-
-	public function test_execution_logs() {
-		$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' );
-
-		$runner = ActionScheduler_Mocker::get_queue_runner();
-		$runner->run( 'Unit Tests' );
-
-		// Expect 3 logs with the correct action ID.
-		$logs = $logger->get_logs( $action_id );
-		$this->assertCount( 3, $logs );
-		foreach ( $logs as $log ) {
-			$this->assertEquals( $action_id, $log->get_action_id() );
-		}
-
-		// Expect created, then started, then completed.
-		$this->assertEquals( 'action created', $logs[0]->get_message() );
-		$this->assertEquals( $started->get_message(), $logs[1]->get_message() );
-		$this->assertEquals( $finished->get_message(), $logs[2]->get_message() );
-	}
-
-	public function test_failed_execution_logs() {
-		$hook = __METHOD__;
-		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' );
-
-		$runner = ActionScheduler_Mocker::get_queue_runner();
-		$runner->run( 'Unit Tests' );
-
-		// Expect 3 logs with the correct action ID.
-		$logs = $logger->get_logs( $action_id );
-		$this->assertCount( 3, $logs );
-		foreach ( $logs as $log ) {
-			$this->assertEquals( $action_id, $log->get_action_id() );
-			$this->assertNotEquals( $finished->get_message(), $log->get_message() );
-		}
-
-		// Expect created, then started, then failed.
-		$this->assertEquals( 'action created', $logs[0]->get_message() );
-		$this->assertEquals( $started->get_message(), $logs[1]->get_message() );
-		$this->assertEquals( $failed->get_message(), $logs[2]->get_message() );
-	}
-
-	public function test_fatal_error_log() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
-		$logger    = ActionScheduler::logger();
-		do_action(
-			'action_scheduler_unexpected_shutdown',
-			$action_id,
-			array(
-				'type'    => E_ERROR,
-				'message' => 'Test error',
-				'file'    => __FILE__,
-				'line'    => __LINE__,
-			)
-		);
-
-		$logs      = $logger->get_logs( $action_id );
-		$found_log = false;
-		foreach ( $logs as $l ) {
-			if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) {
-				$found_log = true;
-			}
-		}
-		$this->assertTrue( $found_log, 'Unexpected shutdown log not found' );
-	}
-
-	public function test_canceled_action_log() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
-		as_unschedule_action( __METHOD__ );
-		$logger   = ActionScheduler::logger();
-		$logs     = $logger->get_logs( $action_id );
-		$expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' );
-		$this->assertEquals( $expected->get_message(), end( $logs )->get_message() );
-	}
-
-	public function test_deleted_action_cleanup() {
-		$time      = as_get_datetime_object( '-10 minutes' );
-		$schedule  = new \ActionScheduler_SimpleSchedule( $time );
-		$action    = new \ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule );
-		$store     = new ActionScheduler_DBStore();
-		$action_id = $store->save_action( $action );
-
-		$logger = new ActionScheduler_DBLogger();
-		$logs   = $logger->get_logs( $action_id );
-		$this->assertNotEmpty( $logs );
-
-		$store->delete_action( $action_id );
-		$logs = $logger->get_logs( $action_id );
-		$this->assertEmpty( $logs );
-	}
-
-	public function _a_hook_callback_that_throws_an_exception() {
-		throw new \RuntimeException( 'Execution failed' );
-	}
+    public function test_default_logger() {
+        $logger = ActionScheduler::logger();
+        $this->assertInstanceOf( 'ActionScheduler_Logger', $logger );
+        $this->assertInstanceOf( ActionScheduler_DBLogger::class, $logger );
+    }
+
+    public function test_add_log_entry() {
+        $action_id = as_schedule_single_action( time(), __METHOD__ );
+        $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() );
+    }
+
+    public function test_storage_logs() {
+        $action_id = as_schedule_single_action( time(), __METHOD__ );
+        $logger    = ActionScheduler::logger();
+        $logs      = $logger->get_logs( $action_id );
+        $expected  = new ActionScheduler_LogEntry( $action_id, 'action created' );
+        $this->assertCount( 1, $logs );
+        $this->assertEquals( $expected->get_action_id(), $logs[0]->get_action_id() );
+        $this->assertEquals( $expected->get_message(), $logs[0]->get_message() );
+    }
+
+    public function test_execution_logs() {
+        $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' );
+
+        $runner = ActionScheduler_Mocker::get_queue_runner();
+        $runner->run( 'Unit Tests' );
+
+        // Expect 3 logs with the correct action ID.
+        $logs = $logger->get_logs( $action_id );
+        $this->assertCount( 3, $logs );
+        foreach ( $logs as $log ) {
+            $this->assertEquals( $action_id, $log->get_action_id() );
+        }
+
+        // Expect created, then started, then completed.
+        $this->assertEquals( 'action created', $logs[0]->get_message() );
+        $this->assertEquals( $started->get_message(), $logs[1]->get_message() );
+        $this->assertEquals( $finished->get_message(), $logs[2]->get_message() );
+    }
+
+    public function test_failed_execution_logs() {
+        $hook = __METHOD__;
+        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' );
+
+        $runner = ActionScheduler_Mocker::get_queue_runner();
+        $runner->run( 'Unit Tests' );
+
+        // Expect 3 logs with the correct action ID.
+        $logs = $logger->get_logs( $action_id );
+        $this->assertCount( 3, $logs );
+        foreach ( $logs as $log ) {
+            $this->assertEquals( $action_id, $log->get_action_id() );
+            $this->assertNotEquals( $finished->get_message(), $log->get_message() );
+        }
+
+        // Expect created, then started, then failed.
+        $this->assertEquals( 'action created', $logs[0]->get_message() );
+        $this->assertEquals( $started->get_message(), $logs[1]->get_message() );
+        $this->assertEquals( $failed->get_message(), $logs[2]->get_message() );
+    }
+
+    public function test_fatal_error_log() {
+        $action_id = as_schedule_single_action( time(), __METHOD__ );
+        $logger    = ActionScheduler::logger();
+        do_action(
+            'action_scheduler_unexpected_shutdown',
+            $action_id,
+            array(
+                'type'    => E_ERROR,
+                'message' => 'Test error',
+                'file'    => __FILE__,
+                'line'    => __LINE__,
+            )
+        );
+
+        $logs      = $logger->get_logs( $action_id );
+        $found_log = false;
+        foreach ( $logs as $l ) {
+            if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) {
+                $found_log = true;
+            }
+        }
+        $this->assertTrue( $found_log, 'Unexpected shutdown log not found' );
+    }
+
+    public function test_canceled_action_log() {
+        $action_id = as_schedule_single_action( time(), __METHOD__ );
+        as_unschedule_action( __METHOD__ );
+        $logger   = ActionScheduler::logger();
+        $logs     = $logger->get_logs( $action_id );
+        $expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' );
+        $this->assertEquals( $expected->get_message(), end( $logs )->get_message() );
+    }
+
+    public function test_deleted_action_cleanup() {
+        $time      = as_get_datetime_object( '-10 minutes' );
+        $schedule  = new \ActionScheduler_SimpleSchedule( $time );
+        $action    = new \ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule );
+        $store     = new ActionScheduler_DBStore();
+        $action_id = $store->save_action( $action );
+
+        $logger = new ActionScheduler_DBLogger();
+        $logs   = $logger->get_logs( $action_id );
+        $this->assertNotEmpty( $logs );
+
+        $store->delete_action( $action_id );
+        $logs = $logger->get_logs( $action_id );
+        $this->assertEmpty( $logs );
+    }
+
+    public function _a_hook_callback_that_throws_an_exception() {
+        throw new \RuntimeException( 'Execution failed' );
+    }
 }

Spacing +58 added lines, -58 removed lines

@@ -9,81 +9,81 @@ 
 class ActionScheduler_DBLogger_Test extends ActionScheduler_UnitTestCase {
 	public function test_default_logger() {
 		$logger = ActionScheduler::logger();
-		$this->assertInstanceOf( 'ActionScheduler_Logger', $logger );
-		$this->assertInstanceOf( ActionScheduler_DBLogger::class, $logger );
+		$this->assertInstanceOf('ActionScheduler_Logger', $logger);
+		$this->assertInstanceOf(ActionScheduler_DBLogger::class, $logger);
 	}
 
 	public function test_add_log_entry() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
+		$action_id = as_schedule_single_action(time(), __METHOD__);
 		$logger    = ActionScheduler::logger();
 		$message   = 'Logging that something happened';
-		$log_id    = $logger->log( $action_id, $message );
-		$entry     = $logger->get_entry( $log_id );
+		$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() );
+		$this->assertEquals($action_id, $entry->get_action_id());
+		$this->assertEquals($message, $entry->get_message());
 	}
 
 	public function test_storage_logs() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
+		$action_id = as_schedule_single_action(time(), __METHOD__);
 		$logger    = ActionScheduler::logger();
-		$logs      = $logger->get_logs( $action_id );
-		$expected  = new ActionScheduler_LogEntry( $action_id, 'action created' );
-		$this->assertCount( 1, $logs );
-		$this->assertEquals( $expected->get_action_id(), $logs[0]->get_action_id() );
-		$this->assertEquals( $expected->get_message(), $logs[0]->get_message() );
+		$logs      = $logger->get_logs($action_id);
+		$expected  = new ActionScheduler_LogEntry($action_id, 'action created');
+		$this->assertCount(1, $logs);
+		$this->assertEquals($expected->get_action_id(), $logs[0]->get_action_id());
+		$this->assertEquals($expected->get_message(), $logs[0]->get_message());
 	}
 
 	public function test_execution_logs() {
-		$action_id = as_schedule_single_action( time(), ActionScheduler_Callbacks::HOOK_WITH_CALLBACK );
+		$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' );
+		$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' );
+		$runner->run('Unit Tests');
 
 		// Expect 3 logs with the correct action ID.
-		$logs = $logger->get_logs( $action_id );
-		$this->assertCount( 3, $logs );
-		foreach ( $logs as $log ) {
-			$this->assertEquals( $action_id, $log->get_action_id() );
+		$logs = $logger->get_logs($action_id);
+		$this->assertCount(3, $logs);
+		foreach ($logs as $log) {
+			$this->assertEquals($action_id, $log->get_action_id());
 		}
 
 		// Expect created, then started, then completed.
-		$this->assertEquals( 'action created', $logs[0]->get_message() );
-		$this->assertEquals( $started->get_message(), $logs[1]->get_message() );
-		$this->assertEquals( $finished->get_message(), $logs[2]->get_message() );
+		$this->assertEquals('action created', $logs[0]->get_message());
+		$this->assertEquals($started->get_message(), $logs[1]->get_message());
+		$this->assertEquals($finished->get_message(), $logs[2]->get_message());
 	}
 
 	public function test_failed_execution_logs() {
 		$hook = __METHOD__;
-		add_action( $hook, array( $this, '_a_hook_callback_that_throws_an_exception' ) );
-		$action_id = as_schedule_single_action( time(), $hook );
+		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' );
+		$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' );
+		$runner->run('Unit Tests');
 
 		// Expect 3 logs with the correct action ID.
-		$logs = $logger->get_logs( $action_id );
-		$this->assertCount( 3, $logs );
-		foreach ( $logs as $log ) {
-			$this->assertEquals( $action_id, $log->get_action_id() );
-			$this->assertNotEquals( $finished->get_message(), $log->get_message() );
+		$logs = $logger->get_logs($action_id);
+		$this->assertCount(3, $logs);
+		foreach ($logs as $log) {
+			$this->assertEquals($action_id, $log->get_action_id());
+			$this->assertNotEquals($finished->get_message(), $log->get_message());
 		}
 
 		// Expect created, then started, then failed.
-		$this->assertEquals( 'action created', $logs[0]->get_message() );
-		$this->assertEquals( $started->get_message(), $logs[1]->get_message() );
-		$this->assertEquals( $failed->get_message(), $logs[2]->get_message() );
+		$this->assertEquals('action created', $logs[0]->get_message());
+		$this->assertEquals($started->get_message(), $logs[1]->get_message());
+		$this->assertEquals($failed->get_message(), $logs[2]->get_message());
 	}
 
 	public function test_fatal_error_log() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
+		$action_id = as_schedule_single_action(time(), __METHOD__);
 		$logger    = ActionScheduler::logger();
 		do_action(
 			'action_scheduler_unexpected_shutdown',
@@ -96,42 +96,42 @@ 
 			)
 		);
 
-		$logs      = $logger->get_logs( $action_id );
+		$logs      = $logger->get_logs($action_id);
 		$found_log = false;
-		foreach ( $logs as $l ) {
-			if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) {
+		foreach ($logs as $l) {
+			if (strpos($l->get_message(), 'unexpected shutdown') === 0) {
 				$found_log = true;
 			}
 		}
-		$this->assertTrue( $found_log, 'Unexpected shutdown log not found' );
+		$this->assertTrue($found_log, 'Unexpected shutdown log not found');
 	}
 
 	public function test_canceled_action_log() {
-		$action_id = as_schedule_single_action( time(), __METHOD__ );
-		as_unschedule_action( __METHOD__ );
+		$action_id = as_schedule_single_action(time(), __METHOD__);
+		as_unschedule_action(__METHOD__);
 		$logger   = ActionScheduler::logger();
-		$logs     = $logger->get_logs( $action_id );
-		$expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' );
-		$this->assertEquals( $expected->get_message(), end( $logs )->get_message() );
+		$logs     = $logger->get_logs($action_id);
+		$expected = new ActionScheduler_LogEntry($action_id, 'action canceled');
+		$this->assertEquals($expected->get_message(), end($logs)->get_message());
 	}
 
 	public function test_deleted_action_cleanup() {
-		$time      = as_get_datetime_object( '-10 minutes' );
-		$schedule  = new \ActionScheduler_SimpleSchedule( $time );
-		$action    = new \ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule );
+		$time      = as_get_datetime_object('-10 minutes');
+		$schedule  = new \ActionScheduler_SimpleSchedule($time);
+		$action    = new \ActionScheduler_Action(ActionScheduler_Callbacks::HOOK_WITH_CALLBACK, array(), $schedule);
 		$store     = new ActionScheduler_DBStore();
-		$action_id = $store->save_action( $action );
+		$action_id = $store->save_action($action);
 
 		$logger = new ActionScheduler_DBLogger();
-		$logs   = $logger->get_logs( $action_id );
-		$this->assertNotEmpty( $logs );
+		$logs   = $logger->get_logs($action_id);
+		$this->assertNotEmpty($logs);
 
-		$store->delete_action( $action_id );
-		$logs = $logger->get_logs( $action_id );
-		$this->assertEmpty( $logs );
+		$store->delete_action($action_id);
+		$logs = $logger->get_logs($action_id);
+		$this->assertEmpty($logs);
 	}
 
 	public function _a_hook_callback_that_throws_an_exception() {
-		throw new \RuntimeException( 'Execution failed' );
+		throw new \RuntimeException('Execution failed');
 	}
 }

src/libraries/action-scheduler/tests/phpunit/logging/ActionScheduler_wpCommentLogger_Test.php

Indentation +219 added lines, -219 removed lines

@@ -6,224 +6,224 @@
  * @package test_cases\logging
  */
 class ActionScheduler_wpCommentLogger_Test extends ActionScheduler_UnitTestCase {
-	private $use_comment_logger;
-
-	public function test_default_logger() {
-		$logger = ActionScheduler::logger();
-		$this->assertInstanceOf( 'ActionScheduler_Logger', $logger );
-		if ( $this->using_comment_logger() ) {
-			$this->assertInstanceOf( 'ActionScheduler_wpCommentLogger', $logger );
-		} else {
-			$this->assertNotInstanceOf( 'ActionScheduler_wpCommentLogger', $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 );
-
-		$this->assertEquals( $action_id, $entry->get_action_id() );
-		$this->assertEquals( $message, $entry->get_message() );
-	}
-
-	public function test_add_log_datetime() {
-		$action_id = as_schedule_single_action( time(), 'a hook' );
-		$logger    = ActionScheduler::logger();
-		$message   = 'Logging that something happened';
-		$date      = new 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() );
-
-		$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',
-			)
-		);
-		$logger  = ActionScheduler::logger();
-		$entry   = $logger->get_entry( $comment );
-		$this->assertSame( '', $entry->get_action_id() );
-		$this->assertSame( '', $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->assertContains( $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(),
-			);
-		}
-
-		foreach ( $logs as $id => $log ) {
-			$logs[ $id ] = array(
-				'action_id' => $log->get_action_id(),
-				'message'   => $log->get_message(),
-			);
-		}
-
-		return $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' );
-
-		$runner = ActionScheduler_Mocker::get_queue_runner();
-		$runner->run( 'Unit Tests' );
-
-		$logs = $logger->get_logs( $action_id );
-		$this->assertContains( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) );
-		$this->assertContains( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) );
-	}
-
-	public function test_failed_execution_comments() {
-		$hook = md5( 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' );
-
-		$runner = ActionScheduler_Mocker::get_queue_runner();
-		$runner->run( 'Unit Tests' );
-
-		$logs = $logger->get_logs( $action_id );
-		$this->assertContains( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) );
-		$this->assertNotContains( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) );
-		$this->assertContains( $this->log_entry_to_array( $failed ), $this->log_entry_to_array( $logs ) );
-	}
-
-	public function test_failed_schedule_next_instance_comments() {
-		$action_id = 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();
-		} 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 );
-		$this->assertContains( $this->log_entry_to_array( $log_entry ), $this->log_entry_to_array( $logs ) );
-	}
-
-	public function test_fatal_error_comments() {
-		$hook      = md5( rand() );
-		$action_id = as_schedule_single_action( time(), $hook );
-		$logger    = ActionScheduler::logger();
-		do_action(
-			'action_scheduler_unexpected_shutdown',
-			$action_id,
-			array(
-				'type'    => E_ERROR,
-				'message' => 'Test error',
-				'file'    => __FILE__,
-				'line'    => __LINE__,
-			)
-		);
-
-		$logs      = $logger->get_logs( $action_id );
-		$found_log = false;
-		foreach ( $logs as $l ) {
-			if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) {
-				$found_log = true;
-			}
-		}
-		$this->assertTrue( $found_log, 'Unexpected shutdown log not found' );
-	}
-
-	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 );
-		$expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' );
-		$this->assertContains( $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 test_filtering_of_get_comments() {
-		if ( ! $this->using_comment_logger() ) {
-			$this->assertTrue( true );
-			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__,
-			)
-		);
-
-		// 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 );
-
-		// 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
-		$this->assertCount( 2, $comments );
-		$this->assertContains( $log_id, wp_list_pluck( $comments, 'comment_ID' ) );
-	}
-
-	private function using_comment_logger() {
-		if ( null === $this->use_comment_logger ) {
-			$this->use_comment_logger = ! ActionScheduler_DataController::dependencies_met();
-		}
-
-		return $this->use_comment_logger;
-	}
+    private $use_comment_logger;
+
+    public function test_default_logger() {
+        $logger = ActionScheduler::logger();
+        $this->assertInstanceOf( 'ActionScheduler_Logger', $logger );
+        if ( $this->using_comment_logger() ) {
+            $this->assertInstanceOf( 'ActionScheduler_wpCommentLogger', $logger );
+        } else {
+            $this->assertNotInstanceOf( 'ActionScheduler_wpCommentLogger', $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 );
+
+        $this->assertEquals( $action_id, $entry->get_action_id() );
+        $this->assertEquals( $message, $entry->get_message() );
+    }
+
+    public function test_add_log_datetime() {
+        $action_id = as_schedule_single_action( time(), 'a hook' );
+        $logger    = ActionScheduler::logger();
+        $message   = 'Logging that something happened';
+        $date      = new 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() );
+
+        $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',
+            )
+        );
+        $logger  = ActionScheduler::logger();
+        $entry   = $logger->get_entry( $comment );
+        $this->assertSame( '', $entry->get_action_id() );
+        $this->assertSame( '', $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->assertContains( $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(),
+            );
+        }
+
+        foreach ( $logs as $id => $log ) {
+            $logs[ $id ] = array(
+                'action_id' => $log->get_action_id(),
+                'message'   => $log->get_message(),
+            );
+        }
+
+        return $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' );
+
+        $runner = ActionScheduler_Mocker::get_queue_runner();
+        $runner->run( 'Unit Tests' );
+
+        $logs = $logger->get_logs( $action_id );
+        $this->assertContains( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) );
+        $this->assertContains( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) );
+    }
+
+    public function test_failed_execution_comments() {
+        $hook = md5( 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' );
+
+        $runner = ActionScheduler_Mocker::get_queue_runner();
+        $runner->run( 'Unit Tests' );
+
+        $logs = $logger->get_logs( $action_id );
+        $this->assertContains( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) );
+        $this->assertNotContains( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) );
+        $this->assertContains( $this->log_entry_to_array( $failed ), $this->log_entry_to_array( $logs ) );
+    }
+
+    public function test_failed_schedule_next_instance_comments() {
+        $action_id = 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();
+        } 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 );
+        $this->assertContains( $this->log_entry_to_array( $log_entry ), $this->log_entry_to_array( $logs ) );
+    }
+
+    public function test_fatal_error_comments() {
+        $hook      = md5( rand() );
+        $action_id = as_schedule_single_action( time(), $hook );
+        $logger    = ActionScheduler::logger();
+        do_action(
+            'action_scheduler_unexpected_shutdown',
+            $action_id,
+            array(
+                'type'    => E_ERROR,
+                'message' => 'Test error',
+                'file'    => __FILE__,
+                'line'    => __LINE__,
+            )
+        );
+
+        $logs      = $logger->get_logs( $action_id );
+        $found_log = false;
+        foreach ( $logs as $l ) {
+            if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) {
+                $found_log = true;
+            }
+        }
+        $this->assertTrue( $found_log, 'Unexpected shutdown log not found' );
+    }
+
+    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 );
+        $expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' );
+        $this->assertContains( $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 test_filtering_of_get_comments() {
+        if ( ! $this->using_comment_logger() ) {
+            $this->assertTrue( true );
+            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__,
+            )
+        );
+
+        // 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 );
+
+        // 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
+        $this->assertCount( 2, $comments );
+        $this->assertContains( $log_id, wp_list_pluck( $comments, 'comment_ID' ) );
+    }
+
+    private function using_comment_logger() {
+        if ( null === $this->use_comment_logger ) {
+            $this->use_comment_logger = ! ActionScheduler_DataController::dependencies_met();
+        }
+
+        return $this->use_comment_logger;
+    }
 }
 

Spacing +78 added lines, -78 removed lines

@@ -10,42 +10,42 @@ 
 
 	public function test_default_logger() {
 		$logger = ActionScheduler::logger();
-		$this->assertInstanceOf( 'ActionScheduler_Logger', $logger );
-		if ( $this->using_comment_logger() ) {
-			$this->assertInstanceOf( 'ActionScheduler_wpCommentLogger', $logger );
+		$this->assertInstanceOf('ActionScheduler_Logger', $logger);
+		if ($this->using_comment_logger()) {
+			$this->assertInstanceOf('ActionScheduler_wpCommentLogger', $logger);
 		} else {
-			$this->assertNotInstanceOf( 'ActionScheduler_wpCommentLogger', $logger );
+			$this->assertNotInstanceOf('ActionScheduler_wpCommentLogger', $logger);
 		}
 	}
 
 	public function test_add_log_entry() {
-		$action_id = as_schedule_single_action( time(), 'a hook' );
+		$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 );
+		$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() );
+		$this->assertEquals($action_id, $entry->get_action_id());
+		$this->assertEquals($message, $entry->get_message());
 	}
 
 	public function test_add_log_datetime() {
-		$action_id = as_schedule_single_action( time(), 'a hook' );
+		$action_id = as_schedule_single_action(time(), 'a hook');
 		$logger    = ActionScheduler::logger();
 		$message   = 'Logging that something happened';
-		$date      = new DateTime( 'now', new DateTimeZone( 'UTC' ) );
-		$log_id    = $logger->log( $action_id, $message, $date );
-		$entry     = $logger->get_entry( $log_id );
+		$date      = new 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() );
+		$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() );
+		$this->assertEquals($action_id, $entry->get_action_id());
+		$this->assertEquals($message, $entry->get_message());
 	}
 
 	public function test_erroneous_entry_id() {
@@ -57,29 +57,29 @@ 
 			)
 		);
 		$logger  = ActionScheduler::logger();
-		$entry   = $logger->get_entry( $comment );
-		$this->assertSame( '', $entry->get_action_id() );
-		$this->assertSame( '', $entry->get_message() );
+		$entry   = $logger->get_entry($comment);
+		$this->assertSame('', $entry->get_action_id());
+		$this->assertSame('', $entry->get_message());
 	}
 
 	public function test_storage_comments() {
-		$action_id = as_schedule_single_action( time(), 'a hook' );
+		$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->assertContains( $this->log_entry_to_array( $expected ), $this->log_entry_to_array( $logs ) );
+		$logs      = $logger->get_logs($action_id);
+		$expected  = new ActionScheduler_LogEntry($action_id, 'action created');
+		$this->assertContains($this->log_entry_to_array($expected), $this->log_entry_to_array($logs));
 	}
 
-	protected function log_entry_to_array( $logs ) {
-		if ( $logs instanceof ActionScheduler_LogEntry ) {
+	protected function log_entry_to_array($logs) {
+		if ($logs instanceof ActionScheduler_LogEntry) {
 			return array(
 				'action_id' => $logs->get_action_id(),
 				'message'   => $logs->get_message(),
 			);
 		}
 
-		foreach ( $logs as $id => $log ) {
-			$logs[ $id ] = array(
+		foreach ($logs as $id => $log) {
+			$logs[$id] = array(
 				'action_id' => $log->get_action_id(),
 				'message'   => $log->get_message(),
 			);
@@ -89,55 +89,55 @@ 
 	}
 
 	public function test_execution_comments() {
-		$action_id = as_schedule_single_action( time(), ActionScheduler_Callbacks::HOOK_WITH_CALLBACK );
+		$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' );
+		$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' );
+		$runner->run('Unit Tests');
 
-		$logs = $logger->get_logs( $action_id );
-		$this->assertContains( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) );
-		$this->assertContains( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) );
+		$logs = $logger->get_logs($action_id);
+		$this->assertContains($this->log_entry_to_array($started), $this->log_entry_to_array($logs));
+		$this->assertContains($this->log_entry_to_array($finished), $this->log_entry_to_array($logs));
 	}
 
 	public function test_failed_execution_comments() {
-		$hook = md5( rand() );
-		add_action( $hook, array( $this, '_a_hook_callback_that_throws_an_exception' ) );
-		$action_id = as_schedule_single_action( time(), $hook );
+		$hook = md5(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' );
+		$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' );
+		$runner->run('Unit Tests');
 
-		$logs = $logger->get_logs( $action_id );
-		$this->assertContains( $this->log_entry_to_array( $started ), $this->log_entry_to_array( $logs ) );
-		$this->assertNotContains( $this->log_entry_to_array( $finished ), $this->log_entry_to_array( $logs ) );
-		$this->assertContains( $this->log_entry_to_array( $failed ), $this->log_entry_to_array( $logs ) );
+		$logs = $logger->get_logs($action_id);
+		$this->assertContains($this->log_entry_to_array($started), $this->log_entry_to_array($logs));
+		$this->assertNotContains($this->log_entry_to_array($finished), $this->log_entry_to_array($logs));
+		$this->assertContains($this->log_entry_to_array($failed), $this->log_entry_to_array($logs));
 	}
 
 	public function test_failed_schedule_next_instance_comments() {
 		$action_id = rand();
 		$logger    = ActionScheduler::logger();
-		$log_entry = new ActionScheduler_LogEntry( $action_id, 'There was a failure scheduling the next instance of this action: Execution failed' );
+		$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();
-		} catch ( Exception $e ) {
-			do_action( 'action_scheduler_failed_to_schedule_next_instance', $action_id, $e, new ActionScheduler_Action( ActionScheduler_Callbacks::HOOK_WITH_CALLBACK ) );
+		} 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 );
-		$this->assertContains( $this->log_entry_to_array( $log_entry ), $this->log_entry_to_array( $logs ) );
+		$logs = $logger->get_logs($action_id);
+		$this->assertContains($this->log_entry_to_array($log_entry), $this->log_entry_to_array($logs));
 	}
 
 	public function test_fatal_error_comments() {
-		$hook      = md5( rand() );
-		$action_id = as_schedule_single_action( time(), $hook );
+		$hook      = md5(rand());
+		$action_id = as_schedule_single_action(time(), $hook);
 		$logger    = ActionScheduler::logger();
 		do_action(
 			'action_scheduler_unexpected_shutdown',
@@ -150,36 +150,36 @@ 
 			)
 		);
 
-		$logs      = $logger->get_logs( $action_id );
+		$logs      = $logger->get_logs($action_id);
 		$found_log = false;
-		foreach ( $logs as $l ) {
-			if ( strpos( $l->get_message(), 'unexpected shutdown' ) === 0 ) {
+		foreach ($logs as $l) {
+			if (strpos($l->get_message(), 'unexpected shutdown') === 0) {
 				$found_log = true;
 			}
 		}
-		$this->assertTrue( $found_log, 'Unexpected shutdown log not found' );
+		$this->assertTrue($found_log, 'Unexpected shutdown log not found');
 	}
 
 	public function test_canceled_action_comments() {
-		$action_id = as_schedule_single_action( time(), 'a hook' );
-		as_unschedule_action( 'a hook' );
+		$action_id = as_schedule_single_action(time(), 'a hook');
+		as_unschedule_action('a hook');
 		$logger   = ActionScheduler::logger();
-		$logs     = $logger->get_logs( $action_id );
-		$expected = new ActionScheduler_LogEntry( $action_id, 'action canceled' );
-		$this->assertContains( $this->log_entry_to_array( $expected ), $this->log_entry_to_array( $logs ) );
+		$logs     = $logger->get_logs($action_id);
+		$expected = new ActionScheduler_LogEntry($action_id, 'action canceled');
+		$this->assertContains($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' );
+		throw new RuntimeException('Execution failed');
 	}
 
 	public function test_filtering_of_get_comments() {
-		if ( ! $this->using_comment_logger() ) {
-			$this->assertTrue( true );
+		if ( ! $this->using_comment_logger()) {
+			$this->assertTrue(true);
 			return;
 		}
 
-		$post_id    = $this->factory->post->create_object(
+		$post_id = $this->factory->post->create_object(
 			array(
 				'post_title' => __FUNCTION__,
 			)
@@ -194,18 +194,18 @@ 
 
 		// 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 );
+		$this->assertCount(1, $comments);
+		$this->assertEquals($comment_id, $comments[0]->comment_ID);
 
-		$action_id = as_schedule_single_action( time(), 'a hook' );
+		$action_id = as_schedule_single_action(time(), 'a hook');
 		$logger    = ActionScheduler::logger();
 		$message   = 'Logging that something happened';
-		$log_id    = $logger->log( $action_id, $message );
+		$log_id    = $logger->log($action_id, $message);
 
 		// 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 );
+		$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(
@@ -214,12 +214,12 @@ 
 			)
 		);
 		// 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->assertCount(2, $comments);
+		$this->assertContains($log_id, wp_list_pluck($comments, 'comment_ID'));
 	}
 
 	private function using_comment_logger() {
-		if ( null === $this->use_comment_logger ) {
+		if (null === $this->use_comment_logger) {
 			$this->use_comment_logger = ! ActionScheduler_DataController::dependencies_met();
 		}
 

src/libraries/action-scheduler/tests/bootstrap.php

Indentation +6 added lines, -6 removed lines

@@ -6,13 +6,13 @@ 
 
 // Check for select constants defined as environment variables
 foreach ( array( 'WP_CONTENT_DIR', 'WP_CONTENT_URL', 'WP_PLUGIN_DIR', 'WP_PLUGIN_URL', 'WPMU_PLUGIN_DIR' ) as $env_constant ) {
-	if ( false !== getenv( $env_constant ) && ! defined( $env_constant ) ) {
-		define( $env_constant, getenv( $env_constant ) );
-	}
+    if ( false !== getenv( $env_constant ) && ! defined( $env_constant ) ) {
+        define( $env_constant, getenv( $env_constant ) );
+    }
 }
 
 if ( ! defined( 'WP_PLUGIN_DIR' ) ) {
-	define( 'WP_PLUGIN_DIR', dirname( dirname( __DIR__ ) ) );
+    define( 'WP_PLUGIN_DIR', dirname( dirname( __DIR__ ) ) );
 }
 
 // PHPUnit polyfills as required by the core WP test framework.
@@ -25,9 +25,9 @@ 
 require_once dirname( __DIR__ ) . '/action-scheduler.php';
 
 if ( class_exists( 'PHPUnit\Framework\TestResult' ) ) { // PHPUnit 6.0 or newer
-	include_once('ActionScheduler_UnitTestCase.php';
+    include_once('ActionScheduler_UnitTestCase.php';
 } else {
-	include_once('phpunit/deprecated/ActionScheduler_UnitTestCase.php';
+    include_once('phpunit/deprecated/ActionScheduler_UnitTestCase.php';
 }
 
 require_once 'phpunit/helpers/ActionScheduler_Callbacks.php';

Spacing +12 added lines, -12 removed lines

@@ -2,29 +2,29 @@
 
 $GLOBALS['wp_tests_options']['template']         = 'twentyseventeen';
 $GLOBALS['wp_tests_options']['stylesheet']       = 'twentyseventeen';
-$GLOBALS['wp_tests_options']['active_plugins'][] = basename( dirname( __DIR__ ) ) . '/action-scheduler.php';
+$GLOBALS['wp_tests_options']['active_plugins'][] = basename(dirname(__DIR__)).'/action-scheduler.php';
 
 // Check for select constants defined as environment variables
-foreach ( array( 'WP_CONTENT_DIR', 'WP_CONTENT_URL', 'WP_PLUGIN_DIR', 'WP_PLUGIN_URL', 'WPMU_PLUGIN_DIR' ) as $env_constant ) {
-	if ( false !== getenv( $env_constant ) && ! defined( $env_constant ) ) {
-		define( $env_constant, getenv( $env_constant ) );
+foreach (array('WP_CONTENT_DIR', 'WP_CONTENT_URL', 'WP_PLUGIN_DIR', 'WP_PLUGIN_URL', 'WPMU_PLUGIN_DIR') as $env_constant) {
+	if (false !== getenv($env_constant) && ! defined($env_constant)) {
+		define($env_constant, getenv($env_constant));
 	}
 }
 
-if ( ! defined( 'WP_PLUGIN_DIR' ) ) {
-	define( 'WP_PLUGIN_DIR', dirname( dirname( __DIR__ ) ) );
+if ( ! defined('WP_PLUGIN_DIR')) {
+	define('WP_PLUGIN_DIR', dirname(dirname(__DIR__)));
 }
 
 // PHPUnit polyfills as required by the core WP test framework.
-require_once __DIR__ . '/../vendor/yoast/phpunit-polyfills/phpunitpolyfills-autoload.php';
+require_once __DIR__.'/../vendor/yoast/phpunit-polyfills/phpunitpolyfills-autoload.php';
 
-$wordpress_tests_dir = getenv( 'WP_TESTS_DIR' ) ? getenv( 'WP_TESTS_DIR' ) : sys_get_temp_dir() . '/wordpress-tests-lib';
-require_once $wordpress_tests_dir . '/includes/functions.php';
-require $wordpress_tests_dir . '/includes/bootstrap.php';
+$wordpress_tests_dir = getenv('WP_TESTS_DIR') ? getenv('WP_TESTS_DIR') : sys_get_temp_dir().'/wordpress-tests-lib';
+require_once $wordpress_tests_dir.'/includes/functions.php';
+require $wordpress_tests_dir.'/includes/bootstrap.php';
 
-require_once dirname( __DIR__ ) . '/action-scheduler.php';
+require_once dirname(__DIR__).'/action-scheduler.php';
 
-if ( class_exists( 'PHPUnit\Framework\TestResult' ) ) { // PHPUnit 6.0 or newer
+if (class_exists('PHPUnit\Framework\TestResult')) { // PHPUnit 6.0 or newer
 	include_once('ActionScheduler_UnitTestCase.php';
 } else {
 	include_once('phpunit/deprecated/ActionScheduler_UnitTestCase.php';

src/libraries/action-scheduler/tests/ActionScheduler_UnitTestCase.php

Indentation +52 added lines, -52 removed lines

@@ -5,56 +5,56 @@
  */
 class ActionScheduler_UnitTestCase extends WP_UnitTestCase {
 
-	protected $existing_timezone;
-
-	/**
-	 * Perform test set-up work.
-	 */
-	public function set_up() {
-		ActionScheduler_Callbacks::add_callbacks();
-		parent::set_up();
-	}
-
-	/**
-	 * Perform test tear-down work.
-	 */
-	public function tear_down() {
-		ActionScheduler_Callbacks::remove_callbacks();
-		parent::tear_down();
-	}
-
-	/**
-	 * Counts the number of test cases executed by run(TestResult result).
-	 *
-	 * @return int
-	 */
-	public function count(): int {
-		return 'UTC' == date_default_timezone_get() ? 2 : 3;
-	}
-
-	/**
-	 * We want to run every test multiple times using a different timezone to make sure
-	 * that they are unaffected by changes to PHP's timezone.
-	 */
-	public function run( PHPUnit\Framework\TestResult $result = null ): \PHPUnit\Framework\TestResult {
-
-		if ( $result === null ) {
-			$result = $this->createResult();
-		}
-
-		if ( 'UTC' != ( $this->existing_timezone = date_default_timezone_get() ) ) {
-			date_default_timezone_set( 'UTC' );
-			$result->run( $this );
-		}
-
-		date_default_timezone_set( 'Pacific/Fiji' ); // UTC+12
-		$result->run( $this );
-
-		date_default_timezone_set( 'Pacific/Tahiti' ); // UTC-10: it's a magical place
-		$result->run( $this );
-
-		date_default_timezone_set( $this->existing_timezone );
-
-		return $result;
-	}
+    protected $existing_timezone;
+
+    /**
+     * Perform test set-up work.
+     */
+    public function set_up() {
+        ActionScheduler_Callbacks::add_callbacks();
+        parent::set_up();
+    }
+
+    /**
+     * Perform test tear-down work.
+     */
+    public function tear_down() {
+        ActionScheduler_Callbacks::remove_callbacks();
+        parent::tear_down();
+    }
+
+    /**
+     * Counts the number of test cases executed by run(TestResult result).
+     *
+     * @return int
+     */
+    public function count(): int {
+        return 'UTC' == date_default_timezone_get() ? 2 : 3;
+    }
+
+    /**
+     * We want to run every test multiple times using a different timezone to make sure
+     * that they are unaffected by changes to PHP's timezone.
+     */
+    public function run( PHPUnit\Framework\TestResult $result = null ): \PHPUnit\Framework\TestResult {
+
+        if ( $result === null ) {
+            $result = $this->createResult();
+        }
+
+        if ( 'UTC' != ( $this->existing_timezone = date_default_timezone_get() ) ) {
+            date_default_timezone_set( 'UTC' );
+            $result->run( $this );
+        }
+
+        date_default_timezone_set( 'Pacific/Fiji' ); // UTC+12
+        $result->run( $this );
+
+        date_default_timezone_set( 'Pacific/Tahiti' ); // UTC-10: it's a magical place
+        $result->run( $this );
+
+        date_default_timezone_set( $this->existing_timezone );
+
+        return $result;
+    }
 }

Spacing +10 added lines, -10 removed lines

@@ -36,24 +36,24 @@
 	 * We want to run every test multiple times using a different timezone to make sure
 	 * that they are unaffected by changes to PHP's timezone.
 	 */
-	public function run( PHPUnit\Framework\TestResult $result = null ): \PHPUnit\Framework\TestResult {
+	public function run(PHPUnit\Framework\TestResult $result = null): \PHPUnit\Framework\TestResult {
 
-		if ( $result === null ) {
+		if ($result === null) {
 			$result = $this->createResult();
 		}
 
-		if ( 'UTC' != ( $this->existing_timezone = date_default_timezone_get() ) ) {
-			date_default_timezone_set( 'UTC' );
-			$result->run( $this );
+		if ('UTC' != ($this->existing_timezone = date_default_timezone_get())) {
+			date_default_timezone_set('UTC');
+			$result->run($this);
 		}
 
-		date_default_timezone_set( 'Pacific/Fiji' ); // UTC+12
-		$result->run( $this );
+		date_default_timezone_set('Pacific/Fiji'); // UTC+12
+		$result->run($this);
 
-		date_default_timezone_set( 'Pacific/Tahiti' ); // UTC-10: it's a magical place
-		$result->run( $this );
+		date_default_timezone_set('Pacific/Tahiti'); // UTC-10: it's a magical place
+		$result->run($this);
 
-		date_default_timezone_set( $this->existing_timezone );
+		date_default_timezone_set($this->existing_timezone);
 
 		return $result;
 	}

src/libraries/action-scheduler/deprecated/functions.php

Indentation +12 added lines, -12 removed lines

@@ -19,8 +19,8 @@ 
  * @return string The job ID
  */
 function wc_schedule_single_action( $timestamp, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_single_action()' );
-	return as_schedule_single_action( $timestamp, $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_single_action()' );
+    return as_schedule_single_action( $timestamp, $hook, $args, $group );
 }
 
 /**
@@ -37,8 +37,8 @@ 
  * @return string The job ID
  */
 function wc_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_recurring_action()' );
-	return as_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_recurring_action()' );
+    return as_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args, $group );
 }
 
 /**
@@ -65,8 +65,8 @@ 
  * @return string The job ID
  */
 function wc_schedule_cron_action( $timestamp, $schedule, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_cron_action()' );
-	return as_schedule_cron_action( $timestamp, $schedule, $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_cron_action()' );
+    return as_schedule_cron_action( $timestamp, $schedule, $hook, $args, $group );
 }
 
 /**
@@ -79,8 +79,8 @@ 
  * @deprecated 2.1.0
  */
 function wc_unschedule_action( $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_unschedule_action()' );
-	as_unschedule_action( $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_unschedule_action()' );
+    as_unschedule_action( $hook, $args, $group );
 }
 
 /**
@@ -93,8 +93,8 @@ 
  * @return int|bool The timestamp for the next occurrence, or false if nothing was found
  */
 function wc_next_scheduled_action( $hook, $args = null, $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_next_scheduled_action()' );
-	return as_next_scheduled_action( $hook, $args, $group );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_next_scheduled_action()' );
+    return as_next_scheduled_action( $hook, $args, $group );
 }
 
 /**
@@ -121,6 +121,6 @@ 
  * @return array
  */
 function wc_get_scheduled_actions( $args = array(), $return_format = OBJECT ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_get_scheduled_actions()' );
-	return as_get_scheduled_actions( $args, $return_format );
+    _deprecated_function( __FUNCTION__, '2.1.0', 'as_get_scheduled_actions()' );
+    return as_get_scheduled_actions( $args, $return_format );
 }

Spacing +18 added lines, -18 removed lines

@@ -18,9 +18,9 @@ 
  *
  * @return string The job ID
  */
-function wc_schedule_single_action( $timestamp, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_single_action()' );
-	return as_schedule_single_action( $timestamp, $hook, $args, $group );
+function wc_schedule_single_action($timestamp, $hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_schedule_single_action()');
+	return as_schedule_single_action($timestamp, $hook, $args, $group);
 }
 
 /**
@@ -36,9 +36,9 @@ 
  *
  * @return string The job ID
  */
-function wc_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_recurring_action()' );
-	return as_schedule_recurring_action( $timestamp, $interval_in_seconds, $hook, $args, $group );
+function wc_schedule_recurring_action($timestamp, $interval_in_seconds, $hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_schedule_recurring_action()');
+	return as_schedule_recurring_action($timestamp, $interval_in_seconds, $hook, $args, $group);
 }
 
 /**
@@ -64,9 +64,9 @@ 
  *
  * @return string The job ID
  */
-function wc_schedule_cron_action( $timestamp, $schedule, $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_schedule_cron_action()' );
-	return as_schedule_cron_action( $timestamp, $schedule, $hook, $args, $group );
+function wc_schedule_cron_action($timestamp, $schedule, $hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_schedule_cron_action()');
+	return as_schedule_cron_action($timestamp, $schedule, $hook, $args, $group);
 }
 
 /**
@@ -78,9 +78,9 @@ 
  *
  * @deprecated 2.1.0
  */
-function wc_unschedule_action( $hook, $args = array(), $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_unschedule_action()' );
-	as_unschedule_action( $hook, $args, $group );
+function wc_unschedule_action($hook, $args = array(), $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_unschedule_action()');
+	as_unschedule_action($hook, $args, $group);
 }
 
 /**
@@ -92,9 +92,9 @@ 
  *
  * @return int|bool The timestamp for the next occurrence, or false if nothing was found
  */
-function wc_next_scheduled_action( $hook, $args = null, $group = '' ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_next_scheduled_action()' );
-	return as_next_scheduled_action( $hook, $args, $group );
+function wc_next_scheduled_action($hook, $args = null, $group = '') {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_next_scheduled_action()');
+	return as_next_scheduled_action($hook, $args, $group);
 }
 
 /**
@@ -120,7 +120,7 @@ 
  *
  * @return array
  */
-function wc_get_scheduled_actions( $args = array(), $return_format = OBJECT ) {
-	_deprecated_function( __FUNCTION__, '2.1.0', 'as_get_scheduled_actions()' );
-	return as_get_scheduled_actions( $args, $return_format );
+function wc_get_scheduled_actions($args = array(), $return_format = OBJECT) {
+	_deprecated_function(__FUNCTION__, '2.1.0', 'as_get_scheduled_actions()');
+	return as_get_scheduled_actions($args, $return_format);
 }