WordPoints / importer

tests/phpunit/tests/admin-functions.php

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->assertEmpty( $feedback->messages );
+		$this->assertTrue($valid);
+		$this->assertEmpty($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->assertEmpty( $feedback->messages );
+		$this->assertFalse($valid);
+		$this->assertEmpty($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->assertEmpty( $feedback->messages );
+		$this->assertTrue($valid);
+		$this->assertEmpty($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->assertEmpty( $feedback->messages );
+		$this->assertFalse($valid);
+		$this->assertEmpty($feedback->messages);
 	}
 }
 

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

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->assertEquals( 10, $this->get_user_points( $user_id ) );
+		$this->assertEquals(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->assertEquals( 10, $this->get_user_points( $user_id ) );
+		$this->assertEquals(10, $this->get_user_points($user_id));
 
-		wp_delete_post( $post_id, true );
+		wp_delete_post($post_id, true);
 
-		$this->assertEquals( 10, $this->get_user_points( $user_id ) );
+		$this->assertEquals(10, $this->get_user_points($user_id));
 	}
 }
 

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

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->assertEquals( 0, $this->get_user_points( $user_id ) );
+		$this->assertEquals(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->assertEquals( 10, $this->get_user_points( $user_id ) );
+		$this->assertEquals(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->assertEquals( 0, $this->get_user_points( $user_id ) );
+		$this->assertEquals(0, $this->get_user_points($user_id));
 	}
 }
 

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

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->assertEquals( 0, $this->get_user_points( $user_id ) );
+		$this->assertEquals(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->assertEquals( 10, $this->get_user_points( $user_id ) );
+		$this->assertEquals(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->assertEquals( 0, $this->get_user_points( $user_id ) );
+		$this->assertEquals(0, $this->get_user_points($user_id));
 	}
 }
 

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

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->assertEquals( 10, $this->get_user_points( $user_id ) );
+		$this->assertEquals(10, $this->get_user_points($user_id));
 
-		self::delete_user( $user_id );
+		self::delete_user($user_id);
 
-		$this->assertEquals( 0, $this->get_user_points( $user_id ) );
+		$this->assertEquals(0, $this->get_user_points($user_id));
 	}
 }
 

tests/phpunit/tests/importer.php

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->assertEquals(
 			$this->importer_components['points']
-			, $this->importer->get_options_for_component( 'points' )
+			, $this->importer->get_options_for_component('points')
 		);
 	}
 
@@ -110,7 +110,7 @@ 
 
 		$this->assertEquals(
 			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->assertEmpty( $this->importer->imports );
+		$this->assertEmpty($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->assertEmpty( $this->importer->imports );
+		$this->assertEmpty($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->assertEquals(
 			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->assertEmpty( $this->importer->imports );
+		$this->assertEmpty($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->assertEmpty( $this->importer->imports );
+		$this->assertEmpty($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->assertEmpty( $this->importer->imports );
+		$this->assertEmpty($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->assertEquals(
-			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->assertEquals(
-			array( 'testing' => 1 )
+			array('testing' => 1)
 			, $this->importer->imports[0]
 		);
 	}

tests/phpunit/tests/importers.php

Spacing +18 added lines, -18 removed lines

@@ -36,7 +36,7 @@ 
 
 		WordPoints_Importers::register(
 			'test'
-			, array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
+			, array('class' => 'WordPoints_Importer_Mock', 'name' => 'Test')
 		);
 
 	}
@@ -48,17 +48,17 @@ 
 
 		$importers = WordPoints_Importers::get();
 
-		foreach ( $this->_backup_importers as $slug => $args ) {
+		foreach ($this->_backup_importers as $slug => $args) {
 
-			if ( ! isset( $importers[ $slug ] ) ) {
-				WordPoints_Importers::register( $slug, $args );
+			if (!isset($importers[$slug])) {
+				WordPoints_Importers::register($slug, $args);
 			}
 
-			unset( $importers[ $slug ] );
+			unset($importers[$slug]);
 		}
 
-		foreach ( $importers as $slug => $args ) {
-			WordPoints_Importers::deregister( $slug );
+		foreach ($importers as $slug => $args) {
+			WordPoints_Importers::deregister($slug);
 		}
 
 		parent::tearDown();
@@ -76,14 +76,14 @@ 
 
 		WordPoints_Importers::register(
 			__METHOD__
-			, array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
+			, array('class' => 'WordPoints_Importer_Mock', 'name' => 'Test')
 		);
 
 		$importers = WordPoints_Importers::get();
 
-		$this->assertArrayHasKey( 'test', $importers );
+		$this->assertArrayHasKey('test', $importers);
 		$this->assertEquals(
-			array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
+			array('class' => 'WordPoints_Importer_Mock', 'name' => 'Test')
 			, $importers['test']
 		);
 	}
@@ -98,11 +98,11 @@ 
 	 */
 	public function test_deregister() {
 
-		WordPoints_Importers::deregister( 'test' );
+		WordPoints_Importers::deregister('test');
 
 		$importers = WordPoints_Importers::get();
 
-		$this->assertArrayNotHasKey( 'test', $importers );
+		$this->assertArrayNotHasKey('test', $importers);
 	}
 
 	/**
@@ -114,7 +114,7 @@ 
 	 */
 	public function test_is_registered() {
 
-		$this->assertTrue( WordPoints_Importers::is_registered( 'test' ) );
+		$this->assertTrue(WordPoints_Importers::is_registered('test'));
 	}
 
 	/**
@@ -126,9 +126,9 @@ 
 	 */
 	public function test_is_registered_unregistered() {
 
-		WordPoints_Importers::deregister( 'test' );
+		WordPoints_Importers::deregister('test');
 
-		$this->assertFalse( WordPoints_Importers::is_registered( 'test' ) );
+		$this->assertFalse(WordPoints_Importers::is_registered('test'));
 	}
 
 	/**
@@ -140,9 +140,9 @@ 
 	 */
 	public function test_get_unregistered_importer() {
 
-		WordPoints_Importers::deregister( 'test' );
+		WordPoints_Importers::deregister('test');
 
-		$this->assertFalse( WordPoints_Importers::get_importer( 'test' ) );
+		$this->assertFalse(WordPoints_Importers::get_importer('test'));
 	}
 
 	/**
@@ -156,7 +156,7 @@ 
 
 		$this->assertInstanceOf(
 			'WordPoints_Importer_Mock'
-			, WordPoints_Importers::get_importer( 'test' )
+			, WordPoints_Importers::get_importer('test')
 		);
 	}
 

tests/phpunit/includes/mocks.php

Spacing +3 added lines, -3 removed lines

@@ -60,7 +60,7 @@ 
 	 *
 	 * @param array $settings The settings for this "component".
 	 */
-	public function do_an_import( $settings ) {
+	public function do_an_import($settings) {
 
 		$this->imports[] = $settings;
 	}
@@ -74,7 +74,7 @@ 
 	 *
 	 * @return true
 	 */
-	public function can_import( $settings ) {
+	public function can_import($settings) {
 
 		$this->can_imports[] = $settings;
 
@@ -90,7 +90,7 @@ 
 	 *
 	 * @return WP_Error An error.
 	 */
-	public function cant_import( $settings ) {
+	public function cant_import($settings) {
 
 		$this->can_imports[] = $settings;
 

tests/phpunit/includes/feedback.php

Spacing +2 added lines, -2 removed lines

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