WordPoints / importer

tests/phpunit/tests/admin-functions.php

Indentation +175 added lines, -175 removed lines

@@ -14,181 +14,181 @@
  */
 class WordPoints_Importer_Admin_Test extends WordPoints_Points_UnitTestCase {
 
-	/**
-	 * 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->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEmpty( $feedback->messages );
+    }
 }
 
 // EOF

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->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 ) );
+    }
 }
 
 // EOF

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->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 )
-		);
+        wp_update_comment(
+            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 )
-		);
+        wp_update_comment(
+            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 ) );
+    }
 }
 
 // EOF

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->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 )
-		);
+        wp_update_comment(
+            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 )
-		);
+        wp_update_comment(
+            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 ) );
+    }
 }
 
 // EOF

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->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 ) );
+    }
 }
 
 // EOF

tests/phpunit/tests/importer.php

Indentation +289 added lines, -289 removed lines

@@ -16,295 +16,295 @@
  */
 class WordPoints_Importer_Importer_Test extends WordPoints_UnitTestCase {
 
-	/**
-	 * 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->assertEquals(
-			$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->assertEquals(
-			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->assertEmpty( $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->assertEmpty( $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->assertEquals(
-			1
-			, $this->filter_was_called( 'wordpoints_import_settings_valid-points' )
-		);
-
-		// The import shouldn't have been performed.
-		$this->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEquals(
-			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->assertEquals(
-			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->assertEquals(
+            $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->assertEquals(
+            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->assertEmpty( $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->assertEmpty( $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->assertEquals(
+            1
+            , $this->filter_was_called( 'wordpoints_import_settings_valid-points' )
+        );
+
+        // The import shouldn't have been performed.
+        $this->assertEmpty( $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->assertEmpty( $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->assertEmpty( $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->assertEquals(
+            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->assertEquals(
+            array( 'testing' => 1 )
+            , $this->importer->imports[0]
+        );
+    }
 }
 
 // EOF

tests/phpunit/tests/importers.php

Indentation +143 added lines, -143 removed lines

@@ -16,149 +16,149 @@
  */
 class WordPoints_Importers_Test extends WP_UnitTestCase {
 
-	/**
-	 * Backup of the importers when the test begins.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var array[]
-	 */
-	protected $_backup_importers;
-
-	/**
-	 * @since 1.0.0
-	 */
-	public function setUp() {
-
-		parent::setUp();
-
-		$this->_backup_importers = WordPoints_Importers::get();
-
-		WordPoints_Importers::register(
-			'test'
-			, array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
-		);
-
-	}
-
-	/**
-	 * @since 1.0.0
-	 */
-	public function tearDown() {
-
-		$importers = WordPoints_Importers::get();
-
-		foreach ( $this->_backup_importers as $slug => $args ) {
-
-			if ( ! isset( $importers[ $slug ] ) ) {
-				WordPoints_Importers::register( $slug, $args );
-			}
-
-			unset( $importers[ $slug ] );
-		}
-
-		foreach ( $importers as $slug => $args ) {
-			WordPoints_Importers::deregister( $slug );
-		}
-
-		parent::tearDown();
-	}
-
-	/**
-	 * Test registration.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importers::register
-	 * @covers WordPoints_Importers::get
-	 */
-	public function test_register() {
-
-		WordPoints_Importers::register(
-			__METHOD__
-			, array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
-		);
-
-		$importers = WordPoints_Importers::get();
-
-		$this->assertArrayHasKey( 'test', $importers );
-		$this->assertEquals(
-			array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
-			, $importers['test']
-		);
-	}
-
-	/**
-	 * Test that deregister() deregisters the importer.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importers::register
-	 * @covers WordPoints_Importers::get
-	 */
-	public function test_deregister() {
-
-		WordPoints_Importers::deregister( 'test' );
-
-		$importers = WordPoints_Importers::get();
-
-		$this->assertArrayNotHasKey( 'test', $importers );
-	}
-
-	/**
-	 * Test that is_registered() returns true for a registered importer.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importers::is_registered
-	 */
-	public function test_is_registered() {
-
-		$this->assertTrue( WordPoints_Importers::is_registered( 'test' ) );
-	}
-
-	/**
-	 * Test that is_registered() returns false for an unregistered importer.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importers::is_registered
-	 */
-	public function test_is_registered_unregistered() {
-
-		WordPoints_Importers::deregister( 'test' );
-
-		$this->assertFalse( WordPoints_Importers::is_registered( 'test' ) );
-	}
-
-	/**
-	 * Test that get_importer() returns false for an unregistered importer.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importers::get_importer
-	 */
-	public function test_get_unregistered_importer() {
-
-		WordPoints_Importers::deregister( 'test' );
-
-		$this->assertFalse( WordPoints_Importers::get_importer( 'test' ) );
-	}
-
-	/**
-	 * Test that get_importer() returns an importer object.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @covers WordPoints_Importers::get_importer
-	 */
-	public function test_get_importer() {
-
-		$this->assertInstanceOf(
-			'WordPoints_Importer_Mock'
-			, WordPoints_Importers::get_importer( 'test' )
-		);
-	}
+    /**
+     * Backup of the importers when the test begins.
+     *
+     * @since 1.0.0
+     *
+     * @var array[]
+     */
+    protected $_backup_importers;
+
+    /**
+     * @since 1.0.0
+     */
+    public function setUp() {
+
+        parent::setUp();
+
+        $this->_backup_importers = WordPoints_Importers::get();
+
+        WordPoints_Importers::register(
+            'test'
+            , array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
+        );
+
+    }
+
+    /**
+     * @since 1.0.0
+     */
+    public function tearDown() {
+
+        $importers = WordPoints_Importers::get();
+
+        foreach ( $this->_backup_importers as $slug => $args ) {
+
+            if ( ! isset( $importers[ $slug ] ) ) {
+                WordPoints_Importers::register( $slug, $args );
+            }
+
+            unset( $importers[ $slug ] );
+        }
+
+        foreach ( $importers as $slug => $args ) {
+            WordPoints_Importers::deregister( $slug );
+        }
+
+        parent::tearDown();
+    }
+
+    /**
+     * Test registration.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importers::register
+     * @covers WordPoints_Importers::get
+     */
+    public function test_register() {
+
+        WordPoints_Importers::register(
+            __METHOD__
+            , array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
+        );
+
+        $importers = WordPoints_Importers::get();
+
+        $this->assertArrayHasKey( 'test', $importers );
+        $this->assertEquals(
+            array( 'class' => 'WordPoints_Importer_Mock', 'name' => 'Test' )
+            , $importers['test']
+        );
+    }
+
+    /**
+     * Test that deregister() deregisters the importer.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importers::register
+     * @covers WordPoints_Importers::get
+     */
+    public function test_deregister() {
+
+        WordPoints_Importers::deregister( 'test' );
+
+        $importers = WordPoints_Importers::get();
+
+        $this->assertArrayNotHasKey( 'test', $importers );
+    }
+
+    /**
+     * Test that is_registered() returns true for a registered importer.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importers::is_registered
+     */
+    public function test_is_registered() {
+
+        $this->assertTrue( WordPoints_Importers::is_registered( 'test' ) );
+    }
+
+    /**
+     * Test that is_registered() returns false for an unregistered importer.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importers::is_registered
+     */
+    public function test_is_registered_unregistered() {
+
+        WordPoints_Importers::deregister( 'test' );
+
+        $this->assertFalse( WordPoints_Importers::is_registered( 'test' ) );
+    }
+
+    /**
+     * Test that get_importer() returns false for an unregistered importer.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importers::get_importer
+     */
+    public function test_get_unregistered_importer() {
+
+        WordPoints_Importers::deregister( 'test' );
+
+        $this->assertFalse( WordPoints_Importers::get_importer( 'test' ) );
+    }
+
+    /**
+     * Test that get_importer() returns an importer object.
+     *
+     * @since 1.0.0
+     *
+     * @covers WordPoints_Importers::get_importer
+     */
+    public function test_get_importer() {
+
+        $this->assertInstanceOf(
+            'WordPoints_Importer_Mock'
+            , WordPoints_Importers::get_importer( 'test' )
+        );
+    }
 
 }
 

tests/phpunit/includes/mocks.php

Indentation +82 added lines, -82 removed lines

@@ -14,88 +14,88 @@
  */
 class WordPoints_Importer_Mock extends WordPoints_Importer {
 
-	/**
-	 * @since 1.0.0
-	 */
-	public $components;
-
-	/**
-	 * The "imports" performed.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var array
-	 */
-	public $imports;
-
-	/**
-	 * The "imports" for that were checked for possible performance.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var array
-	 */
-	public $can_imports;
-
-	/**
-	 * Whether this importer is available.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @var true|WP_Error
-	 */
-	public $is_available = true;
-
-	/**
-	 * @since 1.0.0
-	 */
-	public function is_available() {
-		return $this->is_available;
-	}
-
-	/**
-	 * Mock an import method.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param array $settings The settings for this "component".
-	 */
-	public function do_an_import( $settings ) {
-
-		$this->imports[] = $settings;
-	}
-
-	/**
-	 * Mock a can_import method.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param array $settings The settings for this "component".
-	 *
-	 * @return true
-	 */
-	public function can_import( $settings ) {
-
-		$this->can_imports[] = $settings;
-
-		return true;
-	}
-
-	/**
-	 * Return a WP_Error because we can't do the import for an option.
-	 *
-	 * @since 1.0.0
-	 *
-	 * @param array $settings The settings for this "component".
-	 *
-	 * @return WP_Error An error.
-	 */
-	public function cant_import( $settings ) {
-
-		$this->can_imports[] = $settings;
-
-		return new WP_Error();
-	}
+    /**
+     * @since 1.0.0
+     */
+    public $components;
+
+    /**
+     * The "imports" performed.
+     *
+     * @since 1.0.0
+     *
+     * @var array
+     */
+    public $imports;
+
+    /**
+     * The "imports" for that were checked for possible performance.
+     *
+     * @since 1.0.0
+     *
+     * @var array
+     */
+    public $can_imports;
+
+    /**
+     * Whether this importer is available.
+     *
+     * @since 1.0.0
+     *
+     * @var true|WP_Error
+     */
+    public $is_available = true;
+
+    /**
+     * @since 1.0.0
+     */
+    public function is_available() {
+        return $this->is_available;
+    }
+
+    /**
+     * Mock an import method.
+     *
+     * @since 1.0.0
+     *
+     * @param array $settings The settings for this "component".
+     */
+    public function do_an_import( $settings ) {
+
+        $this->imports[] = $settings;
+    }
+
+    /**
+     * Mock a can_import method.
+     *
+     * @since 1.0.0
+     *
+     * @param array $settings The settings for this "component".
+     *
+     * @return true
+     */
+    public function can_import( $settings ) {
+
+        $this->can_imports[] = $settings;
+
+        return true;
+    }
+
+    /**
+     * Return a WP_Error because we can't do the import for an option.
+     *
+     * @since 1.0.0
+     *
+     * @param array $settings The settings for this "component".
+     *
+     * @return WP_Error An error.
+     */
+    public function cant_import( $settings ) {
+
+        $this->can_imports[] = $settings;
+
+        return new WP_Error();
+    }
 }
 
 // EOF

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;
+    /**
+     * The messages that have been reported.
+     *
+     * @since 1.0.0
+     *
+     * @var string[][]
+     */
+    public $messages;
 
-	/**
-	 * @since 1.0.0
-	 */
-	protected function _send( $message, $type = 'info' ) {
+    /**
+     * @since 1.0.0
+     */
+    protected function _send( $message, $type = 'info' ) {
 
-		$this->messages[ $type ][] = $message;
-	}
+        $this->messages[ $type ][] = $message;
+    }
 }
 
 // EOF