WordPoints / importer

tests/phpunit/tests/importer.php

Indentation +289 added lines, -289 removed lines

@@ -16,295 +16,295 @@
  */
 class WordPoints_Importer_Importer_Test extends WordPoints_PHPUnit_TestCase {
 
-	/**
-	 * The mock importer used in the tests.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var WordPoints_Importer_Mock
-	 */
-	protected $importer;
-
-	/**
-	 * The components assigned to the mock importer.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var array[]
-	 */
-	protected $importer_components;
-
-	/**
-	 * @since 1.0.0
-	 */
-	public function setUp() {
-
-		parent::setUp();
-
-		$this->importer = new WordPoints_Importer_Mock( 'Mock' );
-		$this->importer->components = array(
-			'points' => array(
-				'user_points' => array(
-					'label' => 'User points',
-					'function' => array( $this->importer, 'do_an_import' ),
-					'can_import' => array( $this->importer, 'can_import' ),
-				),
-			),
-		);
-
-		$this->importer_components = $this->importer->components;
-
-		remove_action(
-			'wordpoints_import_settings_valid-points'
-			, 'wordpoints_importer_validate_points_type_setting'
-		);
-	}
-
-	/**
-	 * Test that it returns true when a component is supported.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::supports_component
-	 */
-	public function test_supports_supported_component() {
-
-		$this->assertTrue( $this->importer->supports_component( 'points' ) );
-	}
-
-	/**
-	 * Test that it returns false when a component isn't supported.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::supports_component
-	 */
-	public function test_supports_unsupported_component() {
-
-		$this->assertFalse( $this->importer->supports_component( 'unsupported' ) );
-	}
-
-	/**
-	 * Test that it returns the settings for a component.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::get_options_for_component
-	 */
-	public function test_get_options_for_component() {
-
-		$this->assertSame(
-			$this->importer_components['points']
-			, $this->importer->get_options_for_component( 'points' )
-		);
-	}
-
-	/**
-	 * Test that it returns the settings for an unsupported component.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::get_options_for_component
-	 */
-	public function test_get_options_for_unsupported_component() {
-
-		$this->assertSame(
-			array()
-			, $this->importer->get_options_for_component( 'unsupported' )
-		);
-	}
-
-	/**
-	 * Test that it gives a warning for uninstalled components.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_not_installed() {
-
-		$this->importer->components = array(
-			'uninstalled' => array( 'method' => 'do_an_import' ),
-		);
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-		$this->importer->do_import(
-			array( 'uninstalled' => array( 'do' => 'yes' ) )
-			, $feedback
-		);
-
-		$this->assertCount( 1, $feedback->messages['warning'] );
-
-		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
-	}
-
-	/**
-	 * Test that it gives a warning for unsupported components.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_not_supported() {
-
-		$this->importer->components = array();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-		$this->importer->do_import(
-			array( 'points' => array( 'do' => 'yes' ) )
-			, $feedback
-		);
-
-		$this->assertCount( 1, $feedback->messages['warning'] );
-
-		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
-	}
-
-	/**
-	 * Test that it skips a component if validation fails.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_validates_settings() {
-
-		$this->listen_for_filter( 'wordpoints_import_settings_valid-points' );
-
-		add_filter( 'wordpoints_import_settings_valid-points', '__return_false' );
-
-		$this->importer->do_import(
-			array( 'points' => array( 'do' => 'yes' ) )
-			, new WordPoints_Importer_Tests_Feedback()
-		);
-
-		$this->assertSame(
-			1
-			, $this->filter_was_called( 'wordpoints_import_settings_valid-points' )
-		);
-
-		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
-	}
-
-	/**
-	 * Test that it skips an unsupported option.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_invalid_option() {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$this->importer->do_import(
-			array( 'points' => array( 'do' => 'yes' ) )
-			, $feedback
-		);
-
-		$this->assertCount( 1, $feedback->messages['warning'] );
-
-		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
-	}
-
-	/**
-	 * Test that it skips a disabled option.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_disabled_option() {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$this->importer->components['points']['user_points']['can_import'] =
-			array( $this->importer, 'cant_import' );
-
-		$this->importer->do_import(
-			array( 'points' => array( 'user_points' => '1' ) )
-			, $feedback
-		);
-
-		$this->assertCount( 1, $this->importer->can_imports );
-		$this->assertCount( 1, $feedback->messages['warning'] );
-
-		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
-	}
-
-	/**
-	 * Test that the can_import function is passed any settings.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_can_import_passed_settings() {
-
-		$this->importer->do_import(
-			array(
-				'points' => array(
-					'user_points' => '1',
-					'_data' => array( 'testing' => 1 ),
-				),
-			)
-			, new WordPoints_Importer_Tests_Feedback()
-		);
-
-		$this->assertCount( 1, $this->importer->can_imports );
-		$this->assertSame(
-			array( 'testing' => 1 )
-			, $this->importer->can_imports[0]
-		);
-	}
-
-	/**
-	 * Test that it calls the importer function.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import() {
-
-		$this->importer->do_import(
-			array( 'points' => array( 'user_points' => '1' ) )
-			, new WordPoints_Importer_Tests_Feedback()
-		);
-
-		$this->assertCount( 1, $this->importer->imports );
-	}
-
-	/**
-	 * Test that the import function is passed any settings.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importer::do_import
-	 */
-	public function test_do_import_passed_settings() {
-
-		$this->importer->do_import(
-			array(
-				'points' => array(
-					'user_points' => '1',
-					'_data' => array( 'testing' => 1 ),
-				),
-			)
-			, new WordPoints_Importer_Tests_Feedback()
-		);
-
-		$this->assertCount( 1, $this->importer->imports );
-		$this->assertSame(
-			array( 'testing' => 1 )
-			, $this->importer->imports[0]
-		);
-	}
+    /**
+     * The mock importer used in the tests.
+     *
+     * @since 1.0.0
+     *
+     * @var WordPoints_Importer_Mock
+     */
+    protected $importer;
+
+    /**
+     * The components assigned to the mock importer.
+     *
+     * @since 1.0.0
+     *
+     * @var array[]
+     */
+    protected $importer_components;
+
+    /**
+     * @since 1.0.0
+     */
+    public function setUp() {
+
+        parent::setUp();
+
+        $this->importer = new WordPoints_Importer_Mock( 'Mock' );
+        $this->importer->components = array(
+            'points' => array(
+                'user_points' => array(
+                    'label' => 'User points',
+                    'function' => array( $this->importer, 'do_an_import' ),
+                    'can_import' => array( $this->importer, 'can_import' ),
+                ),
+            ),
+        );
+
+        $this->importer_components = $this->importer->components;
+
+        remove_action(
+            'wordpoints_import_settings_valid-points'
+            , 'wordpoints_importer_validate_points_type_setting'
+        );
+    }
+
+    /**
+     * Test that it returns true when a component is supported.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::supports_component
+     */
+    public function test_supports_supported_component() {
+
+        $this->assertTrue( $this->importer->supports_component( 'points' ) );
+    }
+
+    /**
+     * Test that it returns false when a component isn't supported.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::supports_component
+     */
+    public function test_supports_unsupported_component() {
+
+        $this->assertFalse( $this->importer->supports_component( 'unsupported' ) );
+    }
+
+    /**
+     * Test that it returns the settings for a component.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::get_options_for_component
+     */
+    public function test_get_options_for_component() {
+
+        $this->assertSame(
+            $this->importer_components['points']
+            , $this->importer->get_options_for_component( 'points' )
+        );
+    }
+
+    /**
+     * Test that it returns the settings for an unsupported component.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::get_options_for_component
+     */
+    public function test_get_options_for_unsupported_component() {
+
+        $this->assertSame(
+            array()
+            , $this->importer->get_options_for_component( 'unsupported' )
+        );
+    }
+
+    /**
+     * Test that it gives a warning for uninstalled components.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_not_installed() {
+
+        $this->importer->components = array(
+            'uninstalled' => array( 'method' => 'do_an_import' ),
+        );
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+        $this->importer->do_import(
+            array( 'uninstalled' => array( 'do' => 'yes' ) )
+            , $feedback
+        );
+
+        $this->assertCount( 1, $feedback->messages['warning'] );
+
+        // The import shouldn't have been performed.
+        $this->assertSame( array(), $this->importer->imports );
+    }
+
+    /**
+     * Test that it gives a warning for unsupported components.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_not_supported() {
+
+        $this->importer->components = array();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+        $this->importer->do_import(
+            array( 'points' => array( 'do' => 'yes' ) )
+            , $feedback
+        );
+
+        $this->assertCount( 1, $feedback->messages['warning'] );
+
+        // The import shouldn't have been performed.
+        $this->assertSame( array(), $this->importer->imports );
+    }
+
+    /**
+     * Test that it skips a component if validation fails.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_validates_settings() {
+
+        $this->listen_for_filter( 'wordpoints_import_settings_valid-points' );
+
+        add_filter( 'wordpoints_import_settings_valid-points', '__return_false' );
+
+        $this->importer->do_import(
+            array( 'points' => array( 'do' => 'yes' ) )
+            , new WordPoints_Importer_Tests_Feedback()
+        );
+
+        $this->assertSame(
+            1
+            , $this->filter_was_called( 'wordpoints_import_settings_valid-points' )
+        );
+
+        // The import shouldn't have been performed.
+        $this->assertSame( array(), $this->importer->imports );
+    }
+
+    /**
+     * Test that it skips an unsupported option.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_invalid_option() {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $this->importer->do_import(
+            array( 'points' => array( 'do' => 'yes' ) )
+            , $feedback
+        );
+
+        $this->assertCount( 1, $feedback->messages['warning'] );
+
+        // The import shouldn't have been performed.
+        $this->assertSame( array(), $this->importer->imports );
+    }
+
+    /**
+     * Test that it skips a disabled option.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_disabled_option() {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $this->importer->components['points']['user_points']['can_import'] =
+            array( $this->importer, 'cant_import' );
+
+        $this->importer->do_import(
+            array( 'points' => array( 'user_points' => '1' ) )
+            , $feedback
+        );
+
+        $this->assertCount( 1, $this->importer->can_imports );
+        $this->assertCount( 1, $feedback->messages['warning'] );
+
+        // The import shouldn't have been performed.
+        $this->assertSame( array(), $this->importer->imports );
+    }
+
+    /**
+     * Test that the can_import function is passed any settings.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_can_import_passed_settings() {
+
+        $this->importer->do_import(
+            array(
+                'points' => array(
+                    'user_points' => '1',
+                    '_data' => array( 'testing' => 1 ),
+                ),
+            )
+            , new WordPoints_Importer_Tests_Feedback()
+        );
+
+        $this->assertCount( 1, $this->importer->can_imports );
+        $this->assertSame(
+            array( 'testing' => 1 )
+            , $this->importer->can_imports[0]
+        );
+    }
+
+    /**
+     * Test that it calls the importer function.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import() {
+
+        $this->importer->do_import(
+            array( 'points' => array( 'user_points' => '1' ) )
+            , new WordPoints_Importer_Tests_Feedback()
+        );
+
+        $this->assertCount( 1, $this->importer->imports );
+    }
+
+    /**
+     * Test that the import function is passed any settings.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importer::do_import
+     */
+    public function test_do_import_passed_settings() {
+
+        $this->importer->do_import(
+            array(
+                'points' => array(
+                    'user_points' => '1',
+                    '_data' => array( 'testing' => 1 ),
+                ),
+            )
+            , new WordPoints_Importer_Tests_Feedback()
+        );
+
+        $this->assertCount( 1, $this->importer->imports );
+        $this->assertSame(
+            array( 'testing' => 1 )
+            , $this->importer->imports[0]
+        );
+    }
 }
 
 // EOF

Spacing +35 added lines, -35 removed lines

@@ -41,13 +41,13 @@ 
 
 		parent::setUp();
 
-		$this->importer = new WordPoints_Importer_Mock( 'Mock' );
+		$this->importer = new WordPoints_Importer_Mock('Mock');
 		$this->importer->components = array(
 			'points' => array(
 				'user_points' => array(
 					'label' => 'User points',
-					'function' => array( $this->importer, 'do_an_import' ),
-					'can_import' => array( $this->importer, 'can_import' ),
+					'function' => array($this->importer, 'do_an_import'),
+					'can_import' => array($this->importer, 'can_import'),
 				),
 			),
 		);
@@ -69,7 +69,7 @@ 
 	 */
 	public function test_supports_supported_component() {
 
-		$this->assertTrue( $this->importer->supports_component( 'points' ) );
+		$this->assertTrue($this->importer->supports_component('points'));
 	}
 
 	/**
@@ -81,7 +81,7 @@ 
 	 */
 	public function test_supports_unsupported_component() {
 
-		$this->assertFalse( $this->importer->supports_component( 'unsupported' ) );
+		$this->assertFalse($this->importer->supports_component('unsupported'));
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 
 		$this->assertSame(
 			$this->importer_components['points']
-			, $this->importer->get_options_for_component( 'points' )
+			, $this->importer->get_options_for_component('points')
 		);
 	}
 
@@ -110,7 +110,7 @@ 
 
 		$this->assertSame(
 			array()
-			, $this->importer->get_options_for_component( 'unsupported' )
+			, $this->importer->get_options_for_component('unsupported')
 		);
 	}
 
@@ -124,19 +124,19 @@ 
 	public function test_do_import_not_installed() {
 
 		$this->importer->components = array(
-			'uninstalled' => array( 'method' => 'do_an_import' ),
+			'uninstalled' => array('method' => 'do_an_import'),
 		);
 
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 		$this->importer->do_import(
-			array( 'uninstalled' => array( 'do' => 'yes' ) )
+			array('uninstalled' => array('do' => 'yes'))
 			, $feedback
 		);
 
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertCount(1, $feedback->messages['warning']);
 
 		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
+		$this->assertSame(array(), $this->importer->imports);
 	}
 
 	/**
@@ -152,14 +152,14 @@ 
 
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 		$this->importer->do_import(
-			array( 'points' => array( 'do' => 'yes' ) )
+			array('points' => array('do' => 'yes'))
 			, $feedback
 		);
 
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertCount(1, $feedback->messages['warning']);
 
 		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
+		$this->assertSame(array(), $this->importer->imports);
 	}
 
 	/**
@@ -171,22 +171,22 @@ 
 	 */
 	public function test_do_import_validates_settings() {
 
-		$this->listen_for_filter( 'wordpoints_import_settings_valid-points' );
+		$this->listen_for_filter('wordpoints_import_settings_valid-points');
 
-		add_filter( 'wordpoints_import_settings_valid-points', '__return_false' );
+		add_filter('wordpoints_import_settings_valid-points', '__return_false');
 
 		$this->importer->do_import(
-			array( 'points' => array( 'do' => 'yes' ) )
+			array('points' => array('do' => 'yes'))
 			, new WordPoints_Importer_Tests_Feedback()
 		);
 
 		$this->assertSame(
 			1
-			, $this->filter_was_called( 'wordpoints_import_settings_valid-points' )
+			, $this->filter_was_called('wordpoints_import_settings_valid-points')
 		);
 
 		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
+		$this->assertSame(array(), $this->importer->imports);
 	}
 
 	/**
@@ -201,14 +201,14 @@ 
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 
 		$this->importer->do_import(
-			array( 'points' => array( 'do' => 'yes' ) )
+			array('points' => array('do' => 'yes'))
 			, $feedback
 		);
 
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertCount(1, $feedback->messages['warning']);
 
 		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
+		$this->assertSame(array(), $this->importer->imports);
 	}
 
 	/**
@@ -223,18 +223,18 @@ 
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 
 		$this->importer->components['points']['user_points']['can_import'] =
-			array( $this->importer, 'cant_import' );
+			array($this->importer, 'cant_import');
 
 		$this->importer->do_import(
-			array( 'points' => array( 'user_points' => '1' ) )
+			array('points' => array('user_points' => '1'))
 			, $feedback
 		);
 
-		$this->assertCount( 1, $this->importer->can_imports );
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertCount(1, $this->importer->can_imports);
+		$this->assertCount(1, $feedback->messages['warning']);
 
 		// The import shouldn't have been performed.
-		$this->assertSame( array(), $this->importer->imports );
+		$this->assertSame(array(), $this->importer->imports);
 	}
 
 	/**
@@ -250,15 +250,15 @@ 
 			array(
 				'points' => array(
 					'user_points' => '1',
-					'_data' => array( 'testing' => 1 ),
+					'_data' => array('testing' => 1),
 				),
 			)
 			, new WordPoints_Importer_Tests_Feedback()
 		);
 
-		$this->assertCount( 1, $this->importer->can_imports );
+		$this->assertCount(1, $this->importer->can_imports);
 		$this->assertSame(
-			array( 'testing' => 1 )
+			array('testing' => 1)
 			, $this->importer->can_imports[0]
 		);
 	}
@@ -273,11 +273,11 @@ 
 	public function test_do_import() {
 
 		$this->importer->do_import(
-			array( 'points' => array( 'user_points' => '1' ) )
+			array('points' => array('user_points' => '1'))
 			, new WordPoints_Importer_Tests_Feedback()
 		);
 
-		$this->assertCount( 1, $this->importer->imports );
+		$this->assertCount(1, $this->importer->imports);
 	}
 
 	/**
@@ -293,15 +293,15 @@ 
 			array(
 				'points' => array(
 					'user_points' => '1',
-					'_data' => array( 'testing' => 1 ),
+					'_data' => array('testing' => 1),
 				),
 			)
 			, new WordPoints_Importer_Tests_Feedback()
 		);
 
-		$this->assertCount( 1, $this->importer->imports );
+		$this->assertCount(1, $this->importer->imports);
 		$this->assertSame(
-			array( 'testing' => 1 )
+			array('testing' => 1)
 			, $this->importer->imports[0]
 		);
 	}

tests/phpunit/tests/admin-functions.php

Indentation +175 added lines, -175 removed lines

@@ -14,181 +14,181 @@
  */
 class WordPoints_Importer_Admin_Test extends WordPoints_PHPUnit_TestCase_Points {
 
-	/**
-	 * Test that a points type must be supplied.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers ::wordpoints_importer_validate_points_type_setting
-	 */
-	public function test_validate_points_type_setting_not_set() {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_points_type_setting(
-			true
-			, array()
-			, $feedback
-		);
-
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
-	}
-
-	/**
-	 * Test that the points type supplied must be valid.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers ::wordpoints_importer_validate_points_type_setting
-	 */
-	public function test_validate_points_type_setting_invalid() {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_points_type_setting(
-			true
-			, array( 'points_type' => 'invalid' )
-			, $feedback
-		);
-
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
-	}
-
-	/**
-	 * Test that it returns true when supplied a valid points type.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers ::wordpoints_importer_validate_points_type_setting
-	 */
-	public function test_validate_points_type_setting_valid() {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_points_type_setting(
-			true
-			, array( 'points_type' => 'points' )
-			, $feedback
-		);
-
-		$this->assertTrue( $valid );
-		$this->assertSame( array(), $feedback->messages );
-	}
-
-	/**
-	 * Test that it returns false when supplied a valid points type if $valid is false.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers ::wordpoints_importer_validate_points_type_setting
-	 */
-	public function test_validate_points_type_setting_valid_false() {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_points_type_setting(
-			false
-			, array( 'points_type' => 'points' )
-			, $feedback
-		);
-
-		$this->assertFalse( $valid );
-		$this->assertSame( array(), $feedback->messages );
-	}
-
-	/**
-	 * Test that a rank group must be supplied.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers ::wordpoints_importer_validate_rank_group_setting
-	 */
-	public function test_validate_rank_group_setting_not_set() {
-
-		wordpoints_register_points_ranks();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_rank_group_setting(
-			true
-			, array()
-			, $feedback
-		);
-
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
-	}
-
-	/**
-	 * Test that the rank group supplied must be valid.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers ::wordpoints_importer_validate_rank_group_setting
-	 */
-	public function test_validate_rank_group_setting_invalid() {
-
-		wordpoints_register_points_ranks();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_rank_group_setting(
-			true
-			, array( 'rank_group' => 'invalid' )
-			, $feedback
-		);
-
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
-	}
-
-	/**
-	 * Test that it returns true when supplied a valid rank group.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers ::wordpoints_importer_validate_rank_group_setting
-	 */
-	public function test_validate_rank_group_setting_valid() {
-
-		wordpoints_register_points_ranks();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_rank_group_setting(
-			true
-			, array( 'rank_group' => 'points_type-points' )
-			, $feedback
-		);
-
-		$this->assertTrue( $valid );
-		$this->assertSame( array(), $feedback->messages );
-	}
-
-	/**
-	 * Test that it returns false when supplied a valid rank group if $valid is false.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers ::wordpoints_importer_validate_rank_group_setting
-	 */
-	public function test_validate_rank_group_setting_valid_false() {
-
-		wordpoints_register_points_ranks();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$valid = wordpoints_importer_validate_rank_group_setting(
-			false
-			, array( 'rank_group' => 'points_type-points' )
-			, $feedback
-		);
-
-		$this->assertFalse( $valid );
-		$this->assertSame( array(), $feedback->messages );
-	}
+    /**
+     * Test that a points type must be supplied.
+     *
+     * @since 1.0.0
+     *
+     * @covers ::wordpoints_importer_validate_points_type_setting
+     */
+    public function test_validate_points_type_setting_not_set() {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_points_type_setting(
+            true
+            , array()
+            , $feedback
+        );
+
+        $this->assertFalse( $valid );
+        $this->assertCount( 1, $feedback->messages['warning'] );
+    }
+
+    /**
+     * Test that the points type supplied must be valid.
+     *
+     * @since 1.0.0
+     *
+     * @covers ::wordpoints_importer_validate_points_type_setting
+     */
+    public function test_validate_points_type_setting_invalid() {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_points_type_setting(
+            true
+            , array( 'points_type' => 'invalid' )
+            , $feedback
+        );
+
+        $this->assertFalse( $valid );
+        $this->assertCount( 1, $feedback->messages['warning'] );
+    }
+
+    /**
+     * Test that it returns true when supplied a valid points type.
+     *
+     * @since 1.0.0
+     *
+     * @covers ::wordpoints_importer_validate_points_type_setting
+     */
+    public function test_validate_points_type_setting_valid() {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_points_type_setting(
+            true
+            , array( 'points_type' => 'points' )
+            , $feedback
+        );
+
+        $this->assertTrue( $valid );
+        $this->assertSame( array(), $feedback->messages );
+    }
+
+    /**
+     * Test that it returns false when supplied a valid points type if $valid is false.
+     *
+     * @since 1.0.0
+     *
+     * @covers ::wordpoints_importer_validate_points_type_setting
+     */
+    public function test_validate_points_type_setting_valid_false() {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_points_type_setting(
+            false
+            , array( 'points_type' => 'points' )
+            , $feedback
+        );
+
+        $this->assertFalse( $valid );
+        $this->assertSame( array(), $feedback->messages );
+    }
+
+    /**
+     * Test that a rank group must be supplied.
+     *
+     * @since 1.1.0
+     *
+     * @covers ::wordpoints_importer_validate_rank_group_setting
+     */
+    public function test_validate_rank_group_setting_not_set() {
+
+        wordpoints_register_points_ranks();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_rank_group_setting(
+            true
+            , array()
+            , $feedback
+        );
+
+        $this->assertFalse( $valid );
+        $this->assertCount( 1, $feedback->messages['warning'] );
+    }
+
+    /**
+     * Test that the rank group supplied must be valid.
+     *
+     * @since 1.1.0
+     *
+     * @covers ::wordpoints_importer_validate_rank_group_setting
+     */
+    public function test_validate_rank_group_setting_invalid() {
+
+        wordpoints_register_points_ranks();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_rank_group_setting(
+            true
+            , array( 'rank_group' => 'invalid' )
+            , $feedback
+        );
+
+        $this->assertFalse( $valid );
+        $this->assertCount( 1, $feedback->messages['warning'] );
+    }
+
+    /**
+     * Test that it returns true when supplied a valid rank group.
+     *
+     * @since 1.1.0
+     *
+     * @covers ::wordpoints_importer_validate_rank_group_setting
+     */
+    public function test_validate_rank_group_setting_valid() {
+
+        wordpoints_register_points_ranks();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_rank_group_setting(
+            true
+            , array( 'rank_group' => 'points_type-points' )
+            , $feedback
+        );
+
+        $this->assertTrue( $valid );
+        $this->assertSame( array(), $feedback->messages );
+    }
+
+    /**
+     * Test that it returns false when supplied a valid rank group if $valid is false.
+     *
+     * @since 1.1.0
+     *
+     * @covers ::wordpoints_importer_validate_rank_group_setting
+     */
+    public function test_validate_rank_group_setting_valid_false() {
+
+        wordpoints_register_points_ranks();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $valid = wordpoints_importer_validate_rank_group_setting(
+            false
+            , array( 'rank_group' => 'points_type-points' )
+            , $feedback
+        );
+
+        $this->assertFalse( $valid );
+        $this->assertSame( array(), $feedback->messages );
+    }
 }
 
 // EOF

Spacing +22 added lines, -22 removed lines

@@ -31,8 +31,8 @@ 
 			, $feedback
 		);
 
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertFalse($valid);
+		$this->assertCount(1, $feedback->messages['warning']);
 	}
 
 	/**
@@ -48,12 +48,12 @@ 
 
 		$valid = wordpoints_importer_validate_points_type_setting(
 			true
-			, array( 'points_type' => 'invalid' )
+			, array('points_type' => 'invalid')
 			, $feedback
 		);
 
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertFalse($valid);
+		$this->assertCount(1, $feedback->messages['warning']);
 	}
 
 	/**
@@ -69,12 +69,12 @@ 
 
 		$valid = wordpoints_importer_validate_points_type_setting(
 			true
-			, array( 'points_type' => 'points' )
+			, array('points_type' => 'points')
 			, $feedback
 		);
 
-		$this->assertTrue( $valid );
-		$this->assertSame( array(), $feedback->messages );
+		$this->assertTrue($valid);
+		$this->assertSame(array(), $feedback->messages);
 	}
 
 	/**
@@ -90,12 +90,12 @@ 
 
 		$valid = wordpoints_importer_validate_points_type_setting(
 			false
-			, array( 'points_type' => 'points' )
+			, array('points_type' => 'points')
 			, $feedback
 		);
 
-		$this->assertFalse( $valid );
-		$this->assertSame( array(), $feedback->messages );
+		$this->assertFalse($valid);
+		$this->assertSame(array(), $feedback->messages);
 	}
 
 	/**
@@ -117,8 +117,8 @@ 
 			, $feedback
 		);
 
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertFalse($valid);
+		$this->assertCount(1, $feedback->messages['warning']);
 	}
 
 	/**
@@ -136,12 +136,12 @@ 
 
 		$valid = wordpoints_importer_validate_rank_group_setting(
 			true
-			, array( 'rank_group' => 'invalid' )
+			, array('rank_group' => 'invalid')
 			, $feedback
 		);
 
-		$this->assertFalse( $valid );
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertFalse($valid);
+		$this->assertCount(1, $feedback->messages['warning']);
 	}
 
 	/**
@@ -159,12 +159,12 @@ 
 
 		$valid = wordpoints_importer_validate_rank_group_setting(
 			true
-			, array( 'rank_group' => 'points_type-points' )
+			, array('rank_group' => 'points_type-points')
 			, $feedback
 		);
 
-		$this->assertTrue( $valid );
-		$this->assertSame( array(), $feedback->messages );
+		$this->assertTrue($valid);
+		$this->assertSame(array(), $feedback->messages);
 	}
 
 	/**
@@ -182,12 +182,12 @@ 
 
 		$valid = wordpoints_importer_validate_rank_group_setting(
 			false
-			, array( 'rank_group' => 'points_type-points' )
+			, array('rank_group' => 'points_type-points')
 			, $feedback
 		);
 
-		$this->assertFalse( $valid );
-		$this->assertSame( array(), $feedback->messages );
+		$this->assertFalse($valid);
+		$this->assertSame(array(), $feedback->messages);
 	}
 }
 

tests/phpunit/tests/importers/cubepoints.php

Indentation +850 added lines, -850 removed lines

@@ -16,861 +16,861 @@
  * @group cubepoints
  */
 class WordPoints_CubePoints_Importer_Test
-	extends WordPoints_PHPUnit_TestCase_Points {
-
-	/**
-	 * The importer used in the tests.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var WordPoints_CubePoints_Importer
-	 */
-	protected $importer;
-
-	/**
-	 * @since 1.0.0
-	 */
-	public function setUp() {
-
-		parent::setUp();
-
-		// These are usually inactive by default. We activate them in the tests
-		// bootstrap so that they will be fully loaded, but deactivate them here to
-		// restore default behavior.
-		cp_module_activation_set( 'post_author_points', false );
-		cp_module_activation_set( 'dailypoints', false );
-
-		$this->importer = new WordPoints_CubePoints_Importer( 'Test CubePoints' );
-	}
-
-	/**
-	 * @since 1.1.0
-	 */
-	public function tearDown() {
-
-		WordPoints_Rank_Groups::deregister_group( 'points_type-points' );
-		WordPoints_Rank_Types::deregister_type( 'points-points' );
-
-		parent::tearDown();
-	}
-
-	/**
-	 * Test that it returns true when CubePoints is installed.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::is_cubepoints_installed
-	 */
-	public function test_is_cubepoints_installed() {
-
-		$this->assertTrue( $this->importer->is_cubepoints_installed() );
-	}
-
-	/**
-	 * Test that it returns false when CubePoints is not installed.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::is_cubepoints_installed
-	 */
-	public function test_is_cubepoints_not_installed() {
-
-		delete_option( 'cp_db_version' );
-
-		$this->assertFalse( $this->importer->is_cubepoints_installed() );
-	}
-
-	/**
-	 * Test that it returns true when CubePoints is installed.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::is_available
-	 */
-	public function test_is_cubepoints_is_available() {
-
-		$this->assertTrue( $this->importer->is_available() );
-	}
-
-	/**
-	 * Test that it returns a WP_Error when CubePoints is not installed.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::is_available
-	 */
-	public function test_is_cubepoints_not_available() {
-
-		delete_option( 'cp_db_version' );
-
-		$this->assertWPError( $this->importer->is_available() );
-	}
-
-	/**
-	 * Test that it returns true when CubePoints is active.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::is_cubepoints_active
-	 */
-	public function test_is_cubepoints_active() {
-
-		$this->assertSame(
-			function_exists( 'cp_ready' )
-			, $this->importer->is_cubepoints_active()
-		);
-	}
-
-	/**
-	 * Test importing the excluded users.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_excluded_users
-	 */
-	public function test_import_excluded_users() {
-
-		$user_ids = $this->factory->user->create_many( 3 );
-		$user_logins = array();
-
-		foreach ( $user_ids as $user_id ) {
-			$user_logins[] = get_userdata( $user_id )->user_login;
-		}
-
-		update_option( 'cp_topfilter', $user_logins );
-
-		$this->do_points_import( 'excluded_users' );
-
-		$this->assertSame(
-			$user_ids
-			, wordpoints_get_excluded_users( 'tests' )
-		);
-	}
-
-	/**
-	 * Test importing the excluded users gives a warning if there are none.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_excluded_users
-	 */
-	public function test_import_excluded_users_none() {
-
-		delete_option( 'cp_topfilter' );
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$this->importer->do_import(
-			array(
-				'points' => array(
-					'excluded_users' => '1',
-					'_data' => array( 'points_type' => 'points' ),
-				),
-			)
-			, $feedback
-		);
-
-		$this->assertCount( 1, $feedback->messages['warning'] );
-	}
-
-	/**
-	 * Test importing the points settings to points hooks.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_settings
-	 * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
-	 * @covers WordPoints_CubePoints_Importer::format_settings_for_post_type
-	 */
-	public function test_import_points_settings() {
-
-		update_option( 'cp_comment_points',     10 );
-		update_option( 'cp_post_points',        20 );
-		update_option( 'cp_reg_points',         50 );
-
-		$this->do_points_import( 'settings' );
-
-		$this->assertHookImported(
-			array(
-				'event' => 'comment_leave\post',
-				'target' => array( 'comment\post', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 10,
-				'points_type' => 'points',
-				'log_text' => 'Comment on a Post.',
-				'description' => 'Commenting on a Post.',
-				'legacy_log_type' => 'cubepoints-comment',
-				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-
-		$this->assertHookImported(
-			array(
-				'event' => 'comment_leave\page',
-				'target' => array( 'comment\page', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 10,
-				'points_type' => 'points',
-				'log_text' => 'Comment on a Page.',
-				'description' => 'Commenting on a Page.',
-				'legacy_log_type' => 'cubepoints-comment',
-				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-
-		$this->assertHookImported(
-			array(
-				'event' => 'comment_leave\attachment',
-				'target' => array( 'comment\attachment', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 10,
-				'points_type' => 'points',
-				'log_text' => 'Comment on a Media.',
-				'description' => 'Commenting on a Media.',
-				'legacy_log_type' => 'cubepoints-comment',
-				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-
-		$this->assertHookImported(
-			array(
-				'event' => 'post_publish\post',
-				'target' => array( 'post\post', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 20,
-				'points_type' => 'points',
-				'log_text' => 'Published a Post.',
-				'description' => 'Publishing a Post.',
-				'blocker' => array( 'toggle_off' => true ),
-				'legacy_log_type' => 'cubepoints-post',
-				'legacy_meta_key' => 'post',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-				'points_legacy_repeat_blocker' => array( 'toggle_on' => true ),
-			)
-		);
-
-		$reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
-
-		$this->assertSame(
-			array()
-			, $reaction_store->get_reactions_to_event( 'post_publish\page' )
-		);
-
-		$this->assertSame(
-			array()
-			, $reaction_store->get_reactions_to_event( 'post_publish\attachment' )
-		);
-
-		$this->assertSame(
-			array()
-			, $reaction_store->get_reactions_to_event( 'media_upload' )
-		);
-
-		$this->assertHookImported(
-			array(
-				'event' => 'user_register',
-				'target' => array( 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 50,
-				'points_type' => 'points',
-				'log_text' => 'Registration.',
-				'description' => 'Registration.',
-				'legacy_log_type' => 'cubepoints-register',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-	}
-
-	/**
-	 * Test that it imports legacy points hooks on install.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @coversNothing
-	 */
-	public function test_imported_post_points_hook_does_not_refire() {
-
-		update_option( 'cp_post_points', 20 );
-
-		$user_id = $this->factory->user->create();
-		$post_id = $this->factory->post->create(
-			array(
-				'post_author' => $user_id,
-				'post_type'   => 'post',
-			)
-		);
-
-		$this->assertSame( '120', cp_getPoints( $user_id ) );
-
-		$this->factory->post->update_object(
-			$post_id
-			, array( 'post_status' => 'draft' )
-		);
-
-		$this->assertSame( '120', cp_getPoints( $user_id ) );
-
-		$this->do_points_import( 'settings' );
-		$this->do_points_import( 'user_points' );
-		$this->do_points_import( 'logs' );
-
-		$this->assertSame(
-			120
-			, wordpoints_get_points( $user_id, 'points' )
-		);
-
-		$this->factory->post->update_object(
-			$post_id
-			, array( 'post_status' => 'publish' )
-		);
-
-		$this->assertSame(
-			120
-			, wordpoints_get_points( $user_id, 'points' )
-		);
-	}
-
-	/**
-	 * Test importing the settings from the post author points module to points hooks.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_settings
-	 * @covers WordPoints_CubePoints_Importer::format_settings_for_post_type
-	 */
-	public function test_import_post_author_points() {
-
-		cp_module_activation_set( 'post_author_points', 'active' );
-
-		update_option( 'cp_post_author_points', 15 );
-
-		$this->do_points_import( 'settings' );
-
-		$this->assertHookImported(
-			array(
-				'event' => 'comment_leave\post',
-				'target' => array( 'comment\post', 'post\post', 'post\post', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 15,
-				'points_type' => 'points',
-				'log_text' => 'Received a comment on a Post.',
-				'description' => 'Receiving a comment on a Post.',
-				'legacy_log_type' => 'cubepoints-post_author',
-				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-
-		$this->assertHookImported(
-			array(
-				'event' => 'comment_leave\page',
-				'target' => array( 'comment\page', 'post\page', 'post\page', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 15,
-				'points_type' => 'points',
-				'log_text' => 'Received a comment on a Page.',
-				'description' => 'Receiving a comment on a Page.',
-				'legacy_log_type' => 'cubepoints-post_author',
-				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-
-		$this->assertHookImported(
-			array(
-				'event' => 'comment_leave\attachment',
-				'target' => array( 'comment\attachment', 'post\attachment', 'post\attachment', 'author', 'user' ),
-				'reactor' => 'points_legacy',
-				'points' => 15,
-				'points_type' => 'points',
-				'log_text' => 'Received a comment on a Media.',
-				'description' => 'Receiving a comment on a Media.',
-				'legacy_log_type' => 'cubepoints-post_author',
-				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-			)
-		);
-	}
-
-	/**
-	 * Test importing the settings from the daily points module to points hooks.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_settings
-	 * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
-	 */
-	public function test_import_periodic_points() {
-
-		cp_module_activation_set( 'dailypoints', 'active' );
-
-		update_option( 'cp_module_dailypoints_points', 30 );
-		update_option( 'cp_module_dailypoints_time', DAY_IN_SECONDS );
-
-		$this->do_points_import( 'settings' );
-
-		$this->assertHookImported(
-			array(
-				'event' => 'user_visit',
-				'target' => array( 'current:user' ),
-				'reactor' => 'points_legacy',
-				'points' => 30,
-				'points_type' => 'points',
-				'log_text' => 'Visiting the site.',
-				'description' => 'Visiting the site.',
-				'points_legacy_periods' => array(
-					'fire' => array(
-						array(
-							'length' => DAY_IN_SECONDS,
-							'args' => array( array( 'current:user' ) ),
-							'relative' => true,
-						),
-					),
-				),
-				'points_legacy_reversals' => array(),
-				'legacy_log_type' => 'cubepoints-dailypoints',
-			)
-		);
-	}
-
-	/**
-	 * Test that the imported user visit hook respects CubePoints's started periods.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_settings
-	 * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
-	 */
-	public function test_import_periodic_points_respect_old_periods() {
-
-		if ( version_compare( $GLOBALS['wp_version'], '4.5', '>=' ) ) {
-			$this->setExpectedDeprecated( 'get_currentuserinfo' );
-		}
-
-		cp_module_activation_set( 'dailypoints', 'active' );
-
-		update_option( 'cp_module_dailypoints_points', 30 );
-		update_option( 'cp_module_dailypoints_time', DAY_IN_SECONDS );
-
-		$user_id = $this->factory->user->create();
-
-		wp_set_current_user( $user_id );
-
-		$this->assertSame( '100', cp_getPoints( $user_id ) );
-
-		cp_module_dailypoints_checkTimer();
-
-		$this->assertSame( '130', cp_getPoints( $user_id ) );
-
-		// Running again shouldn't hit again.
-		cp_module_dailypoints_checkTimer();
-
-		$this->assertSame( '130', cp_getPoints( $user_id ) );
-
-		$this->do_points_import( 'settings' );
-		$this->do_points_import( 'user_points' );
-		$this->do_points_import( 'logs' );
-
-		$this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
-
-		wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
-
-		$this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
-
-		// Fast-forward and try again.
-		global $wpdb;
-
-		$id = $wpdb->get_var(
-			"
+    extends WordPoints_PHPUnit_TestCase_Points {
+
+    /**
+     * The importer used in the tests.
+     *
+     * @since 1.0.0
+     *
+     * @var WordPoints_CubePoints_Importer
+     */
+    protected $importer;
+
+    /**
+     * @since 1.0.0
+     */
+    public function setUp() {
+
+        parent::setUp();
+
+        // These are usually inactive by default. We activate them in the tests
+        // bootstrap so that they will be fully loaded, but deactivate them here to
+        // restore default behavior.
+        cp_module_activation_set( 'post_author_points', false );
+        cp_module_activation_set( 'dailypoints', false );
+
+        $this->importer = new WordPoints_CubePoints_Importer( 'Test CubePoints' );
+    }
+
+    /**
+     * @since 1.1.0
+     */
+    public function tearDown() {
+
+        WordPoints_Rank_Groups::deregister_group( 'points_type-points' );
+        WordPoints_Rank_Types::deregister_type( 'points-points' );
+
+        parent::tearDown();
+    }
+
+    /**
+     * Test that it returns true when CubePoints is installed.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::is_cubepoints_installed
+     */
+    public function test_is_cubepoints_installed() {
+
+        $this->assertTrue( $this->importer->is_cubepoints_installed() );
+    }
+
+    /**
+     * Test that it returns false when CubePoints is not installed.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::is_cubepoints_installed
+     */
+    public function test_is_cubepoints_not_installed() {
+
+        delete_option( 'cp_db_version' );
+
+        $this->assertFalse( $this->importer->is_cubepoints_installed() );
+    }
+
+    /**
+     * Test that it returns true when CubePoints is installed.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::is_available
+     */
+    public function test_is_cubepoints_is_available() {
+
+        $this->assertTrue( $this->importer->is_available() );
+    }
+
+    /**
+     * Test that it returns a WP_Error when CubePoints is not installed.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::is_available
+     */
+    public function test_is_cubepoints_not_available() {
+
+        delete_option( 'cp_db_version' );
+
+        $this->assertWPError( $this->importer->is_available() );
+    }
+
+    /**
+     * Test that it returns true when CubePoints is active.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::is_cubepoints_active
+     */
+    public function test_is_cubepoints_active() {
+
+        $this->assertSame(
+            function_exists( 'cp_ready' )
+            , $this->importer->is_cubepoints_active()
+        );
+    }
+
+    /**
+     * Test importing the excluded users.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_excluded_users
+     */
+    public function test_import_excluded_users() {
+
+        $user_ids = $this->factory->user->create_many( 3 );
+        $user_logins = array();
+
+        foreach ( $user_ids as $user_id ) {
+            $user_logins[] = get_userdata( $user_id )->user_login;
+        }
+
+        update_option( 'cp_topfilter', $user_logins );
+
+        $this->do_points_import( 'excluded_users' );
+
+        $this->assertSame(
+            $user_ids
+            , wordpoints_get_excluded_users( 'tests' )
+        );
+    }
+
+    /**
+     * Test importing the excluded users gives a warning if there are none.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_excluded_users
+     */
+    public function test_import_excluded_users_none() {
+
+        delete_option( 'cp_topfilter' );
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $this->importer->do_import(
+            array(
+                'points' => array(
+                    'excluded_users' => '1',
+                    '_data' => array( 'points_type' => 'points' ),
+                ),
+            )
+            , $feedback
+        );
+
+        $this->assertCount( 1, $feedback->messages['warning'] );
+    }
+
+    /**
+     * Test importing the points settings to points hooks.
+     *
+     * @since 1.1.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_settings
+     * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
+     * @covers WordPoints_CubePoints_Importer::format_settings_for_post_type
+     */
+    public function test_import_points_settings() {
+
+        update_option( 'cp_comment_points',     10 );
+        update_option( 'cp_post_points',        20 );
+        update_option( 'cp_reg_points',         50 );
+
+        $this->do_points_import( 'settings' );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'comment_leave\post',
+                'target' => array( 'comment\post', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 10,
+                'points_type' => 'points',
+                'log_text' => 'Comment on a Post.',
+                'description' => 'Commenting on a Post.',
+                'legacy_log_type' => 'cubepoints-comment',
+                'legacy_meta_key' => 'comment',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'comment_leave\page',
+                'target' => array( 'comment\page', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 10,
+                'points_type' => 'points',
+                'log_text' => 'Comment on a Page.',
+                'description' => 'Commenting on a Page.',
+                'legacy_log_type' => 'cubepoints-comment',
+                'legacy_meta_key' => 'comment',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'comment_leave\attachment',
+                'target' => array( 'comment\attachment', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 10,
+                'points_type' => 'points',
+                'log_text' => 'Comment on a Media.',
+                'description' => 'Commenting on a Media.',
+                'legacy_log_type' => 'cubepoints-comment',
+                'legacy_meta_key' => 'comment',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'post_publish\post',
+                'target' => array( 'post\post', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 20,
+                'points_type' => 'points',
+                'log_text' => 'Published a Post.',
+                'description' => 'Publishing a Post.',
+                'blocker' => array( 'toggle_off' => true ),
+                'legacy_log_type' => 'cubepoints-post',
+                'legacy_meta_key' => 'post',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+                'points_legacy_repeat_blocker' => array( 'toggle_on' => true ),
+            )
+        );
+
+        $reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+
+        $this->assertSame(
+            array()
+            , $reaction_store->get_reactions_to_event( 'post_publish\page' )
+        );
+
+        $this->assertSame(
+            array()
+            , $reaction_store->get_reactions_to_event( 'post_publish\attachment' )
+        );
+
+        $this->assertSame(
+            array()
+            , $reaction_store->get_reactions_to_event( 'media_upload' )
+        );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'user_register',
+                'target' => array( 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 50,
+                'points_type' => 'points',
+                'log_text' => 'Registration.',
+                'description' => 'Registration.',
+                'legacy_log_type' => 'cubepoints-register',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+    }
+
+    /**
+     * Test that it imports legacy points hooks on install.
+     *
+     * @since 1.0.0
+     *
+     * @coversNothing
+     */
+    public function test_imported_post_points_hook_does_not_refire() {
+
+        update_option( 'cp_post_points', 20 );
+
+        $user_id = $this->factory->user->create();
+        $post_id = $this->factory->post->create(
+            array(
+                'post_author' => $user_id,
+                'post_type'   => 'post',
+            )
+        );
+
+        $this->assertSame( '120', cp_getPoints( $user_id ) );
+
+        $this->factory->post->update_object(
+            $post_id
+            , array( 'post_status' => 'draft' )
+        );
+
+        $this->assertSame( '120', cp_getPoints( $user_id ) );
+
+        $this->do_points_import( 'settings' );
+        $this->do_points_import( 'user_points' );
+        $this->do_points_import( 'logs' );
+
+        $this->assertSame(
+            120
+            , wordpoints_get_points( $user_id, 'points' )
+        );
+
+        $this->factory->post->update_object(
+            $post_id
+            , array( 'post_status' => 'publish' )
+        );
+
+        $this->assertSame(
+            120
+            , wordpoints_get_points( $user_id, 'points' )
+        );
+    }
+
+    /**
+     * Test importing the settings from the post author points module to points hooks.
+     *
+     * @since 1.1.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_settings
+     * @covers WordPoints_CubePoints_Importer::format_settings_for_post_type
+     */
+    public function test_import_post_author_points() {
+
+        cp_module_activation_set( 'post_author_points', 'active' );
+
+        update_option( 'cp_post_author_points', 15 );
+
+        $this->do_points_import( 'settings' );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'comment_leave\post',
+                'target' => array( 'comment\post', 'post\post', 'post\post', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 15,
+                'points_type' => 'points',
+                'log_text' => 'Received a comment on a Post.',
+                'description' => 'Receiving a comment on a Post.',
+                'legacy_log_type' => 'cubepoints-post_author',
+                'legacy_meta_key' => 'comment',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'comment_leave\page',
+                'target' => array( 'comment\page', 'post\page', 'post\page', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 15,
+                'points_type' => 'points',
+                'log_text' => 'Received a comment on a Page.',
+                'description' => 'Receiving a comment on a Page.',
+                'legacy_log_type' => 'cubepoints-post_author',
+                'legacy_meta_key' => 'comment',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'comment_leave\attachment',
+                'target' => array( 'comment\attachment', 'post\attachment', 'post\attachment', 'author', 'user' ),
+                'reactor' => 'points_legacy',
+                'points' => 15,
+                'points_type' => 'points',
+                'log_text' => 'Received a comment on a Media.',
+                'description' => 'Receiving a comment on a Media.',
+                'legacy_log_type' => 'cubepoints-post_author',
+                'legacy_meta_key' => 'comment',
+                'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+            )
+        );
+    }
+
+    /**
+     * Test importing the settings from the daily points module to points hooks.
+     *
+     * @since 1.1.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_settings
+     * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
+     */
+    public function test_import_periodic_points() {
+
+        cp_module_activation_set( 'dailypoints', 'active' );
+
+        update_option( 'cp_module_dailypoints_points', 30 );
+        update_option( 'cp_module_dailypoints_time', DAY_IN_SECONDS );
+
+        $this->do_points_import( 'settings' );
+
+        $this->assertHookImported(
+            array(
+                'event' => 'user_visit',
+                'target' => array( 'current:user' ),
+                'reactor' => 'points_legacy',
+                'points' => 30,
+                'points_type' => 'points',
+                'log_text' => 'Visiting the site.',
+                'description' => 'Visiting the site.',
+                'points_legacy_periods' => array(
+                    'fire' => array(
+                        array(
+                            'length' => DAY_IN_SECONDS,
+                            'args' => array( array( 'current:user' ) ),
+                            'relative' => true,
+                        ),
+                    ),
+                ),
+                'points_legacy_reversals' => array(),
+                'legacy_log_type' => 'cubepoints-dailypoints',
+            )
+        );
+    }
+
+    /**
+     * Test that the imported user visit hook respects CubePoints's started periods.
+     *
+     * @since 1.2.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_settings
+     * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
+     */
+    public function test_import_periodic_points_respect_old_periods() {
+
+        if ( version_compare( $GLOBALS['wp_version'], '4.5', '>=' ) ) {
+            $this->setExpectedDeprecated( 'get_currentuserinfo' );
+        }
+
+        cp_module_activation_set( 'dailypoints', 'active' );
+
+        update_option( 'cp_module_dailypoints_points', 30 );
+        update_option( 'cp_module_dailypoints_time', DAY_IN_SECONDS );
+
+        $user_id = $this->factory->user->create();
+
+        wp_set_current_user( $user_id );
+
+        $this->assertSame( '100', cp_getPoints( $user_id ) );
+
+        cp_module_dailypoints_checkTimer();
+
+        $this->assertSame( '130', cp_getPoints( $user_id ) );
+
+        // Running again shouldn't hit again.
+        cp_module_dailypoints_checkTimer();
+
+        $this->assertSame( '130', cp_getPoints( $user_id ) );
+
+        $this->do_points_import( 'settings' );
+        $this->do_points_import( 'user_points' );
+        $this->do_points_import( 'logs' );
+
+        $this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
+
+        wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
+
+        $this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
+
+        // Fast-forward and try again.
+        global $wpdb;
+
+        $id = $wpdb->get_var(
+            "
 				SELECT `id`
 				FROM `{$wpdb->wordpoints_points_logs}`
 				ORDER BY `id` DESC
 				LIMIT 1
 			"
-		);
-
-		// Don't go all the way yet.
-		$updated = $wpdb->update(
-			$wpdb->wordpoints_points_logs
-			, array( 'date' => date( 'Y-m-d H:i:s', current_time( 'timestamp', true ) - DAY_IN_SECONDS + HOUR_IN_SECONDS ) )
-			, array( 'id' => $id )
-			, array( '%s' )
-			, array( '%d' )
-		);
-
-		$this->assertSame( 1, $updated );
-
-		// The periods cache will still hold the old date.
-		$this->flush_cache();
-
-		wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
-
-		// Points should have been awarded again yet.
-		$this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
-
-		// This time go all the way.
-		$updated = $wpdb->update(
-			$wpdb->wordpoints_points_logs
-			, array( 'date' => date( 'Y-m-d H:i:s', current_time( 'timestamp', true ) - DAY_IN_SECONDS - 1 ) )
-			, array( 'id' => $id )
-			, array( '%s' )
-			, array( '%d' )
-		);
-
-		$this->assertSame( 1, $updated );
-
-		// The periods cache will still hold the old date.
-		$this->flush_cache();
-
-		wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
-
-		// Points should have been awarded again.
-		$this->assertSame( 160, wordpoints_get_points( $user_id, 'points' ) );
-	}
-
-	/**
-	 * Test the imported periods when the site has a positive GMT offset.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_settings
-	 * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
-	 */
-	public function test_import_periods_positive_gmt_offset() {
-
-		update_option( 'gmt_offset', 5 );
-
-		$this->test_import_periodic_points_respect_old_periods();
-	}
-
-	/**
-	 * Test the imported periods when the site has a negative GMT offset.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_settings
-	 * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
-	 */
-	public function test_import_periods_negative_gmt_offset() {
-
-		update_option( 'gmt_offset', -5 );
-
-		$this->test_import_periodic_points_respect_old_periods();
-	}
-
-	/**
-	 * Test importing the user's points.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_user_points
-	 * @covers WordPoints_CubePoints_Importer::get_next_user_points_batch
-	 */
-	public function test_import_user_points() {
-
-		$user_points = array();
-
-		foreach ( array( 20, 10, 45 ) as $points ) {
-
-			$user_id = $this->factory->user->create();
-			cp_updatePoints( $user_id, $points );
-			$user_points[ $user_id ] = $points;
-		}
-
-		$this->do_points_import( 'user_points' );
-
-		foreach ( $user_points as $user_id => $points ) {
-			$this->assertSame( $points, wordpoints_get_points( $user_id, 'points' ) );
-		}
-	}
-
-	/**
-	 * Test importing the points logs.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_logs
-	 * @covers WordPoints_CubePoints_Importer::import_points_log
-	 * @covers WordPoints_CubePoints_Importer::get_next_points_logs_batch
-	 * @covers WordPoints_CubePoints_Importer::render_points_log_text
-	 */
-	public function test_import_points_logs() {
-
-		remove_action( 'publish_post', 'cp_newPost' );
-
-		$user_id = $this->factory->user->create();
-		cp_points( 'misc', $user_id, 10, 'Testing things.' );
-
-		$user_id_2 = $this->factory->user->create();
-		$post_id = $this->factory->post->create();
-		cp_points( 'post', $user_id_2, 25, $post_id );
-
-		$this->do_points_import( 'logs' );
-
-		$query = new WordPoints_Points_Logs_Query( array( 'order_by' => 'id' ) );
-		$logs = $query->get();
-
-		$this->assertCount( 4, $logs );
-
-		$log = $logs[2];
-
-		$this->assertSame( (string) $user_id, $log->user_id );
-		$this->assertSame( '10', $log->points );
-		$this->assertSame( 'Testing things.', $log->text );
-		$this->assertSame( 'cubepoints-misc', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'misc', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( 'Testing things.', wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-
-		$log = $logs[0];
-
-		$this->assertSame( (string) $user_id_2, $log->user_id );
-		$this->assertSame( '25', $log->points );
-		$this->assertStringMatchesFormat( 'Post on "<a href="%s">Post title %s</a>"', $log->text );
-		$this->assertSame( 'cubepoints-post', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'post', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $post_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $post_id, wordpoints_get_points_log_meta( $log->id, 'post', true ) );
-	}
-
-	/**
-	 * Test importing points logs that have been reversed.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_points_logs
-	 * @covers WordPoints_CubePoints_Importer::import_points_log
-	 * @covers WordPoints_CubePoints_Importer::get_next_points_logs_batch
-	 * @covers WordPoints_CubePoints_Importer::render_points_log_text
-	 */
-	public function test_import_points_logs_reversals() {
-
-		remove_action( 'publish_post', 'cp_newPost' );
-		remove_action( 'cp_comment_add', 'cp_module_post_author_points_comment_add' );
-		remove_action( 'cp_comment_remove', 'cp_module_post_author_points_comment_remove' );
-
-		update_option( 'cp_comment_points', 10 );
-		update_option( 'cp_del_comment_points', 10 );
-
-		$user_id = $this->factory->user->create();
-		$post_id = $this->factory->post->create();
-
-		$comment_id = $this->factory->comment->create(
-			array( 'user_id' => $user_id, 'comment_post_ID' => $post_id, 'comment_approved' => 0 )
-		);
-
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
-		);
-
-		$user_id_2 = $this->factory->user->create();
-		$comment_id_2 = $this->factory->comment->create(
-			array( 'user_id' => $user_id_2, 'comment_post_ID' => $post_id, 'comment_approved' => 0 )
-		);
-
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id_2, 'comment_approved' => 1 )
-		);
-
-		// Now reverse the two transactions.
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
-		);
-
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id_2, 'comment_approved' => 0 )
-		);
-
-		$this->do_points_import( 'logs' );
-
-		$query = new WordPoints_Points_Logs_Query(
-			array( 'order_by' => 'id', 'order' => 'ASC' )
-		);
-
-		$logs = $query->get();
-
-		$this->assertCount( 6, $logs );
-
-		// The first log will be for when the first user was created, so we skip it.
-		$log = $logs[1];
-
-		$this->assertSame( (string) $user_id, $log->user_id );
-		$this->assertSame( '10', $log->points );
-		$this->assertSame( 'cubepoints-comment', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[4]->id, wordpoints_get_points_log_meta( $log->id, 'auto_reversed', true ) );
-
-		// The third log is for when the second user was created, so we skip it, too.
-		$log = $logs[3];
-
-		$this->assertSame( (string) $user_id_2, $log->user_id );
-		$this->assertSame( '10', $log->points );
-		$this->assertSame( 'cubepoints-comment', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[5]->id, wordpoints_get_points_log_meta( $log->id, 'auto_reversed', true ) );
-
-		$log = $logs[4];
-
-		$this->assertSame( (string) $user_id, $log->user_id );
-		$this->assertSame( '-10', $log->points );
-		$this->assertSame( 'cubepoints-comment_remove', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment_remove', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[1]->id, wordpoints_get_points_log_meta( $log->id, 'original_log_id', true ) );
-
-		$log = $logs[5];
-
-		$this->assertSame( (string) $user_id_2, $log->user_id );
-		$this->assertSame( '-10', $log->points );
-		$this->assertSame( 'cubepoints-comment_remove', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment_remove', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[3]->id, wordpoints_get_points_log_meta( $log->id, 'original_log_id', true ) );
-	}
-
-	/**
-	 * Test that ranks are imported.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_ranks
-	 */
-	public function test_ranks_import() {
-
-		update_option(
-			'cp_module_ranks_data'
-			, array( 0 => 'Newbie', 1000 => 'Biggie', 5000 => 'Oldie' )
-		);
-
-		wordpoints_register_points_ranks();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$this->importer->do_import(
-			array(
-				'ranks' => array(
-					'ranks' => '1',
-					'_data' => array( 'rank_group' => 'points_type-points' ),
-				),
-			)
-			, $feedback
-		);
-
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['success'] );
-
-		$group = WordPoints_Rank_Groups::get_group( 'points_type-points' );
-
-		$base_rank = wordpoints_get_rank( $group->get_rank( 0 ) );
-		$this->assertSame( 'base', $base_rank->type );
-		$this->assertSame( 'Newbie', $base_rank->name );
-
-		$second_rank = wordpoints_get_rank( $group->get_rank( 1 ) );
-		$this->assertSame( '1000', $second_rank->points );
-		$this->assertSame( 'Biggie', $second_rank->name );
-
-		$third_rank = wordpoints_get_rank( $group->get_rank( 2 ) );
-		$this->assertSame( '5000', $third_rank->points );
-		$this->assertSame( 'Oldie', $third_rank->name );
-	}
-
-	/**
-	 * Test that there is an error if there are no ranks import.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @covers WordPoints_CubePoints_Importer::import_ranks
-	 */
-	public function test_error_if_no_ranks_to_import() {
-
-		wordpoints_register_points_ranks();
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$this->importer->do_import(
-			array(
-				'ranks' => array(
-					'ranks' => '1',
-					'_data' => array( 'rank_group' => 'points_type-points' ),
-				),
-			)
-			, $feedback
-		);
-
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['error'] );
-
-	}
-
-	//
-	// Helpers.
-	//
-
-	/**
-	 * Do the import for the points settings.
-	 *
-	 * @since 1.1.0
-	 *
-	 * @param string $type The type of points import.
-	 */
-	protected function do_points_import( $type ) {
-
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$this->importer->do_import(
-			array(
-				'points' => array(
-					$type => '1',
-					'_data' => array( 'points_type' => 'points' ),
-				),
-			)
-			, $feedback
-		);
-
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['success'] );
-	}
-
-	/**
-	 * Assert that a hook was imported.
-	 *
-	 * Actually just checks that the hook exists.
-	 *
-	 * @since 1.1.0
-	 * @since 1.2.0 Now just accepts a single parameter, $settings.
-	 *
-	 * @param array $settings The expected reaction settings.
-	 */
-	protected function assertHookImported( $settings ) {
-
-		$reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
-
-		$reactions = $reaction_store->get_reactions_to_event( $settings['event'] );
-
-		$this->assertNotEmpty( $reactions );
-
-		foreach ( $reactions as $reaction ) {
-			if ( $settings === $reaction->get_all_meta() ) {
-				$this->assertSame( $settings, $reaction->get_all_meta() );
-				return;
-			}
-		}
-
-		$this->assertSameSetsWithIndex( $settings, $reaction->get_all_meta() );
-	}
+        );
+
+        // Don't go all the way yet.
+        $updated = $wpdb->update(
+            $wpdb->wordpoints_points_logs
+            , array( 'date' => date( 'Y-m-d H:i:s', current_time( 'timestamp', true ) - DAY_IN_SECONDS + HOUR_IN_SECONDS ) )
+            , array( 'id' => $id )
+            , array( '%s' )
+            , array( '%d' )
+        );
+
+        $this->assertSame( 1, $updated );
+
+        // The periods cache will still hold the old date.
+        $this->flush_cache();
+
+        wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
+
+        // Points should have been awarded again yet.
+        $this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
+
+        // This time go all the way.
+        $updated = $wpdb->update(
+            $wpdb->wordpoints_points_logs
+            , array( 'date' => date( 'Y-m-d H:i:s', current_time( 'timestamp', true ) - DAY_IN_SECONDS - 1 ) )
+            , array( 'id' => $id )
+            , array( '%s' )
+            , array( '%d' )
+        );
+
+        $this->assertSame( 1, $updated );
+
+        // The periods cache will still hold the old date.
+        $this->flush_cache();
+
+        wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
+
+        // Points should have been awarded again.
+        $this->assertSame( 160, wordpoints_get_points( $user_id, 'points' ) );
+    }
+
+    /**
+     * Test the imported periods when the site has a positive GMT offset.
+     *
+     * @since 1.2.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_settings
+     * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
+     */
+    public function test_import_periods_positive_gmt_offset() {
+
+        update_option( 'gmt_offset', 5 );
+
+        $this->test_import_periodic_points_respect_old_periods();
+    }
+
+    /**
+     * Test the imported periods when the site has a negative GMT offset.
+     *
+     * @since 1.2.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_settings
+     * @covers WordPoints_CubePoints_Importer::import_daily_points_hook
+     */
+    public function test_import_periods_negative_gmt_offset() {
+
+        update_option( 'gmt_offset', -5 );
+
+        $this->test_import_periodic_points_respect_old_periods();
+    }
+
+    /**
+     * Test importing the user's points.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_user_points
+     * @covers WordPoints_CubePoints_Importer::get_next_user_points_batch
+     */
+    public function test_import_user_points() {
+
+        $user_points = array();
+
+        foreach ( array( 20, 10, 45 ) as $points ) {
+
+            $user_id = $this->factory->user->create();
+            cp_updatePoints( $user_id, $points );
+            $user_points[ $user_id ] = $points;
+        }
+
+        $this->do_points_import( 'user_points' );
+
+        foreach ( $user_points as $user_id => $points ) {
+            $this->assertSame( $points, wordpoints_get_points( $user_id, 'points' ) );
+        }
+    }
+
+    /**
+     * Test importing the points logs.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_logs
+     * @covers WordPoints_CubePoints_Importer::import_points_log
+     * @covers WordPoints_CubePoints_Importer::get_next_points_logs_batch
+     * @covers WordPoints_CubePoints_Importer::render_points_log_text
+     */
+    public function test_import_points_logs() {
+
+        remove_action( 'publish_post', 'cp_newPost' );
+
+        $user_id = $this->factory->user->create();
+        cp_points( 'misc', $user_id, 10, 'Testing things.' );
+
+        $user_id_2 = $this->factory->user->create();
+        $post_id = $this->factory->post->create();
+        cp_points( 'post', $user_id_2, 25, $post_id );
+
+        $this->do_points_import( 'logs' );
+
+        $query = new WordPoints_Points_Logs_Query( array( 'order_by' => 'id' ) );
+        $logs = $query->get();
+
+        $this->assertCount( 4, $logs );
+
+        $log = $logs[2];
+
+        $this->assertSame( (string) $user_id, $log->user_id );
+        $this->assertSame( '10', $log->points );
+        $this->assertSame( 'Testing things.', $log->text );
+        $this->assertSame( 'cubepoints-misc', $log->log_type );
+        $this->assertSame( 'points', $log->points_type );
+        $this->assertSame( 'misc', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
+        $this->assertSame( 'Testing things.', wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+
+        $log = $logs[0];
+
+        $this->assertSame( (string) $user_id_2, $log->user_id );
+        $this->assertSame( '25', $log->points );
+        $this->assertStringMatchesFormat( 'Post on "<a href="%s">Post title %s</a>"', $log->text );
+        $this->assertSame( 'cubepoints-post', $log->log_type );
+        $this->assertSame( 'points', $log->points_type );
+        $this->assertSame( 'post', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
+        $this->assertSame( (string) $post_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+        $this->assertSame( (string) $post_id, wordpoints_get_points_log_meta( $log->id, 'post', true ) );
+    }
+
+    /**
+     * Test importing points logs that have been reversed.
+     *
+     * @since 1.2.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_points_logs
+     * @covers WordPoints_CubePoints_Importer::import_points_log
+     * @covers WordPoints_CubePoints_Importer::get_next_points_logs_batch
+     * @covers WordPoints_CubePoints_Importer::render_points_log_text
+     */
+    public function test_import_points_logs_reversals() {
+
+        remove_action( 'publish_post', 'cp_newPost' );
+        remove_action( 'cp_comment_add', 'cp_module_post_author_points_comment_add' );
+        remove_action( 'cp_comment_remove', 'cp_module_post_author_points_comment_remove' );
+
+        update_option( 'cp_comment_points', 10 );
+        update_option( 'cp_del_comment_points', 10 );
+
+        $user_id = $this->factory->user->create();
+        $post_id = $this->factory->post->create();
+
+        $comment_id = $this->factory->comment->create(
+            array( 'user_id' => $user_id, 'comment_post_ID' => $post_id, 'comment_approved' => 0 )
+        );
+
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
+        );
+
+        $user_id_2 = $this->factory->user->create();
+        $comment_id_2 = $this->factory->comment->create(
+            array( 'user_id' => $user_id_2, 'comment_post_ID' => $post_id, 'comment_approved' => 0 )
+        );
+
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id_2, 'comment_approved' => 1 )
+        );
+
+        // Now reverse the two transactions.
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
+        );
+
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id_2, 'comment_approved' => 0 )
+        );
+
+        $this->do_points_import( 'logs' );
+
+        $query = new WordPoints_Points_Logs_Query(
+            array( 'order_by' => 'id', 'order' => 'ASC' )
+        );
+
+        $logs = $query->get();
+
+        $this->assertCount( 6, $logs );
+
+        // The first log will be for when the first user was created, so we skip it.
+        $log = $logs[1];
+
+        $this->assertSame( (string) $user_id, $log->user_id );
+        $this->assertSame( '10', $log->points );
+        $this->assertSame( 'cubepoints-comment', $log->log_type );
+        $this->assertSame( 'points', $log->points_type );
+        $this->assertSame( 'comment', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
+        $this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+        $this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
+        $this->assertSame( $logs[4]->id, wordpoints_get_points_log_meta( $log->id, 'auto_reversed', true ) );
+
+        // The third log is for when the second user was created, so we skip it, too.
+        $log = $logs[3];
+
+        $this->assertSame( (string) $user_id_2, $log->user_id );
+        $this->assertSame( '10', $log->points );
+        $this->assertSame( 'cubepoints-comment', $log->log_type );
+        $this->assertSame( 'points', $log->points_type );
+        $this->assertSame( 'comment', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
+        $this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+        $this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
+        $this->assertSame( $logs[5]->id, wordpoints_get_points_log_meta( $log->id, 'auto_reversed', true ) );
+
+        $log = $logs[4];
+
+        $this->assertSame( (string) $user_id, $log->user_id );
+        $this->assertSame( '-10', $log->points );
+        $this->assertSame( 'cubepoints-comment_remove', $log->log_type );
+        $this->assertSame( 'points', $log->points_type );
+        $this->assertSame( 'comment_remove', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
+        $this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+        $this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
+        $this->assertSame( $logs[1]->id, wordpoints_get_points_log_meta( $log->id, 'original_log_id', true ) );
+
+        $log = $logs[5];
+
+        $this->assertSame( (string) $user_id_2, $log->user_id );
+        $this->assertSame( '-10', $log->points );
+        $this->assertSame( 'cubepoints-comment_remove', $log->log_type );
+        $this->assertSame( 'points', $log->points_type );
+        $this->assertSame( 'comment_remove', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
+        $this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+        $this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
+        $this->assertSame( $logs[3]->id, wordpoints_get_points_log_meta( $log->id, 'original_log_id', true ) );
+    }
+
+    /**
+     * Test that ranks are imported.
+     *
+     * @since 1.1.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_ranks
+     */
+    public function test_ranks_import() {
+
+        update_option(
+            'cp_module_ranks_data'
+            , array( 0 => 'Newbie', 1000 => 'Biggie', 5000 => 'Oldie' )
+        );
+
+        wordpoints_register_points_ranks();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $this->importer->do_import(
+            array(
+                'ranks' => array(
+                    'ranks' => '1',
+                    '_data' => array( 'rank_group' => 'points_type-points' ),
+                ),
+            )
+            , $feedback
+        );
+
+        $this->assertCount( 4, $feedback->messages['info'] );
+        $this->assertCount( 1, $feedback->messages['success'] );
+
+        $group = WordPoints_Rank_Groups::get_group( 'points_type-points' );
+
+        $base_rank = wordpoints_get_rank( $group->get_rank( 0 ) );
+        $this->assertSame( 'base', $base_rank->type );
+        $this->assertSame( 'Newbie', $base_rank->name );
+
+        $second_rank = wordpoints_get_rank( $group->get_rank( 1 ) );
+        $this->assertSame( '1000', $second_rank->points );
+        $this->assertSame( 'Biggie', $second_rank->name );
+
+        $third_rank = wordpoints_get_rank( $group->get_rank( 2 ) );
+        $this->assertSame( '5000', $third_rank->points );
+        $this->assertSame( 'Oldie', $third_rank->name );
+    }
+
+    /**
+     * Test that there is an error if there are no ranks import.
+     *
+     * @since 1.1.0
+     *
+     * @covers WordPoints_CubePoints_Importer::import_ranks
+     */
+    public function test_error_if_no_ranks_to_import() {
+
+        wordpoints_register_points_ranks();
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $this->importer->do_import(
+            array(
+                'ranks' => array(
+                    'ranks' => '1',
+                    '_data' => array( 'rank_group' => 'points_type-points' ),
+                ),
+            )
+            , $feedback
+        );
+
+        $this->assertCount( 4, $feedback->messages['info'] );
+        $this->assertCount( 1, $feedback->messages['error'] );
+
+    }
+
+    //
+    // Helpers.
+    //
+
+    /**
+     * Do the import for the points settings.
+     *
+     * @since 1.1.0
+     *
+     * @param string $type The type of points import.
+     */
+    protected function do_points_import( $type ) {
+
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $this->importer->do_import(
+            array(
+                'points' => array(
+                    $type => '1',
+                    '_data' => array( 'points_type' => 'points' ),
+                ),
+            )
+            , $feedback
+        );
+
+        $this->assertCount( 4, $feedback->messages['info'] );
+        $this->assertCount( 1, $feedback->messages['success'] );
+    }
+
+    /**
+     * Assert that a hook was imported.
+     *
+     * Actually just checks that the hook exists.
+     *
+     * @since 1.1.0
+     * @since 1.2.0 Now just accepts a single parameter, $settings.
+     *
+     * @param array $settings The expected reaction settings.
+     */
+    protected function assertHookImported( $settings ) {
+
+        $reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+
+        $reactions = $reaction_store->get_reactions_to_event( $settings['event'] );
+
+        $this->assertNotEmpty( $reactions );
+
+        foreach ( $reactions as $reaction ) {
+            if ( $settings === $reaction->get_all_meta() ) {
+                $this->assertSame( $settings, $reaction->get_all_meta() );
+                return;
+            }
+        }
+
+        $this->assertSameSetsWithIndex( $settings, $reaction->get_all_meta() );
+    }
 }
 
 // EOF

Spacing +199 added lines, -199 removed lines

@@ -37,10 +37,10 @@ 
 		// These are usually inactive by default. We activate them in the tests
 		// bootstrap so that they will be fully loaded, but deactivate them here to
 		// restore default behavior.
-		cp_module_activation_set( 'post_author_points', false );
-		cp_module_activation_set( 'dailypoints', false );
+		cp_module_activation_set('post_author_points', false);
+		cp_module_activation_set('dailypoints', false);
 
-		$this->importer = new WordPoints_CubePoints_Importer( 'Test CubePoints' );
+		$this->importer = new WordPoints_CubePoints_Importer('Test CubePoints');
 	}
 
 	/**
@@ -48,8 +48,8 @@ 
 	 */
 	public function tearDown() {
 
-		WordPoints_Rank_Groups::deregister_group( 'points_type-points' );
-		WordPoints_Rank_Types::deregister_type( 'points-points' );
+		WordPoints_Rank_Groups::deregister_group('points_type-points');
+		WordPoints_Rank_Types::deregister_type('points-points');
 
 		parent::tearDown();
 	}
@@ -63,7 +63,7 @@ 
 	 */
 	public function test_is_cubepoints_installed() {
 
-		$this->assertTrue( $this->importer->is_cubepoints_installed() );
+		$this->assertTrue($this->importer->is_cubepoints_installed());
 	}
 
 	/**
@@ -75,9 +75,9 @@ 
 	 */
 	public function test_is_cubepoints_not_installed() {
 
-		delete_option( 'cp_db_version' );
+		delete_option('cp_db_version');
 
-		$this->assertFalse( $this->importer->is_cubepoints_installed() );
+		$this->assertFalse($this->importer->is_cubepoints_installed());
 	}
 
 	/**
@@ -89,7 +89,7 @@ 
 	 */
 	public function test_is_cubepoints_is_available() {
 
-		$this->assertTrue( $this->importer->is_available() );
+		$this->assertTrue($this->importer->is_available());
 	}
 
 	/**
@@ -101,9 +101,9 @@ 
 	 */
 	public function test_is_cubepoints_not_available() {
 
-		delete_option( 'cp_db_version' );
+		delete_option('cp_db_version');
 
-		$this->assertWPError( $this->importer->is_available() );
+		$this->assertWPError($this->importer->is_available());
 	}
 
 	/**
@@ -116,7 +116,7 @@ 
 	public function test_is_cubepoints_active() {
 
 		$this->assertSame(
-			function_exists( 'cp_ready' )
+			function_exists('cp_ready')
 			, $this->importer->is_cubepoints_active()
 		);
 	}
@@ -130,20 +130,20 @@ 
 	 */
 	public function test_import_excluded_users() {
 
-		$user_ids = $this->factory->user->create_many( 3 );
+		$user_ids = $this->factory->user->create_many(3);
 		$user_logins = array();
 
-		foreach ( $user_ids as $user_id ) {
-			$user_logins[] = get_userdata( $user_id )->user_login;
+		foreach ($user_ids as $user_id) {
+			$user_logins[] = get_userdata($user_id)->user_login;
 		}
 
-		update_option( 'cp_topfilter', $user_logins );
+		update_option('cp_topfilter', $user_logins);
 
-		$this->do_points_import( 'excluded_users' );
+		$this->do_points_import('excluded_users');
 
 		$this->assertSame(
 			$user_ids
-			, wordpoints_get_excluded_users( 'tests' )
+			, wordpoints_get_excluded_users('tests')
 		);
 	}
 
@@ -156,7 +156,7 @@ 
 	 */
 	public function test_import_excluded_users_none() {
 
-		delete_option( 'cp_topfilter' );
+		delete_option('cp_topfilter');
 
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 
@@ -164,13 +164,13 @@ 
 			array(
 				'points' => array(
 					'excluded_users' => '1',
-					'_data' => array( 'points_type' => 'points' ),
+					'_data' => array('points_type' => 'points'),
 				),
 			)
 			, $feedback
 		);
 
-		$this->assertCount( 1, $feedback->messages['warning'] );
+		$this->assertCount(1, $feedback->messages['warning']);
 	}
 
 	/**
@@ -184,16 +184,16 @@ 
 	 */
 	public function test_import_points_settings() {
 
-		update_option( 'cp_comment_points',     10 );
-		update_option( 'cp_post_points',        20 );
-		update_option( 'cp_reg_points',         50 );
+		update_option('cp_comment_points', 10);
+		update_option('cp_post_points', 20);
+		update_option('cp_reg_points', 50);
 
-		$this->do_points_import( 'settings' );
+		$this->do_points_import('settings');
 
 		$this->assertHookImported(
 			array(
 				'event' => 'comment_leave\post',
-				'target' => array( 'comment\post', 'author', 'user' ),
+				'target' => array('comment\post', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 10,
 				'points_type' => 'points',
@@ -201,14 +201,14 @@ 
 				'description' => 'Commenting on a Post.',
 				'legacy_log_type' => 'cubepoints-comment',
 				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 
 		$this->assertHookImported(
 			array(
 				'event' => 'comment_leave\page',
-				'target' => array( 'comment\page', 'author', 'user' ),
+				'target' => array('comment\page', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 10,
 				'points_type' => 'points',
@@ -216,14 +216,14 @@ 
 				'description' => 'Commenting on a Page.',
 				'legacy_log_type' => 'cubepoints-comment',
 				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 
 		$this->assertHookImported(
 			array(
 				'event' => 'comment_leave\attachment',
-				'target' => array( 'comment\attachment', 'author', 'user' ),
+				'target' => array('comment\attachment', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 10,
 				'points_type' => 'points',
@@ -231,55 +231,55 @@ 
 				'description' => 'Commenting on a Media.',
 				'legacy_log_type' => 'cubepoints-comment',
 				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 
 		$this->assertHookImported(
 			array(
 				'event' => 'post_publish\post',
-				'target' => array( 'post\post', 'author', 'user' ),
+				'target' => array('post\post', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 20,
 				'points_type' => 'points',
 				'log_text' => 'Published a Post.',
 				'description' => 'Publishing a Post.',
-				'blocker' => array( 'toggle_off' => true ),
+				'blocker' => array('toggle_off' => true),
 				'legacy_log_type' => 'cubepoints-post',
 				'legacy_meta_key' => 'post',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
-				'points_legacy_repeat_blocker' => array( 'toggle_on' => true ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
+				'points_legacy_repeat_blocker' => array('toggle_on' => true),
 			)
 		);
 
-		$reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+		$reaction_store = wordpoints_hooks()->get_reaction_store('points');
 
 		$this->assertSame(
 			array()
-			, $reaction_store->get_reactions_to_event( 'post_publish\page' )
+			, $reaction_store->get_reactions_to_event('post_publish\page')
 		);
 
 		$this->assertSame(
 			array()
-			, $reaction_store->get_reactions_to_event( 'post_publish\attachment' )
+			, $reaction_store->get_reactions_to_event('post_publish\attachment')
 		);
 
 		$this->assertSame(
 			array()
-			, $reaction_store->get_reactions_to_event( 'media_upload' )
+			, $reaction_store->get_reactions_to_event('media_upload')
 		);
 
 		$this->assertHookImported(
 			array(
 				'event' => 'user_register',
-				'target' => array( 'user' ),
+				'target' => array('user'),
 				'reactor' => 'points_legacy',
 				'points' => 50,
 				'points_type' => 'points',
 				'log_text' => 'Registration.',
 				'description' => 'Registration.',
 				'legacy_log_type' => 'cubepoints-register',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 	}
@@ -293,7 +293,7 @@ 
 	 */
 	public function test_imported_post_points_hook_does_not_refire() {
 
-		update_option( 'cp_post_points', 20 );
+		update_option('cp_post_points', 20);
 
 		$user_id = $this->factory->user->create();
 		$post_id = $this->factory->post->create(
@@ -303,32 +303,32 @@ 
 			)
 		);
 
-		$this->assertSame( '120', cp_getPoints( $user_id ) );
+		$this->assertSame('120', cp_getPoints($user_id));
 
 		$this->factory->post->update_object(
 			$post_id
-			, array( 'post_status' => 'draft' )
+			, array('post_status' => 'draft')
 		);
 
-		$this->assertSame( '120', cp_getPoints( $user_id ) );
+		$this->assertSame('120', cp_getPoints($user_id));
 
-		$this->do_points_import( 'settings' );
-		$this->do_points_import( 'user_points' );
-		$this->do_points_import( 'logs' );
+		$this->do_points_import('settings');
+		$this->do_points_import('user_points');
+		$this->do_points_import('logs');
 
 		$this->assertSame(
 			120
-			, wordpoints_get_points( $user_id, 'points' )
+			, wordpoints_get_points($user_id, 'points')
 		);
 
 		$this->factory->post->update_object(
 			$post_id
-			, array( 'post_status' => 'publish' )
+			, array('post_status' => 'publish')
 		);
 
 		$this->assertSame(
 			120
-			, wordpoints_get_points( $user_id, 'points' )
+			, wordpoints_get_points($user_id, 'points')
 		);
 	}
 
@@ -342,16 +342,16 @@ 
 	 */
 	public function test_import_post_author_points() {
 
-		cp_module_activation_set( 'post_author_points', 'active' );
+		cp_module_activation_set('post_author_points', 'active');
 
-		update_option( 'cp_post_author_points', 15 );
+		update_option('cp_post_author_points', 15);
 
-		$this->do_points_import( 'settings' );
+		$this->do_points_import('settings');
 
 		$this->assertHookImported(
 			array(
 				'event' => 'comment_leave\post',
-				'target' => array( 'comment\post', 'post\post', 'post\post', 'author', 'user' ),
+				'target' => array('comment\post', 'post\post', 'post\post', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 15,
 				'points_type' => 'points',
@@ -359,14 +359,14 @@ 
 				'description' => 'Receiving a comment on a Post.',
 				'legacy_log_type' => 'cubepoints-post_author',
 				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 
 		$this->assertHookImported(
 			array(
 				'event' => 'comment_leave\page',
-				'target' => array( 'comment\page', 'post\page', 'post\page', 'author', 'user' ),
+				'target' => array('comment\page', 'post\page', 'post\page', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 15,
 				'points_type' => 'points',
@@ -374,14 +374,14 @@ 
 				'description' => 'Receiving a comment on a Page.',
 				'legacy_log_type' => 'cubepoints-post_author',
 				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 
 		$this->assertHookImported(
 			array(
 				'event' => 'comment_leave\attachment',
-				'target' => array( 'comment\attachment', 'post\attachment', 'post\attachment', 'author', 'user' ),
+				'target' => array('comment\attachment', 'post\attachment', 'post\attachment', 'author', 'user'),
 				'reactor' => 'points_legacy',
 				'points' => 15,
 				'points_type' => 'points',
@@ -389,7 +389,7 @@ 
 				'description' => 'Receiving a comment on a Media.',
 				'legacy_log_type' => 'cubepoints-post_author',
 				'legacy_meta_key' => 'comment',
-				'points_legacy_reversals' => array( 'toggle_off' => 'toggle_on' ),
+				'points_legacy_reversals' => array('toggle_off' => 'toggle_on'),
 			)
 		);
 	}
@@ -404,17 +404,17 @@ 
 	 */
 	public function test_import_periodic_points() {
 
-		cp_module_activation_set( 'dailypoints', 'active' );
+		cp_module_activation_set('dailypoints', 'active');
 
-		update_option( 'cp_module_dailypoints_points', 30 );
-		update_option( 'cp_module_dailypoints_time', DAY_IN_SECONDS );
+		update_option('cp_module_dailypoints_points', 30);
+		update_option('cp_module_dailypoints_time', DAY_IN_SECONDS);
 
-		$this->do_points_import( 'settings' );
+		$this->do_points_import('settings');
 
 		$this->assertHookImported(
 			array(
 				'event' => 'user_visit',
-				'target' => array( 'current:user' ),
+				'target' => array('current:user'),
 				'reactor' => 'points_legacy',
 				'points' => 30,
 				'points_type' => 'points',
@@ -424,7 +424,7 @@ 
 					'fire' => array(
 						array(
 							'length' => DAY_IN_SECONDS,
-							'args' => array( array( 'current:user' ) ),
+							'args' => array(array('current:user')),
 							'relative' => true,
 						),
 					),
@@ -445,39 +445,39 @@ 
 	 */
 	public function test_import_periodic_points_respect_old_periods() {
 
-		if ( version_compare( $GLOBALS['wp_version'], '4.5', '>=' ) ) {
-			$this->setExpectedDeprecated( 'get_currentuserinfo' );
+		if (version_compare($GLOBALS['wp_version'], '4.5', '>=')) {
+			$this->setExpectedDeprecated('get_currentuserinfo');
 		}
 
-		cp_module_activation_set( 'dailypoints', 'active' );
+		cp_module_activation_set('dailypoints', 'active');
 
-		update_option( 'cp_module_dailypoints_points', 30 );
-		update_option( 'cp_module_dailypoints_time', DAY_IN_SECONDS );
+		update_option('cp_module_dailypoints_points', 30);
+		update_option('cp_module_dailypoints_time', DAY_IN_SECONDS);
 
 		$user_id = $this->factory->user->create();
 
-		wp_set_current_user( $user_id );
+		wp_set_current_user($user_id);
 
-		$this->assertSame( '100', cp_getPoints( $user_id ) );
+		$this->assertSame('100', cp_getPoints($user_id));
 
 		cp_module_dailypoints_checkTimer();
 
-		$this->assertSame( '130', cp_getPoints( $user_id ) );
+		$this->assertSame('130', cp_getPoints($user_id));
 
 		// Running again shouldn't hit again.
 		cp_module_dailypoints_checkTimer();
 
-		$this->assertSame( '130', cp_getPoints( $user_id ) );
+		$this->assertSame('130', cp_getPoints($user_id));
 
-		$this->do_points_import( 'settings' );
-		$this->do_points_import( 'user_points' );
-		$this->do_points_import( 'logs' );
+		$this->do_points_import('settings');
+		$this->do_points_import('user_points');
+		$this->do_points_import('logs');
 
-		$this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
+		$this->assertSame(130, wordpoints_get_points($user_id, 'points'));
 
-		wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
+		wordpoints_hooks()->get_sub_app('router')->{'wp,10'}();
 
-		$this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
+		$this->assertSame(130, wordpoints_get_points($user_id, 'points'));
 
 		// Fast-forward and try again.
 		global $wpdb;
@@ -494,40 +494,40 @@ 
 		// Don't go all the way yet.
 		$updated = $wpdb->update(
 			$wpdb->wordpoints_points_logs
-			, array( 'date' => date( 'Y-m-d H:i:s', current_time( 'timestamp', true ) - DAY_IN_SECONDS + HOUR_IN_SECONDS ) )
-			, array( 'id' => $id )
-			, array( '%s' )
-			, array( '%d' )
+			, array('date' => date('Y-m-d H:i:s', current_time('timestamp', true) - DAY_IN_SECONDS + HOUR_IN_SECONDS))
+			, array('id' => $id)
+			, array('%s')
+			, array('%d')
 		);
 
-		$this->assertSame( 1, $updated );
+		$this->assertSame(1, $updated);
 
 		// The periods cache will still hold the old date.
 		$this->flush_cache();
 
-		wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
+		wordpoints_hooks()->get_sub_app('router')->{'wp,10'}();
 
 		// Points should have been awarded again yet.
-		$this->assertSame( 130, wordpoints_get_points( $user_id, 'points' ) );
+		$this->assertSame(130, wordpoints_get_points($user_id, 'points'));
 
 		// This time go all the way.
 		$updated = $wpdb->update(
 			$wpdb->wordpoints_points_logs
-			, array( 'date' => date( 'Y-m-d H:i:s', current_time( 'timestamp', true ) - DAY_IN_SECONDS - 1 ) )
-			, array( 'id' => $id )
-			, array( '%s' )
-			, array( '%d' )
+			, array('date' => date('Y-m-d H:i:s', current_time('timestamp', true) - DAY_IN_SECONDS - 1))
+			, array('id' => $id)
+			, array('%s')
+			, array('%d')
 		);
 
-		$this->assertSame( 1, $updated );
+		$this->assertSame(1, $updated);
 
 		// The periods cache will still hold the old date.
 		$this->flush_cache();
 
-		wordpoints_hooks()->get_sub_app( 'router' )->{'wp,10'}();
+		wordpoints_hooks()->get_sub_app('router')->{'wp,10'}();
 
 		// Points should have been awarded again.
-		$this->assertSame( 160, wordpoints_get_points( $user_id, 'points' ) );
+		$this->assertSame(160, wordpoints_get_points($user_id, 'points'));
 	}
 
 	/**
@@ -540,7 +540,7 @@ 
 	 */
 	public function test_import_periods_positive_gmt_offset() {
 
-		update_option( 'gmt_offset', 5 );
+		update_option('gmt_offset', 5);
 
 		$this->test_import_periodic_points_respect_old_periods();
 	}
@@ -555,7 +555,7 @@ 
 	 */
 	public function test_import_periods_negative_gmt_offset() {
 
-		update_option( 'gmt_offset', -5 );
+		update_option('gmt_offset', -5);
 
 		$this->test_import_periodic_points_respect_old_periods();
 	}
@@ -572,17 +572,17 @@ 
 
 		$user_points = array();
 
-		foreach ( array( 20, 10, 45 ) as $points ) {
+		foreach (array(20, 10, 45) as $points) {
 
 			$user_id = $this->factory->user->create();
-			cp_updatePoints( $user_id, $points );
-			$user_points[ $user_id ] = $points;
+			cp_updatePoints($user_id, $points);
+			$user_points[$user_id] = $points;
 		}
 
-		$this->do_points_import( 'user_points' );
+		$this->do_points_import('user_points');
 
-		foreach ( $user_points as $user_id => $points ) {
-			$this->assertSame( $points, wordpoints_get_points( $user_id, 'points' ) );
+		foreach ($user_points as $user_id => $points) {
+			$this->assertSame($points, wordpoints_get_points($user_id, 'points'));
 		}
 	}
 
@@ -598,42 +598,42 @@ 
 	 */
 	public function test_import_points_logs() {
 
-		remove_action( 'publish_post', 'cp_newPost' );
+		remove_action('publish_post', 'cp_newPost');
 
 		$user_id = $this->factory->user->create();
-		cp_points( 'misc', $user_id, 10, 'Testing things.' );
+		cp_points('misc', $user_id, 10, 'Testing things.');
 
 		$user_id_2 = $this->factory->user->create();
 		$post_id = $this->factory->post->create();
-		cp_points( 'post', $user_id_2, 25, $post_id );
+		cp_points('post', $user_id_2, 25, $post_id);
 
-		$this->do_points_import( 'logs' );
+		$this->do_points_import('logs');
 
-		$query = new WordPoints_Points_Logs_Query( array( 'order_by' => 'id' ) );
+		$query = new WordPoints_Points_Logs_Query(array('order_by' => 'id'));
 		$logs = $query->get();
 
-		$this->assertCount( 4, $logs );
+		$this->assertCount(4, $logs);
 
 		$log = $logs[2];
 
-		$this->assertSame( (string) $user_id, $log->user_id );
-		$this->assertSame( '10', $log->points );
-		$this->assertSame( 'Testing things.', $log->text );
-		$this->assertSame( 'cubepoints-misc', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'misc', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( 'Testing things.', wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
+		$this->assertSame((string) $user_id, $log->user_id);
+		$this->assertSame('10', $log->points);
+		$this->assertSame('Testing things.', $log->text);
+		$this->assertSame('cubepoints-misc', $log->log_type);
+		$this->assertSame('points', $log->points_type);
+		$this->assertSame('misc', wordpoints_get_points_log_meta($log->id, 'cubepoints_type', true));
+		$this->assertSame('Testing things.', wordpoints_get_points_log_meta($log->id, 'cubepoints_data', true));
 
 		$log = $logs[0];
 
-		$this->assertSame( (string) $user_id_2, $log->user_id );
-		$this->assertSame( '25', $log->points );
-		$this->assertStringMatchesFormat( 'Post on "<a href="%s">Post title %s</a>"', $log->text );
-		$this->assertSame( 'cubepoints-post', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'post', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $post_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $post_id, wordpoints_get_points_log_meta( $log->id, 'post', true ) );
+		$this->assertSame((string) $user_id_2, $log->user_id);
+		$this->assertSame('25', $log->points);
+		$this->assertStringMatchesFormat('Post on "<a href="%s">Post title %s</a>"', $log->text);
+		$this->assertSame('cubepoints-post', $log->log_type);
+		$this->assertSame('points', $log->points_type);
+		$this->assertSame('post', wordpoints_get_points_log_meta($log->id, 'cubepoints_type', true));
+		$this->assertSame((string) $post_id, wordpoints_get_points_log_meta($log->id, 'cubepoints_data', true));
+		$this->assertSame((string) $post_id, wordpoints_get_points_log_meta($log->id, 'post', true));
 	}
 
 	/**
@@ -648,97 +648,97 @@ 
 	 */
 	public function test_import_points_logs_reversals() {
 
-		remove_action( 'publish_post', 'cp_newPost' );
-		remove_action( 'cp_comment_add', 'cp_module_post_author_points_comment_add' );
-		remove_action( 'cp_comment_remove', 'cp_module_post_author_points_comment_remove' );
+		remove_action('publish_post', 'cp_newPost');
+		remove_action('cp_comment_add', 'cp_module_post_author_points_comment_add');
+		remove_action('cp_comment_remove', 'cp_module_post_author_points_comment_remove');
 
-		update_option( 'cp_comment_points', 10 );
-		update_option( 'cp_del_comment_points', 10 );
+		update_option('cp_comment_points', 10);
+		update_option('cp_del_comment_points', 10);
 
 		$user_id = $this->factory->user->create();
 		$post_id = $this->factory->post->create();
 
 		$comment_id = $this->factory->comment->create(
-			array( 'user_id' => $user_id, 'comment_post_ID' => $post_id, 'comment_approved' => 0 )
+			array('user_id' => $user_id, 'comment_post_ID' => $post_id, 'comment_approved' => 0)
 		);
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
+			array('comment_ID' => $comment_id, 'comment_approved' => 1)
 		);
 
 		$user_id_2 = $this->factory->user->create();
 		$comment_id_2 = $this->factory->comment->create(
-			array( 'user_id' => $user_id_2, 'comment_post_ID' => $post_id, 'comment_approved' => 0 )
+			array('user_id' => $user_id_2, 'comment_post_ID' => $post_id, 'comment_approved' => 0)
 		);
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id_2, 'comment_approved' => 1 )
+			array('comment_ID' => $comment_id_2, 'comment_approved' => 1)
 		);
 
 		// Now reverse the two transactions.
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
+			array('comment_ID' => $comment_id, 'comment_approved' => 0)
 		);
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id_2, 'comment_approved' => 0 )
+			array('comment_ID' => $comment_id_2, 'comment_approved' => 0)
 		);
 
-		$this->do_points_import( 'logs' );
+		$this->do_points_import('logs');
 
 		$query = new WordPoints_Points_Logs_Query(
-			array( 'order_by' => 'id', 'order' => 'ASC' )
+			array('order_by' => 'id', 'order' => 'ASC')
 		);
 
 		$logs = $query->get();
 
-		$this->assertCount( 6, $logs );
+		$this->assertCount(6, $logs);
 
 		// The first log will be for when the first user was created, so we skip it.
 		$log = $logs[1];
 
-		$this->assertSame( (string) $user_id, $log->user_id );
-		$this->assertSame( '10', $log->points );
-		$this->assertSame( 'cubepoints-comment', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[4]->id, wordpoints_get_points_log_meta( $log->id, 'auto_reversed', true ) );
+		$this->assertSame((string) $user_id, $log->user_id);
+		$this->assertSame('10', $log->points);
+		$this->assertSame('cubepoints-comment', $log->log_type);
+		$this->assertSame('points', $log->points_type);
+		$this->assertSame('comment', wordpoints_get_points_log_meta($log->id, 'cubepoints_type', true));
+		$this->assertSame((string) $comment_id, wordpoints_get_points_log_meta($log->id, 'cubepoints_data', true));
+		$this->assertSame((string) $comment_id, wordpoints_get_points_log_meta($log->id, 'comment', true));
+		$this->assertSame($logs[4]->id, wordpoints_get_points_log_meta($log->id, 'auto_reversed', true));
 
 		// The third log is for when the second user was created, so we skip it, too.
 		$log = $logs[3];
 
-		$this->assertSame( (string) $user_id_2, $log->user_id );
-		$this->assertSame( '10', $log->points );
-		$this->assertSame( 'cubepoints-comment', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[5]->id, wordpoints_get_points_log_meta( $log->id, 'auto_reversed', true ) );
+		$this->assertSame((string) $user_id_2, $log->user_id);
+		$this->assertSame('10', $log->points);
+		$this->assertSame('cubepoints-comment', $log->log_type);
+		$this->assertSame('points', $log->points_type);
+		$this->assertSame('comment', wordpoints_get_points_log_meta($log->id, 'cubepoints_type', true));
+		$this->assertSame((string) $comment_id_2, wordpoints_get_points_log_meta($log->id, 'cubepoints_data', true));
+		$this->assertSame((string) $comment_id_2, wordpoints_get_points_log_meta($log->id, 'comment', true));
+		$this->assertSame($logs[5]->id, wordpoints_get_points_log_meta($log->id, 'auto_reversed', true));
 
 		$log = $logs[4];
 
-		$this->assertSame( (string) $user_id, $log->user_id );
-		$this->assertSame( '-10', $log->points );
-		$this->assertSame( 'cubepoints-comment_remove', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment_remove', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[1]->id, wordpoints_get_points_log_meta( $log->id, 'original_log_id', true ) );
+		$this->assertSame((string) $user_id, $log->user_id);
+		$this->assertSame('-10', $log->points);
+		$this->assertSame('cubepoints-comment_remove', $log->log_type);
+		$this->assertSame('points', $log->points_type);
+		$this->assertSame('comment_remove', wordpoints_get_points_log_meta($log->id, 'cubepoints_type', true));
+		$this->assertSame((string) $comment_id, wordpoints_get_points_log_meta($log->id, 'cubepoints_data', true));
+		$this->assertSame((string) $comment_id, wordpoints_get_points_log_meta($log->id, 'comment', true));
+		$this->assertSame($logs[1]->id, wordpoints_get_points_log_meta($log->id, 'original_log_id', true));
 
 		$log = $logs[5];
 
-		$this->assertSame( (string) $user_id_2, $log->user_id );
-		$this->assertSame( '-10', $log->points );
-		$this->assertSame( 'cubepoints-comment_remove', $log->log_type );
-		$this->assertSame( 'points', $log->points_type );
-		$this->assertSame( 'comment_remove', wordpoints_get_points_log_meta( $log->id, 'cubepoints_type', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'cubepoints_data', true ) );
-		$this->assertSame( (string) $comment_id_2, wordpoints_get_points_log_meta( $log->id, 'comment', true ) );
-		$this->assertSame( $logs[3]->id, wordpoints_get_points_log_meta( $log->id, 'original_log_id', true ) );
+		$this->assertSame((string) $user_id_2, $log->user_id);
+		$this->assertSame('-10', $log->points);
+		$this->assertSame('cubepoints-comment_remove', $log->log_type);
+		$this->assertSame('points', $log->points_type);
+		$this->assertSame('comment_remove', wordpoints_get_points_log_meta($log->id, 'cubepoints_type', true));
+		$this->assertSame((string) $comment_id_2, wordpoints_get_points_log_meta($log->id, 'cubepoints_data', true));
+		$this->assertSame((string) $comment_id_2, wordpoints_get_points_log_meta($log->id, 'comment', true));
+		$this->assertSame($logs[3]->id, wordpoints_get_points_log_meta($log->id, 'original_log_id', true));
 	}
 
 	/**
@@ -752,7 +752,7 @@ 
 
 		update_option(
 			'cp_module_ranks_data'
-			, array( 0 => 'Newbie', 1000 => 'Biggie', 5000 => 'Oldie' )
+			, array(0 => 'Newbie', 1000 => 'Biggie', 5000 => 'Oldie')
 		);
 
 		wordpoints_register_points_ranks();
@@ -763,28 +763,28 @@ 
 			array(
 				'ranks' => array(
 					'ranks' => '1',
-					'_data' => array( 'rank_group' => 'points_type-points' ),
+					'_data' => array('rank_group' => 'points_type-points'),
 				),
 			)
 			, $feedback
 		);
 
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['success'] );
+		$this->assertCount(4, $feedback->messages['info']);
+		$this->assertCount(1, $feedback->messages['success']);
 
-		$group = WordPoints_Rank_Groups::get_group( 'points_type-points' );
+		$group = WordPoints_Rank_Groups::get_group('points_type-points');
 
-		$base_rank = wordpoints_get_rank( $group->get_rank( 0 ) );
-		$this->assertSame( 'base', $base_rank->type );
-		$this->assertSame( 'Newbie', $base_rank->name );
+		$base_rank = wordpoints_get_rank($group->get_rank(0));
+		$this->assertSame('base', $base_rank->type);
+		$this->assertSame('Newbie', $base_rank->name);
 
-		$second_rank = wordpoints_get_rank( $group->get_rank( 1 ) );
-		$this->assertSame( '1000', $second_rank->points );
-		$this->assertSame( 'Biggie', $second_rank->name );
+		$second_rank = wordpoints_get_rank($group->get_rank(1));
+		$this->assertSame('1000', $second_rank->points);
+		$this->assertSame('Biggie', $second_rank->name);
 
-		$third_rank = wordpoints_get_rank( $group->get_rank( 2 ) );
-		$this->assertSame( '5000', $third_rank->points );
-		$this->assertSame( 'Oldie', $third_rank->name );
+		$third_rank = wordpoints_get_rank($group->get_rank(2));
+		$this->assertSame('5000', $third_rank->points);
+		$this->assertSame('Oldie', $third_rank->name);
 	}
 
 	/**
@@ -804,14 +804,14 @@ 
 			array(
 				'ranks' => array(
 					'ranks' => '1',
-					'_data' => array( 'rank_group' => 'points_type-points' ),
+					'_data' => array('rank_group' => 'points_type-points'),
 				),
 			)
 			, $feedback
 		);
 
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['error'] );
+		$this->assertCount(4, $feedback->messages['info']);
+		$this->assertCount(1, $feedback->messages['error']);
 
 	}
 
@@ -826,7 +826,7 @@ 
 	 *
 	 * @param string $type The type of points import.
 	 */
-	protected function do_points_import( $type ) {
+	protected function do_points_import($type) {
 
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 
@@ -834,14 +834,14 @@ 
 			array(
 				'points' => array(
 					$type => '1',
-					'_data' => array( 'points_type' => 'points' ),
+					'_data' => array('points_type' => 'points'),
 				),
 			)
 			, $feedback
 		);
 
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['success'] );
+		$this->assertCount(4, $feedback->messages['info']);
+		$this->assertCount(1, $feedback->messages['success']);
 	}
 
 	/**
@@ -854,22 +854,22 @@ 
 	 *
 	 * @param array $settings The expected reaction settings.
 	 */
-	protected function assertHookImported( $settings ) {
+	protected function assertHookImported($settings) {
 
-		$reaction_store = wordpoints_hooks()->get_reaction_store( 'points' );
+		$reaction_store = wordpoints_hooks()->get_reaction_store('points');
 
-		$reactions = $reaction_store->get_reactions_to_event( $settings['event'] );
+		$reactions = $reaction_store->get_reactions_to_event($settings['event']);
 
-		$this->assertNotEmpty( $reactions );
+		$this->assertNotEmpty($reactions);
 
-		foreach ( $reactions as $reaction ) {
-			if ( $settings === $reaction->get_all_meta() ) {
-				$this->assertSame( $settings, $reaction->get_all_meta() );
+		foreach ($reactions as $reaction) {
+			if ($settings === $reaction->get_all_meta()) {
+				$this->assertSame($settings, $reaction->get_all_meta());
 				return;
 			}
 		}
 
-		$this->assertSameSetsWithIndex( $settings, $reaction->get_all_meta() );
+		$this->assertSameSetsWithIndex($settings, $reaction->get_all_meta());
 	}
 }
 

tests/phpunit/tests/importers/cubepoints/hooks/post-publish.php

Indentation +22 added lines, -22 removed lines

@@ -18,37 +18,37 @@
  * @coversNothing
  */
 class WordPoints_CubePoints_Importer_Post_Publish_Hook_Test
-	extends WordPoints_Importer_Hook_UnitTestCase {
+    extends WordPoints_Importer_Hook_UnitTestCase {
 
-	/**
-	 * @since 1.2.0
-	 */
-	protected $cubepoints_option = 'cp_post_points';
+    /**
+     * @since 1.2.0
+     */
+    protected $cubepoints_option = 'cp_post_points';
 
-	/**
-	 * @since 1.2.0
-	 *
-	 * @dataProvider data_provider_types
-	 */
-	public function test( $type ) {
+    /**
+     * @since 1.2.0
+     *
+     * @dataProvider data_provider_types
+     */
+    public function test( $type ) {
 
-		$this->before( $type );
+        $this->before( $type );
 
-		$user_id = $this->factory->user->create();
-		$post_id = $this->factory->post->create(
-			array( 'post_author' => $user_id, 'post_status' => 'publish' )
-		);
+        $user_id = $this->factory->user->create();
+        $post_id = $this->factory->post->create(
+            array( 'post_author' => $user_id, 'post_status' => 'publish' )
+        );
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+        $this->assertSame( 10, $this->get_user_points( $user_id ) );
 
-		wp_update_post( array( 'ID' => $post_id, 'post_status' => 'draft' ) );
+        wp_update_post( array( 'ID' => $post_id, 'post_status' => 'draft' ) );
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+        $this->assertSame( 10, $this->get_user_points( $user_id ) );
 
-		wp_delete_post( $post_id, true );
+        wp_delete_post( $post_id, true );
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
-	}
+        $this->assertSame( 10, $this->get_user_points( $user_id ) );
+    }
 }
 
 // EOF

Spacing +8 added lines, -8 removed lines

@@ -30,24 +30,24 @@
 	 *
 	 * @dataProvider data_provider_types
 	 */
-	public function test( $type ) {
+	public function test($type) {
 
-		$this->before( $type );
+		$this->before($type);
 
 		$user_id = $this->factory->user->create();
 		$post_id = $this->factory->post->create(
-			array( 'post_author' => $user_id, 'post_status' => 'publish' )
+			array('post_author' => $user_id, 'post_status' => 'publish')
 		);
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+		$this->assertSame(10, $this->get_user_points($user_id));
 
-		wp_update_post( array( 'ID' => $post_id, 'post_status' => 'draft' ) );
+		wp_update_post(array('ID' => $post_id, 'post_status' => 'draft'));
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+		$this->assertSame(10, $this->get_user_points($user_id));
 
-		wp_delete_post( $post_id, true );
+		wp_delete_post($post_id, true);
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+		$this->assertSame(10, $this->get_user_points($user_id));
 	}
 }
 

tests/phpunit/tests/importers/cubepoints/hooks/comment-receive.php

Indentation +35 added lines, -35 removed lines

@@ -18,53 +18,53 @@
  * @coversNothing
  */
 class WordPoints_CubePoints_Importer_Comment_Receive_Hook_Test
-	extends WordPoints_Importer_Hook_UnitTestCase {
+    extends WordPoints_Importer_Hook_UnitTestCase {
 
-	/**
-	 * @since 1.2.0
-	 */
-	protected $cubepoints_option = 'cp_post_author_points';
+    /**
+     * @since 1.2.0
+     */
+    protected $cubepoints_option = 'cp_post_author_points';
 
-	/**
-	 * @since 1.2.0
-	 *
-	 * @dataProvider data_provider_types
-	 */
-	public function test( $type ) {
+    /**
+     * @since 1.2.0
+     *
+     * @dataProvider data_provider_types
+     */
+    public function test( $type ) {
 
-		cp_module_activation_set( 'post_author_points', 'active' );
+        cp_module_activation_set( 'post_author_points', 'active' );
 
-		update_option( 'cp_post_points', 0 );
+        update_option( 'cp_post_points', 0 );
 
-		$this->before( $type );
+        $this->before( $type );
 
-		$user_id = $this->factory->user->create();
-		$post_id = $this->factory->post->create(
-			array( 'post_author' => $user_id )
-		);
+        $user_id = $this->factory->user->create();
+        $post_id = $this->factory->post->create(
+            array( 'post_author' => $user_id )
+        );
 
-		$comment_id = $this->factory->comment->create(
-			array(
-				'comment_post_ID' => $post_id,
-				'comment_approved' => 0,
-				'user_id' => $this->factory->user->create(),
-			)
-		);
+        $comment_id = $this->factory->comment->create(
+            array(
+                'comment_post_ID' => $post_id,
+                'comment_approved' => 0,
+                'user_id' => $this->factory->user->create(),
+            )
+        );
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+        $this->assertSame( 0, $this->get_user_points( $user_id ) );
 
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
-		);
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
+        );
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+        $this->assertSame( 10, $this->get_user_points( $user_id ) );
 
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
-		);
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
+        );
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
-	}
+        $this->assertSame( 0, $this->get_user_points( $user_id ) );
+    }
 }
 
 // EOF

Spacing +10 added lines, -10 removed lines

@@ -30,17 +30,17 @@ 
 	 *
 	 * @dataProvider data_provider_types
 	 */
-	public function test( $type ) {
+	public function test($type) {
 
-		cp_module_activation_set( 'post_author_points', 'active' );
+		cp_module_activation_set('post_author_points', 'active');
 
-		update_option( 'cp_post_points', 0 );
+		update_option('cp_post_points', 0);
 
-		$this->before( $type );
+		$this->before($type);
 
 		$user_id = $this->factory->user->create();
 		$post_id = $this->factory->post->create(
-			array( 'post_author' => $user_id )
+			array('post_author' => $user_id)
 		);
 
 		$comment_id = $this->factory->comment->create(
@@ -51,19 +51,19 @@ 
 			)
 		);
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+		$this->assertSame(0, $this->get_user_points($user_id));
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
+			array('comment_ID' => $comment_id, 'comment_approved' => 1)
 		);
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+		$this->assertSame(10, $this->get_user_points($user_id));
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
+			array('comment_ID' => $comment_id, 'comment_approved' => 0)
 		);
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+		$this->assertSame(0, $this->get_user_points($user_id));
 	}
 }
 

tests/phpunit/tests/importers/cubepoints/hooks/user-register.php

Indentation +17 added lines, -17 removed lines

@@ -18,30 +18,30 @@
  * @coversNothing
  */
 class WordPoints_CubePoints_Importer_User_Register_Hook_Test
-	extends WordPoints_Importer_Hook_UnitTestCase {
+    extends WordPoints_Importer_Hook_UnitTestCase {
 
-	/**
-	 * @since 1.2.0
-	 */
-	protected $cubepoints_option = 'cp_reg_points';
+    /**
+     * @since 1.2.0
+     */
+    protected $cubepoints_option = 'cp_reg_points';
 
-	/**
-	 * @since 1.2.0
-	 *
-	 * @dataProvider data_provider_types
-	 */
-	public function test( $type ) {
+    /**
+     * @since 1.2.0
+     *
+     * @dataProvider data_provider_types
+     */
+    public function test( $type ) {
 
-		$this->before( $type );
+        $this->before( $type );
 
-		$user_id = $this->factory->user->create();
+        $user_id = $this->factory->user->create();
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+        $this->assertSame( 10, $this->get_user_points( $user_id ) );
 
-		self::delete_user( $user_id );
+        self::delete_user( $user_id );
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
-	}
+        $this->assertSame( 0, $this->get_user_points( $user_id ) );
+    }
 }
 
 // EOF

Spacing +5 added lines, -5 removed lines

@@ -30,17 +30,17 @@
 	 *
 	 * @dataProvider data_provider_types
 	 */
-	public function test( $type ) {
+	public function test($type) {
 
-		$this->before( $type );
+		$this->before($type);
 
 		$user_id = $this->factory->user->create();
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+		$this->assertSame(10, $this->get_user_points($user_id));
 
-		self::delete_user( $user_id );
+		self::delete_user($user_id);
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+		$this->assertSame(0, $this->get_user_points($user_id));
 	}
 }
 

tests/phpunit/tests/importers/cubepoints/hooks/comment-leave.php

Indentation +34 added lines, -34 removed lines

@@ -18,51 +18,51 @@
  * @coversNothing
  */
 class WordPoints_CubePoints_Importer_Comment_Leave_Hook_Test
-	extends WordPoints_Importer_Hook_UnitTestCase {
+    extends WordPoints_Importer_Hook_UnitTestCase {
 
-	/**
-	 * @since 1.2.0
-	 */
-	protected $cubepoints_option = 'cp_comment_points';
+    /**
+     * @since 1.2.0
+     */
+    protected $cubepoints_option = 'cp_comment_points';
 
-	/**
-	 * @since 1.2.0
-	 *
-	 * @dataProvider data_provider_types
-	 */
-	public function test( $type ) {
+    /**
+     * @since 1.2.0
+     *
+     * @dataProvider data_provider_types
+     */
+    public function test( $type ) {
 
-		$this->before( $type );
+        $this->before( $type );
 
-		update_option( 'cp_del_comment_points', 10 );
+        update_option( 'cp_del_comment_points', 10 );
 
-		$user_id = $this->factory->user->create();
-		$post_id = $this->factory->post->create(
-			array( 'post_author' => $this->factory->user->create() )
-		);
+        $user_id = $this->factory->user->create();
+        $post_id = $this->factory->post->create(
+            array( 'post_author' => $this->factory->user->create() )
+        );
 
-		$comment_id = $this->factory->comment->create(
-			array(
-				'comment_post_ID' => $post_id,
-				'comment_approved' => 0,
-				'user_id' => $user_id,
-			)
-		);
+        $comment_id = $this->factory->comment->create(
+            array(
+                'comment_post_ID' => $post_id,
+                'comment_approved' => 0,
+                'user_id' => $user_id,
+            )
+        );
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+        $this->assertSame( 0, $this->get_user_points( $user_id ) );
 
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
-		);
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
+        );
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+        $this->assertSame( 10, $this->get_user_points( $user_id ) );
 
-		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
-		);
+        wp_update_comment(
+            array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
+        );
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
-	}
+        $this->assertSame( 0, $this->get_user_points( $user_id ) );
+    }
 }
 
 // EOF

Spacing +9 added lines, -9 removed lines

@@ -30,15 +30,15 @@ 
 	 *
 	 * @dataProvider data_provider_types
 	 */
-	public function test( $type ) {
+	public function test($type) {
 
-		$this->before( $type );
+		$this->before($type);
 
-		update_option( 'cp_del_comment_points', 10 );
+		update_option('cp_del_comment_points', 10);
 
 		$user_id = $this->factory->user->create();
 		$post_id = $this->factory->post->create(
-			array( 'post_author' => $this->factory->user->create() )
+			array('post_author' => $this->factory->user->create())
 		);
 
 		$comment_id = $this->factory->comment->create(
@@ -49,19 +49,19 @@ 
 			)
 		);
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+		$this->assertSame(0, $this->get_user_points($user_id));
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 1 )
+			array('comment_ID' => $comment_id, 'comment_approved' => 1)
 		);
 
-		$this->assertSame( 10, $this->get_user_points( $user_id ) );
+		$this->assertSame(10, $this->get_user_points($user_id));
 
 		wp_update_comment(
-			array( 'comment_ID' => $comment_id, 'comment_approved' => 0 )
+			array('comment_ID' => $comment_id, 'comment_approved' => 0)
 		);
 
-		$this->assertSame( 0, $this->get_user_points( $user_id ) );
+		$this->assertSame(0, $this->get_user_points($user_id));
 	}
 }
 

tests/phpunit/includes/feedback.php

Indentation +14 added lines, -14 removed lines

@@ -14,22 +14,22 @@
  */
 class WordPoints_Importer_Tests_Feedback extends WordPoints_Importer_Feedback {
 
-	/**
-	 * The messages that have been reported.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var string[][]
-	 */
-	public $messages = array();
+    /**
+     * The messages that have been reported.
+     *
+     * @since 1.0.0
+     *
+     * @var string[][]
+     */
+    public $messages = array();
 
-	/**
-	 * @since 1.0.0
-	 */
-	protected function _send( $message, $type = 'info' ) { // @codingStandardsIgnoreLine
+    /**
+     * @since 1.0.0
+     */
+    protected function _send( $message, $type = 'info' ) { // @codingStandardsIgnoreLine
 
-		$this->messages[ $type ][] = $message;
-	}
+        $this->messages[ $type ][] = $message;
+    }
 }
 
 // EOF

Spacing +2 added lines, -2 removed lines

@@ -26,9 +26,9 @@
 	/**
 	 * @since 1.0.0
 	 */
-	protected function _send( $message, $type = 'info' ) { // @codingStandardsIgnoreLine
+	protected function _send($message, $type = 'info') { // @codingStandardsIgnoreLine
 
-		$this->messages[ $type ][] = $message;
+		$this->messages[$type][] = $message;
 	}
 }
 

tests/phpunit/includes/testcases/hook.php

Indentation +107 added lines, -107 removed lines

@@ -21,113 +21,113 @@
  * @coversNothing
  */
 abstract class WordPoints_Importer_Hook_UnitTestCase
-	extends WordPoints_PHPUnit_TestCase_Points {
-
-	/**
-	 * The option where CubePoints stores the number of points to award for this.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @var string
-	 */
-	protected $cubepoints_option;
-
-	/**
-	 * The type of hook being tested.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @var string
-	 */
-	protected $type;
-
-	/**
-	 * Test that the hook behaves properly.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @param string $type The type of test.
-	 *
-	 * @dataProvider data_provider_types
-	 */
-	abstract public function test( $type );
-
-	/**
-	 * Data provider for types of tests.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @return array
-	 */
-	public function data_provider_types() {
-		return array(
-			'cubepoints' => array( 'cubepoints' ),
-			'wordpoints' => array( 'wordpoints' ),
-		);
-	}
-
-	/**
-	 * Set up before a test.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @param string $type The type of test being run.
-	 */
-	protected function before( $type ) {
-
-		update_option( 'cp_reg_points', 0 );
-
-		$this->type = $type;
-
-		update_option( $this->cubepoints_option, 10 );
-
-		if ( 'wordpoints' === $this->type ) {
-			$this->do_points_import( 'settings' );
-		}
-	}
-
-	/**
-	 * Get the number of points a user has.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @param int $user_id The user ID.
-	 *
-	 * @return int The number of points the user has.
-	 */
-	protected function get_user_points( $user_id ) {
-		if ( 'cubepoints' === $this->type ) {
-			return (int) cp_getPoints( $user_id );
-		} else {
-			return wordpoints_get_points( $user_id, 'points' );
-		}
-	}
-
-	/**
-	 * Do the import for the points settings.
-	 *
-	 * @since 1.2.0
-	 *
-	 * @param string $type The type of points import.
-	 */
-	protected function do_points_import( $type ) {
-
-		$importer = new WordPoints_CubePoints_Importer( 'Test CubePoints' );
-		$feedback = new WordPoints_Importer_Tests_Feedback();
-
-		$importer->do_import(
-			array(
-				'points' => array(
-					$type => '1',
-					'_data' => array( 'points_type' => 'points' ),
-				),
-			)
-			, $feedback
-		);
-
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['success'] );
-	}
+    extends WordPoints_PHPUnit_TestCase_Points {
+
+    /**
+     * The option where CubePoints stores the number of points to award for this.
+     *
+     * @since 1.2.0
+     *
+     * @var string
+     */
+    protected $cubepoints_option;
+
+    /**
+     * The type of hook being tested.
+     *
+     * @since 1.2.0
+     *
+     * @var string
+     */
+    protected $type;
+
+    /**
+     * Test that the hook behaves properly.
+     *
+     * @since 1.2.0
+     *
+     * @param string $type The type of test.
+     *
+     * @dataProvider data_provider_types
+     */
+    abstract public function test( $type );
+
+    /**
+     * Data provider for types of tests.
+     *
+     * @since 1.2.0
+     *
+     * @return array
+     */
+    public function data_provider_types() {
+        return array(
+            'cubepoints' => array( 'cubepoints' ),
+            'wordpoints' => array( 'wordpoints' ),
+        );
+    }
+
+    /**
+     * Set up before a test.
+     *
+     * @since 1.2.0
+     *
+     * @param string $type The type of test being run.
+     */
+    protected function before( $type ) {
+
+        update_option( 'cp_reg_points', 0 );
+
+        $this->type = $type;
+
+        update_option( $this->cubepoints_option, 10 );
+
+        if ( 'wordpoints' === $this->type ) {
+            $this->do_points_import( 'settings' );
+        }
+    }
+
+    /**
+     * Get the number of points a user has.
+     *
+     * @since 1.2.0
+     *
+     * @param int $user_id The user ID.
+     *
+     * @return int The number of points the user has.
+     */
+    protected function get_user_points( $user_id ) {
+        if ( 'cubepoints' === $this->type ) {
+            return (int) cp_getPoints( $user_id );
+        } else {
+            return wordpoints_get_points( $user_id, 'points' );
+        }
+    }
+
+    /**
+     * Do the import for the points settings.
+     *
+     * @since 1.2.0
+     *
+     * @param string $type The type of points import.
+     */
+    protected function do_points_import( $type ) {
+
+        $importer = new WordPoints_CubePoints_Importer( 'Test CubePoints' );
+        $feedback = new WordPoints_Importer_Tests_Feedback();
+
+        $importer->do_import(
+            array(
+                'points' => array(
+                    $type => '1',
+                    '_data' => array( 'points_type' => 'points' ),
+                ),
+            )
+            , $feedback
+        );
+
+        $this->assertCount( 4, $feedback->messages['info'] );
+        $this->assertCount( 1, $feedback->messages['success'] );
+    }
 }
 
 // EOF

Spacing +17 added lines, -17 removed lines

@@ -50,7 +50,7 @@ 
 	 *
 	 * @dataProvider data_provider_types
 	 */
-	abstract public function test( $type );
+	abstract public function test($type);
 
 	/**
 	 * Data provider for types of tests.
@@ -61,8 +61,8 @@ 
 	 */
 	public function data_provider_types() {
 		return array(
-			'cubepoints' => array( 'cubepoints' ),
-			'wordpoints' => array( 'wordpoints' ),
+			'cubepoints' => array('cubepoints'),
+			'wordpoints' => array('wordpoints'),
 		);
 	}
 
@@ -73,16 +73,16 @@ 
 	 *
 	 * @param string $type The type of test being run.
 	 */
-	protected function before( $type ) {
+	protected function before($type) {
 
-		update_option( 'cp_reg_points', 0 );
+		update_option('cp_reg_points', 0);
 
 		$this->type = $type;
 
-		update_option( $this->cubepoints_option, 10 );
+		update_option($this->cubepoints_option, 10);
 
-		if ( 'wordpoints' === $this->type ) {
-			$this->do_points_import( 'settings' );
+		if ('wordpoints' === $this->type) {
+			$this->do_points_import('settings');
 		}
 	}
 
@@ -95,11 +95,11 @@ 
 	 *
 	 * @return int The number of points the user has.
 	 */
-	protected function get_user_points( $user_id ) {
-		if ( 'cubepoints' === $this->type ) {
-			return (int) cp_getPoints( $user_id );
+	protected function get_user_points($user_id) {
+		if ('cubepoints' === $this->type) {
+			return (int) cp_getPoints($user_id);
 		} else {
-			return wordpoints_get_points( $user_id, 'points' );
+			return wordpoints_get_points($user_id, 'points');
 		}
 	}
 
@@ -110,23 +110,23 @@ 
 	 *
 	 * @param string $type The type of points import.
 	 */
-	protected function do_points_import( $type ) {
+	protected function do_points_import($type) {
 
-		$importer = new WordPoints_CubePoints_Importer( 'Test CubePoints' );
+		$importer = new WordPoints_CubePoints_Importer('Test CubePoints');
 		$feedback = new WordPoints_Importer_Tests_Feedback();
 
 		$importer->do_import(
 			array(
 				'points' => array(
 					$type => '1',
-					'_data' => array( 'points_type' => 'points' ),
+					'_data' => array('points_type' => 'points'),
 				),
 			)
 			, $feedback
 		);
 
-		$this->assertCount( 4, $feedback->messages['info'] );
-		$this->assertCount( 1, $feedback->messages['success'] );
+		$this->assertCount(4, $feedback->messages['info']);
+		$this->assertCount(1, $feedback->messages['success']);
 	}
 }