Skip to content

Commit

Permalink
Tests: make all classes either abstract or final
Browse files Browse the repository at this point in the history
  • Loading branch information
jrfnl committed Sep 26, 2023
1 parent 6ea82d8 commit 36420e4
Show file tree
Hide file tree
Showing 9 changed files with 9 additions and 9 deletions.
2 changes: 1 addition & 1 deletion tests/ConfigurationTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Configuration unit tests.
*/
class ConfigurationTest extends Whip_TestCase {
final class ConfigurationTest extends Whip_TestCase {

/**
* Tests the creation of a Whip_Configuration with invalid input.
Expand Down
2 changes: 1 addition & 1 deletion tests/MessageDismisserTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Message Dismisser unit tests.
*/
class MessageDismisserTest extends Whip_TestCase {
final class MessageDismisserTest extends Whip_TestCase {

/**
* Tests if Whip_MessageDismisser correctly updates Whip_DismissStorage.
Expand Down
2 changes: 1 addition & 1 deletion tests/MessageTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Message Unit tests.
*/
class MessageTest extends Whip_TestCase {
final class MessageTest extends Whip_TestCase {

/**
* Tests if Whip_BasicMessage correctly handles a string for its body argument.
Expand Down
2 changes: 1 addition & 1 deletion tests/MessagesManagerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Message Manager unit tests.
*/
class MessagesManagerTest extends Whip_TestCase {
final class MessagesManagerTest extends Whip_TestCase {

/**
* Creates a MessagesManager, calls hasMessages and tests if it returns false
Expand Down
2 changes: 1 addition & 1 deletion tests/RequirementsCheckerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Requirements checker unit tests.
*/
class RequirementsCheckerTest extends Whip_TestCase {
final class RequirementsCheckerTest extends Whip_TestCase {

/**
* Tests if Whip_RequirementsChecker is successfully created when given valid arguments.
Expand Down
2 changes: 1 addition & 1 deletion tests/TestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Base test case class from which all tests should extend.
*/
class Whip_TestCase extends PHPUnit_Framework_TestCase {
abstract class Whip_TestCase extends PHPUnit_Framework_TestCase {

/**
* Helper method to test exceptions.
Expand Down
2 changes: 1 addition & 1 deletion tests/VersionRequirementTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Version requirements unit tests.
*/
class VersionRequirementTest extends Whip_TestCase {
final class VersionRequirementTest extends Whip_TestCase {

/**
* Creates a new Whip_VersionRequirement with component php and version 5.2 and
Expand Down
2 changes: 1 addition & 1 deletion tests/WPMessageDismissListenerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
*
* @coversDefaultClass Whip_WPMessageDismissListener
*/
class WPMessageDismissListener extends Whip_TestCase {
final class WPMessageDismissListener extends Whip_TestCase {

/**
* Tests the listen method.
Expand Down
2 changes: 1 addition & 1 deletion tests/doubles/Whip_DismissStorageMock.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
/**
* Test helper.
*/
class Whip_DismissStorageMock implements Whip_DismissStorage {
final class Whip_DismissStorageMock implements Whip_DismissStorage {

/**
* Holds the dismissed state.
Expand Down

0 comments on commit 36420e4

Please sign in to comment.