Skip to content

Commit

Permalink
[Cases] Made case classes abastract
Browse files Browse the repository at this point in the history
  • Loading branch information
bainternet committed Jul 1, 2021
1 parent a65c349 commit 0ba568d
Show file tree
Hide file tree
Showing 5 changed files with 39 additions and 23 deletions.
2 changes: 1 addition & 1 deletion src/cases/ajax-test.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
/**
* Class Ajax_Test
*/
class Ajax_Test extends WP_Ajax_UnitTestCase {
abstract class Ajax_Test extends WP_Ajax_UnitTestCase {
use Globals, Hooks, Remote_Request;

/**
Expand Down
11 changes: 9 additions & 2 deletions src/cases/module-test.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
/**
* Class Module_Test
*/
class Module_Test extends Unit_Test {
abstract class Module_Test extends Unit_Test {
use Plugin;

public $name = '';
Expand All @@ -15,6 +15,9 @@ class Module_Test extends Unit_Test {
*/
public $module;

/**
* @throws \Exception
*/
public function setUp(): void {
// Support Modules named with more then one word
// ex: Kits_Dashboard
Expand All @@ -23,7 +26,11 @@ public function setUp(): void {
return ucwords( $string );
}, $module_class_name );
$module_class_name = implode( '', $module_class_name );
$this->module = $this->get_plugin_module( $module_class_name );
if ( ! empty( $module_class_name ) ) {
$this->module = $this->get_plugin_module( $module_class_name );
} else {
throw new \Exception( 'Module not found' );
}
parent::setUp();
}

Expand Down
2 changes: 1 addition & 1 deletion src/cases/rest-route-test.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
/**
* Class Rest_Route
*/
class Rest_Route_Test extends Unit_Test {
abstract class Rest_Route_Test extends Unit_Test {
use Plugin, Rest;
/**
* @var string namespace of route
Expand Down
20 changes: 1 addition & 19 deletions src/cases/unit-test.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
/**
* Class Eunit_Test_Case
*/
class Unit_Test extends \WP_UnitTestCase {
abstract class Unit_Test extends \WP_UnitTestCase {
/**
* @var string
*/
Expand All @@ -31,22 +31,4 @@ class Unit_Test extends \WP_UnitTestCase {
public function get_namespace() {
return getenv( 'EUNIT_TEST_CASE_NAMESPACE' );
}

/**
* setUp
*/
public function setUp() : void {
parent::setUp();
// Users
$this->subscriber = $this->factory->user->create( [ 'role' => 'subscriber' ] );
$this->editor = $this->factory->user->create( [ 'role' => 'editor' ] );
$this->administrator = $this->factory->user->create( [ 'role' => 'administrator' ] );
}

/**
* tearDown
*/
public function tearDown() : void {
parent::tearDown();
}
}
27 changes: 27 additions & 0 deletions src/traits/users.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
<?php
namespace Eunit\Traits;

/**
* Trait Users
*/
trait Users {
/**
* get_plugin
* @return mixed
*/
public function init_users() : void {
// Users
$this->subscriber = $this->factory->user->create( [ 'role' => 'subscriber' ] );
$this->editor = $this->factory->user->create( [ 'role' => 'editor' ] );
$this->administrator = $this->factory->user->create( [ 'role' => 'administrator' ] );
}

/**
* cleanup_users
*/
public function cleanup_users() : void {
\wp_delete_user( $this->subscriber );
\wp_delete_user( $this->editor );
\wp_delete_user( $this->administrator );
}
}

0 comments on commit 0ba568d

Please sign in to comment.