Finishing up

Author: Mr-Ming

app/boilerplates/simple/app/admin/class-app.php

@@ -41,7 +41,7 @@ class App {
 	 * @param string $installed_url Installed URL.
 	 * @param string $version       Version.
 	 */
-	function __construct( $installed_dir, $installed_url, $version ) {
+	public function __construct( $installed_dir, $installed_url, $version ) {
 		$this->installed_dir = $installed_dir;
 		$this->installed_url = $installed_url;
 		$this->version       = $version;

app/boilerplates/simple/app/class-plugin.php

@@ -37,7 +37,6 @@ class Plugin extends Abstract_Plugin {
 	 */
 	protected static $current_file = __FILE__;
 
-
 	/**
 	 * Initialize the plugin - for public (front end)
 	 *
@@ -51,15 +50,18 @@ public function onload( $instance ) {
 
 	/**
 	 * Initialize public / shared functionality using new Class(), add_action() or add_filter().
+	 *
+	 * @return void
 	 */
 	public function init() {
 		do_action( static::$action_prefix . 'before_init' );
-
 		do_action( static::$action_prefix . 'after_init' );
 	}
 
 	/**
 	 * Initialize functionality only loaded for logged-in users.
+	 *
+	 * @return void
 	 */
 	public function authenticated_init() {
 		if ( is_user_logged_in() ) {
@@ -77,7 +79,7 @@ public function authenticated_init() {
 	/**
 	 * Defines and Globals.
 	 *
-	 * @return mixed|void
+	 * @return void
 	 */
 	protected function defines_and_globals() {
 	}

app/boilerplates/simple/composer-test.json

@@ -5,12 +5,12 @@
   },
   "require-dev": {
 	"wp-coding-standards/wpcs": "^0.10",
-    "rregeer/phpunit-coverage-check": "^0.1.6",
-    "phpunit/phpunit": "^6"
+	"rregeer/phpunit-coverage-check": "^0.1.6",
+	"phpunit/phpunit": "^6"
   },
   "scripts": {
 	"test": "./bin/phpcs --standard=./.circleci/codesniffer.ruleset.xml --extensions='php,css' ./",
-    "fix": "./bin/phpcbf --standard=./.circleci/codesniffer.ruleset.xml --extensions='php,js,css' --ignore=*/lib/* ./",
-    "phpunit": "./bin/phpunit --stop-on-error"
+	"fix": "./bin/phpcbf --standard=./.circleci/codesniffer.ruleset.xml --extensions='php,js,css' --ignore=*/lib/* ./",
+	"phpunit": "./bin/phpunit --stop-on-error"
   }
 }

app/boilerplates/simple/tests/admin/test-class-app.php

@@ -0,0 +1,30 @@
+<?php
+/**
+ * PHPUnit Test
+ *
+ * @package <%= PKG %>
+ */
+
+/**
+ * App_Test
+ */
+final class App_Test extends WP_UnitTestCase {
+
+	/**
+	 * Test __construct
+	 *
+	 * @covers <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Admin\App::__construct
+	 */
+	public function test_construct() {
+		$installed_dir = '/var/installed/dir';
+		$installed_url = '/var/installed/url';
+		$version       = 1.0;
+
+		$app = new <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Admin\App( $installed_dir, $installed_url, $version );
+
+		$this->assertEquals( $installed_dir, $app->installed_dir );
+		$this->assertEquals( $installed_url, $app->installed_url );
+		$this->assertEquals( $version, $app->version );
+	}
+
+}

app/boilerplates/simple/tests/test-class-plugin.php

@@ -6,66 +6,47 @@
  */
 
 /**
- * Plugin Test
+ * Plugin_Test
  */
 final class Plugin_Test extends WP_UnitTestCase {
 
 	/**
 	 * Test onload
 	 *
-	 * @covers OneCMS\Taxonomy_Paths\Plugin::activate
-	 */
-	public function test_activate() {
-		$plugin   = new OneCMS\Taxonomy_Paths\Plugin();
-
-		$this->assertNull(
-			$plugin->activate()
-		);
-	}
-
-	/**
-	 * Test onload
-	 *
-	 * @covers OneCMS\Taxonomy_Paths\Plugin::onload
+	 * @covers <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin::onload
 	 */
 	public function test_onload() {
 		$instance = new stdClass();
-		$plugin   = new OneCMS\Taxonomy_Paths\Plugin();
+		$plugin   = new <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin();
+		$result   = $plugin->onload( $instance );
 
-		$this->assertNull(
-			$plugin->onload( $instance )
-		);
+		$this->assertNull( $result );
 	}
 
 	/**
 	 * Test init
 	 *
-	 * @covers OneCMS\Taxonomy_Paths\Plugin::init
+	 * @covers <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin::init
 	 */
 	public function test_init() {
 		global $wp_actions;
 
-		$plugin = new OneCMS\Taxonomy_Paths\Plugin();
-		$init   = $plugin->init();
+		$plugin = new <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin();
+		$result = $plugin->init();
+
+		$this->assertTrue( isset( $wp_actions['<%= US_SLUG %>_before_init'] ) );
+		$this->assertTrue( isset( $wp_actions['<%= US_SLUG %>_after_init'] ) );
 
-		$this->assertTrue( isset( $wp_actions['onecms_taxonomy_paths_before_init'] ) );
-		$this->assertTrue( isset( $wp_actions['onecms_taxonomy_paths_after_init'] ) );
-		$this->assertNotFalse(
-			has_filter(
-				'onecms_enable_taxonomy_paths',
-				false
-			)
-		);
-		$this->assertNull( $init );
+		$this->assertNull( $result );
 	}
 
 	/**
 	 * Test authenticated_init when user is not logged in
 	 *
-	 * @covers OneCMS\Taxonomy_Paths\Plugin::authenticated_init
+	 * @covers <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin::authenticated_init
 	 */
 	public function test_authenticated_init_when_user_is_not_logged_in() {
-		$plugin             = new OneCMS\Taxonomy_Paths\Plugin();
+		$plugin             = new <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin();
 		$authenticated_init = $plugin->authenticated_init();
 
 		$this->assertFalse( isset( $plugin->admin ) );
@@ -75,13 +56,13 @@ public function test_authenticated_init_when_user_is_not_logged_in() {
 	/**
 	 * Test authenticated_init when user is logged in
 	 *
-	 * @covers OneCMS\Taxonomy_Paths\Plugin::authenticated_init
+	 * @covers <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin::authenticated_init
 	 */
 	public function test_authenticated_init_when_user_is_logged_in() {
 		$user_id = $this->factory->user->create();
 		wp_set_current_user( $user_id );
 
-		$plugin             = new OneCMS\Taxonomy_Paths\Plugin();
+		$plugin             = new <%= PRIMARY_NAMESPACE %>\<%= SECONDARY_NAMESPACE %>\Plugin();
 		$authenticated_init = $plugin->authenticated_init();
 
 		$this->assertTrue( isset( $plugin->admin ) );