Enable php_unit_internal_class rule (#874)

stripe · Feb 13, 2020 · 87b4701 · 87b4701
1 parent cbcbec1
commit 87b4701
Show file tree

Hide file tree

Showing 88 changed files with 527 additions and 102 deletions.
diff --git a/.php_cs.dist b/.php_cs.dist
@@ -15,7 +15,6 @@ return PhpCsFixer\Config::create()
             'spacing' => 'one',
         ],
         'ordered_class_elements' => false,
-        'php_unit_internal_class' => false,
         'php_unit_test_class_requires_covers' => false,
         'phpdoc_align' => false,
         'protected_to_private' => false,

diff --git a/tests/Stripe/AccountLinkTest.php b/tests/Stripe/AccountLinkTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class AccountLinkTest extends TestCase
+/**
+ * @internal
+ */
+final class AccountLinkTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     public function testIsCreatable()
     {
         $this->expectsRequest(

diff --git a/tests/Stripe/AccountTest.php b/tests/Stripe/AccountTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class AccountTest extends TestCase
+/**
+ * @internal
+ */
+final class AccountTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'acct_123';
     const TEST_CAPABILITY_ID = 'acap_123';
     const TEST_EXTERNALACCOUNT_ID = 'ba_123';

diff --git a/tests/Stripe/AlipayAccountTest.php b/tests/Stripe/AlipayAccountTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class AlipayAccountTest extends TestCase
+/**
+ * @internal
+ */
+final class AlipayAccountTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'aliacc_123';
 
     // Because of the wildcard nature of sources, stripe-mock cannot currently

diff --git a/tests/Stripe/ApiRequestorTest.php b/tests/Stripe/ApiRequestorTest.php
@@ -4,8 +4,13 @@
 
 use Stripe\HttpClient\CurlClient;
 
-class ApiRequestorTest extends TestCase
+/**
+ * @internal
+ */
+final class ApiRequestorTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     public function testEncodeObjects()
     {
         $reflector = new \ReflectionClass(\Stripe\ApiRequestor::class);

diff --git a/tests/Stripe/ApplePayDomainTest.php b/tests/Stripe/ApplePayDomainTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class ApplePayDomainTest extends TestCase
+/**
+ * @internal
+ */
+final class ApplePayDomainTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'apwc_123';
 
     public function testIsListable()

diff --git a/tests/Stripe/ApplicationFeeRefundTest.php b/tests/Stripe/ApplicationFeeRefundTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class ApplicationFeeRefundTest extends TestCase
+/**
+ * @internal
+ */
+final class ApplicationFeeRefundTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'fr_123';
     const TEST_FEE_ID = 'fee_123';
 

diff --git a/tests/Stripe/ApplicationFeeTest.php b/tests/Stripe/ApplicationFeeTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class ApplicationFeeTest extends TestCase
+/**
+ * @internal
+ */
+final class ApplicationFeeTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'fee_123';
     const TEST_FEEREFUND_ID = 'fr_123';
 

diff --git a/tests/Stripe/BalanceTest.php b/tests/Stripe/BalanceTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class BalanceTest extends TestCase
+/**
+ * @internal
+ */
+final class BalanceTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     public function testIsRetrievable()
     {
         $this->expectsRequest(

diff --git a/tests/Stripe/BalanceTransactionTest.php b/tests/Stripe/BalanceTransactionTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class BalanceTransactionTest extends TestCase
+/**
+ * @internal
+ */
+final class BalanceTransactionTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'txn_123';
 
     public function testIsListable()

diff --git a/tests/Stripe/BankAccountTest.php b/tests/Stripe/BankAccountTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class BankAccountTest extends TestCase
+/**
+ * @internal
+ */
+final class BankAccountTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'ba_123';
 
     // Because of the wildcard nature of sources, stripe-mock cannot currently

diff --git a/tests/Stripe/BitcoinReceiverTest.php b/tests/Stripe/BitcoinReceiverTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class BitcoinReceiverTest extends TestCase
+/**
+ * @internal
+ */
+final class BitcoinReceiverTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'btcrcv_123';
 
     // Because of the wildcard nature of sources, stripe-mock cannot currently

diff --git a/tests/Stripe/CapabilityTest.php b/tests/Stripe/CapabilityTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CapabilityTest extends TestCase
+/**
+ * @internal
+ */
+final class CapabilityTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_ACCOUNT_ID = 'acct_123';
     const TEST_RESOURCE_ID = 'acap_123';
 

diff --git a/tests/Stripe/CardTest.php b/tests/Stripe/CardTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CardTest extends TestCase
+/**
+ * @internal
+ */
+final class CardTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'card_123';
 
     // Because of the wildcard nature of sources, stripe-mock cannot currently

diff --git a/tests/Stripe/ChargeTest.php b/tests/Stripe/ChargeTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class ChargeTest extends TestCase
+/**
+ * @internal
+ */
+final class ChargeTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'ch_123';
 
     public function testIsListable()

diff --git a/tests/Stripe/Checkout/SessionTest.php b/tests/Stripe/Checkout/SessionTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe\Checkout;
 
-class SessionTest extends \Stripe\TestCase
+/**
+ * @internal
+ */
+final class SessionTest extends \PHPUnit\Framework\TestCase
 {
+    use \Stripe\TestHelper;
+
     const TEST_RESOURCE_ID = 'cs_123';
 
     public function testIsCreatable()

diff --git a/tests/Stripe/CollectionTest.php b/tests/Stripe/CollectionTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CollectionTest extends TestCase
+/**
+ * @internal
+ */
+final class CollectionTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     /** @var \Stripe\Collection */
     private $fixture;
 

diff --git a/tests/Stripe/CountrySpecTest.php b/tests/Stripe/CountrySpecTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CountrySpecTest extends TestCase
+/**
+ * @internal
+ */
+final class CountrySpecTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'US';
 
     public function testIsListable()

diff --git a/tests/Stripe/CouponTest.php b/tests/Stripe/CouponTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CouponTest extends TestCase
+/**
+ * @internal
+ */
+final class CouponTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = '25OFF';
 
     public function testIsListable()

diff --git a/tests/Stripe/CreditNoteTest.php b/tests/Stripe/CreditNoteTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CreditNoteTest extends TestCase
+/**
+ * @internal
+ */
+final class CreditNoteTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'cn_123';
 
     public function testIsListable()

diff --git a/tests/Stripe/CustomerBalanceTransactionTest.php b/tests/Stripe/CustomerBalanceTransactionTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CustomerBalanceTransactionTest extends TestCase
+/**
+ * @internal
+ */
+final class CustomerBalanceTransactionTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_CUSTOMER_ID = 'cus_123';
     const TEST_RESOURCE_ID = 'cbtxn_123';
 

diff --git a/tests/Stripe/CustomerTest.php b/tests/Stripe/CustomerTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class CustomerTest extends TestCase
+/**
+ * @internal
+ */
+final class CustomerTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'cus_123';
     const TEST_SOURCE_ID = 'ba_123';
     const TEST_TAX_ID_ID = 'txi_123';

diff --git a/tests/Stripe/DisputeTest.php b/tests/Stripe/DisputeTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class DisputeTest extends TestCase
+/**
+ * @internal
+ */
+final class DisputeTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'dp_123';
 
     public function testIsListable()

diff --git a/tests/Stripe/EphemeralKeyTest.php b/tests/Stripe/EphemeralKeyTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class EphemeralKeyTest extends TestCase
+/**
+ * @internal
+ */
+final class EphemeralKeyTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     public function testIsCreatable()
     {
         $this->expectsRequest(

diff --git a/tests/Stripe/ErrorObjectTest.php b/tests/Stripe/ErrorObjectTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class ErrorObjectTest extends TestCase
+/**
+ * @internal
+ */
+final class ErrorObjectTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     public function testDefaultValues()
     {
         $error = ErrorObject::constructFrom([]);

diff --git a/tests/Stripe/EventTest.php b/tests/Stripe/EventTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe;
 
-class EventTest extends TestCase
+/**
+ * @internal
+ */
+final class EventTest extends \PHPUnit\Framework\TestCase
 {
+    use TestHelper;
+
     const TEST_RESOURCE_ID = 'evt_123';
 
     public function testIsListable()

diff --git a/tests/Stripe/Exception/ApiErrorExceptionTest.php b/tests/Stripe/Exception/ApiErrorExceptionTest.php
@@ -2,8 +2,13 @@
 
 namespace Stripe\Exception;
 
-class ApiErrorExceptionTest extends \Stripe\TestCase
+/**
+ * @internal
+ */
+final class ApiErrorExceptionTest extends \PHPUnit\Framework\TestCase
 {
+    use \Stripe\TestHelper;
+
     public function createFixture()
     {
         $mock = $this->getMockForAbstractClass(ApiErrorException::class);