Set return type of base TestCase methods

From the [PHPUnit 8 release notes][1], the `TestCase` methods below now declare a `void` return type:

- `setUpBeforeClass()`
- `setUp()`
- `assertPreConditions()`
- `assertPostConditions()`
- `tearDown()`
- `tearDownAfterClass()`
- `onNotSuccessfulTest()`

[1]: https://phpunit.de/announcements/phpunit-8.html
This commit is contained in:
Shift 2022-06-21 10:00:57 +00:00
parent c2ba8fa234
commit 4f7a8f9752
No known key found for this signature in database
GPG Key ID: 5A96F038425C5A1C
168 changed files with 171 additions and 171 deletions

View File

@ -35,7 +35,7 @@ class AccountEmailQuotaTest extends TestCase
use AppSetup;
use MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class ActivityApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -26,7 +26,7 @@ class ApplePayDomainMerchantUrlTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class CancelInvoiceTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -42,7 +42,7 @@ class ClientMergeTest extends TestCase
private $primary_contact;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class ClientApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class ClientDeletedInvoiceCreationTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class ClientGatewayTokenApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -26,7 +26,7 @@ class ClientModelTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -33,7 +33,7 @@ class CreditsTest extends TestCase
use DatabaseTransactions;
use AppSetup;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class InvoicesTest extends TestCase
use DatabaseTransactions;
use AppSetup;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -24,7 +24,7 @@ class ClientPresenterTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -39,7 +39,7 @@ class ClientTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -33,7 +33,7 @@ class CompanyGatewayApiTest extends TestCase
use MockAccountData;
use CompanyGatewayFeesAndLimitsSaver;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -34,7 +34,7 @@ class CompanyGatewayResolutionTest extends TestCase
public $cg1;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -26,7 +26,7 @@ class CompanyGatewayTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class CompanySettingsTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -34,7 +34,7 @@ class CompanyTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class CompanyTokenApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -27,7 +27,7 @@ class CreditTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class DeleteInvoiceTest extends TestCase
use MockAccountData;
use MakesHash;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class DesignApiTest extends TestCase
public $id;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class DocumentsApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -43,7 +43,7 @@ class EntityPaidToDateTest extends TestCase
use MockAccountData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class ExpenseApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class ExpenseCategoryApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -27,7 +27,7 @@ class ClientCsvTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class ExportCompanyTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -27,7 +27,7 @@ class ExportCsvTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -43,7 +43,7 @@ class ProfitAndLossReportTest extends TestCase
public $faker;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class FilterApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class GroupSettingTest extends TestCase
//use DatabaseTransactions;
use MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -39,7 +39,7 @@ class BaseTransformerTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -40,7 +40,7 @@ class CsvImportTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class FreshbooksTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -74,7 +74,7 @@ class ImportCompanyTest extends TestCase
public $ids;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();
@ -1113,7 +1113,7 @@ class ImportCompanyTest extends TestCase
return $this->ids[$resource]["{$old}"];
}
public function tearDown() :void
protected function tearDown() :void
{
$backup_json_file = sys_get_temp_dir().'/backup/backup.json';

View File

@ -36,7 +36,7 @@ class ImportCsvTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -38,7 +38,7 @@ class Invoice2GoTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class InvoicelyTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -36,7 +36,7 @@ class WaveTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class ZohoTest extends TestCase
use MockAccountData;
// use DatabaseTransactions;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class InventoryManagementTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class InvitationTest extends TestCase
use DatabaseTransactions;
// use RefreshDatabase;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();
}

View File

@ -27,7 +27,7 @@ class InvoiceAmountPaymentTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class InvoiceEmailTest extends TestCase
use DatabaseTransactions;
use GeneratesCounter;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -27,7 +27,7 @@ class InvoiceLinkTasksTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -33,7 +33,7 @@ class InvoiceTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -50,7 +50,7 @@ class LoadTest extends TestCase
public int $count = 1;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class LoginTest extends TestCase
{
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();
Session::start();

View File

@ -29,7 +29,7 @@ class MigrationTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class MultiPaymentDeleteTest extends TestCase
private $faker;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class PlanTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class PaymentTermsApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -44,7 +44,7 @@ class PaymentTest extends TestCase
use MockAccountData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -44,7 +44,7 @@ class CreditPaymentTest extends TestCase
use MockUnitData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -44,7 +44,7 @@ class StorePaymentValidationTest extends TestCase
use MockAccountData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -34,7 +34,7 @@ class UnappliedPaymentDeleteTest extends TestCase
use MockUnitData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class UnappliedPaymentRefundTest extends TestCase
use MockUnitData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -26,7 +26,7 @@ class PdfCreatorTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -22,7 +22,7 @@ class ExampleIntegrationTest extends TestCase
{
use MakesInvoiceValues, MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class PingTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();
$this->makeTestData();

View File

@ -25,7 +25,7 @@ class PreviewTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class ProductTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class ProjectApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class PurchaseOrderTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -32,7 +32,7 @@ class QuoteTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class RecurringExpenseApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -34,7 +34,7 @@ class RecurringInvoiceTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -26,7 +26,7 @@ class RecurringInvoicesCronTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -30,7 +30,7 @@ class RecurringQuoteTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -34,7 +34,7 @@ class RecurringQuotesTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class RefundTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ class ReminderTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -36,7 +36,7 @@ class ReverseInvoiceTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -22,7 +22,7 @@ class SchedulerTest extends TestCase
use WithoutEvents;
// use RefreshDatabase;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class ShopInvoiceTest extends TestCase
use MakesHash;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -35,7 +35,7 @@ class SubscriptionApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp(): void
protected function setUp(): void
{
parent::setUp();

View File

@ -27,7 +27,7 @@ class SystemLogApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class TaskApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class TaskStatusApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class TaskStatusSortOnUpdateTest extends TestCase
use MockAccountData;
use MakesHash;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class TaxRateApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -32,7 +32,7 @@ class UpdateExchangeRatesTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -44,7 +44,7 @@ class UpdatePaymentTest extends TestCase
use MockAccountData;
use WithoutEvents;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -37,7 +37,7 @@ class UserTest extends TestCase
private $default_email = 'attach@gmail.com';
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -29,7 +29,7 @@ class VendorApiTest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class WebhookAPITest extends TestCase
use DatabaseTransactions;
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -24,7 +24,7 @@ class CheckCacheTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class CheckLockedInvoiceValidationTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -26,7 +26,7 @@ class CheckRemindersTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -46,7 +46,7 @@ class CompanyLedgerTest extends TestCase
public $account;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -22,7 +22,7 @@ class ContainerTest extends TestCase
{
use MockAccountData;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -28,7 +28,7 @@ class DownloadHistoricalInvoiceTest extends TestCase
use DatabaseTransactions;
use MakesHash;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -92,7 +92,7 @@ class EventTest extends TestCase
use MakesHash;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -33,7 +33,7 @@ class HtmlGenerationTest extends TestCase
use MockAccountData;
use MakesHash;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class InvoiceUploadTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -24,7 +24,7 @@ class MarkInvoicePaidTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -31,7 +31,7 @@ use Tests\TestCase;
*/
class MultiDBUserTest extends TestCase
{
public function setUp() :void
protected function setUp() :void
{
parent::setUp();
@ -244,7 +244,7 @@ class MultiDBUserTest extends TestCase
// }
// }
public function tearDown() :void
protected function tearDown() :void
{
DB::connection('db-ninja-01')->table('users')->delete();
DB::connection('db-ninja-02')->table('users')->delete();

View File

@ -44,7 +44,7 @@ class AuthorizeTest extends TestCase
public $customer_payment_profile = 1512424103;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

View File

@ -25,7 +25,7 @@ class PostmarkWebhookTest extends TestCase
use MockAccountData;
use DatabaseTransactions;
public function setUp() :void
protected function setUp() :void
{
parent::setUp();

Some files were not shown because too many files have changed in this diff Show More