More phpunit8 prep
authoreileen <emcnaughton@wikimedia.org>
Thu, 25 Mar 2021 02:03:58 +0000 (15:03 +1300)
committereileen <emcnaughton@wikimedia.org>
Thu, 25 Mar 2021 02:04:09 +0000 (15:04 +1300)
15 files changed:
ext/afform/mock/tests/phpunit/api/v4/AfformTestCase.php
tests/phpunit/CRM/Core/Payment/PayPalIPNTest.php
tests/phpunit/CRM/Event/BAO/ParticipantStatusTest.php
tests/phpunit/CRM/Utils/versionCheckTest.php
tests/phpunit/Civi/Core/CiviFacadeTest.php
tests/phpunit/Civi/Core/Event/GenericHookEventTest.php
tests/phpunit/Civi/Core/SettingsBagTest.php
tests/phpunit/Civi/Core/SettingsManagerTest.php
tests/phpunit/Civi/Core/SettingsStackTest.php
tests/phpunit/Civi/Test/ExampleHookTest.php
tests/phpunit/Civi/Test/ExampleTransactionalTest.php
tests/phpunit/api/v4/Action/ChainTest.php
tests/phpunit/api/v4/Action/FkJoinTest.php
tests/phpunit/api/v4/Traits/OptionCleanupTrait.php
tests/phpunit/api/v4/UnitTestCase.php

index 4847abf604e1445c84bb3ea08253fa12b930c0cc..a2dcb6af553bd8cd6441c47cce322a7482ba64e1 100644 (file)
@@ -22,18 +22,10 @@ abstract class api_v4_AfformTestCase extends \PHPUnit\Framework\TestCase impleme
   /**
    * The setup() method is executed before the test is executed (optional).
    */
-  public function setUp() {
+  public function setUp(): void {
     parent::setUp();
     CRM_Core_Config::singleton()->userPermissionTemp = new CRM_Core_Permission_Temp();
     CRM_Core_Config::singleton()->userPermissionTemp->grant('administer CiviCRM');
   }
 
-  /**
-   * The tearDown() method is executed after the test was executed (optional)
-   * This can be used for cleanup.
-   */
-  public function tearDown() {
-    parent::tearDown();
-  }
-
 }
index 80a030be6ab99585cd62ea2940c379b450101375..910a6d300f9a4858b5e0f35c02f413c717ad5c36 100644 (file)
@@ -26,7 +26,7 @@ class CRM_Core_Payment_PayPalIPNTest extends CiviUnitTestCase {
   /**
    * Set up function.
    */
-  public function setUp() {
+  public function setUp(): void {
     parent::setUp();
     $this->_paymentProcessorID = $this->paymentProcessorCreate(['is_test' => 0, 'payment_processor_type_id' => 'PayPal_Standard']);
     $this->_contactID = $this->individualCreate();
@@ -42,7 +42,7 @@ class CRM_Core_Payment_PayPalIPNTest extends CiviUnitTestCase {
   /**
    * Tear down function.
    */
-  public function tearDown() {
+  public function tearDown(): void {
     $this->quickCleanUpFinancialEntities();
     parent::tearDown();
   }
index 1d67e841c47bf35e41ba0a0c5eece6225b0084f2..f70b6f95884d51e728864227886004bf7ccd3025 100644 (file)
  */
 class CRM_Event_BAO_ParticipantStatusTest extends CiviUnitTestCase {
 
-  /**
-   * Sets up the fixture, for example, opens a network connection.
-   * This method is called before a test is executed.
-   */
-  protected function setUp() {
-    parent::setUp();
-  }
-
-  /**
-   * Tears down the fixture, for example, closes a network connection.
-   * This method is called after a test is executed.
-   */
-  protected function tearDown() {
-  }
-
   /**
    *  create() and deleteParticipantStatusType() method
    */
index b8cdb51bf1a425858e399ed6c9131e6ff2808823..09292b514ba0989be49de95aa498e2e47c595aa7 100644 (file)
@@ -8,10 +8,6 @@ use Civi\Test\Invasive;
  */
 class CRM_Utils_versionCheckTest extends CiviUnitTestCase {
 
-  public function setUp() {
-    parent::setUp();
-  }
-
   /**
    * @var array
    */
@@ -91,7 +87,7 @@ class CRM_Utils_versionCheckTest extends CiviUnitTestCase {
     ],
   ];
 
-  public function tearDown() {
+  public function tearDown(): void {
     parent::tearDown();
     $vc = new CRM_Utils_VersionCheck();
     if (file_exists($vc->cacheFile)) {
index 76dbc4bdbe42200cfbe4bd9a2b6816209eea75d1..1fcae645d3bc0e3faa87fba6d28d536348881b09 100644 (file)
@@ -14,7 +14,7 @@ class CiviFacadeTest extends \CiviUnitTestCase {
     $this->mandates = [];
   }
 
-  public function tearDown() {
+  public function tearDown(): void {
     $GLOBALS['civicrm_setting'] = $this->origSetting;
     parent::tearDown();
   }
index 34bc465104d221c1212f5b46e68e7d335bc94e54..3ae516c552f41775c8b01838953eea41c9bb5f2f 100644 (file)
@@ -3,7 +3,7 @@ namespace Civi\Core\Event;
 
 class GenericHookEventTest extends \CiviUnitTestCase {
 
-  public function tearDown() {
+  public function tearDown(): void {
     \CRM_Utils_Hook::singleton()->reset();
     parent::tearDown();
   }
index 6271337b29c090d63fca864c992ec3fa863d2342..e68ac6f29e2b2e3b86db98b812a33a31554a693f 100644 (file)
@@ -14,7 +14,7 @@ class SettingsBagTest extends \CiviUnitTestCase {
     $this->mandates = [];
   }
 
-  public function tearDown() {
+  public function tearDown(): void {
     $GLOBALS['civicrm_setting'] = $this->origSetting;
     parent::tearDown();
   }
index 71be2b80a77942fd2428830064b38a878dba3097..77b99ad115c139d26631ef9aeab3d5346000dcdc 100644 (file)
@@ -38,7 +38,7 @@ class SettingsManagerTest extends \CiviUnitTestCase {
     ];
   }
 
-  public function tearDown() {
+  public function tearDown(): void {
     $GLOBALS['civicrm_setting'] = $this->origSetting;
     parent::tearDown();
   }
index ac4cee90567b7c3668cc4ff06eba54da3caeabe4..258c68d20e167e09f823d9a20f9d1b7a84f91981 100644 (file)
@@ -3,15 +3,11 @@ namespace Civi\Core;
 
 class SettingsStackTest extends \CiviUnitTestCase {
 
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
     $this->useTransaction(TRUE);
   }
 
-  public function tearDown() {
-    parent::tearDown();
-  }
-
   /**
    * Temporarily modify -- then restore -- settings.
    */
index fb81845ccb57923701b8551e41860fc7025bbb7e..b84d80ceac972881f43d903583e28563e0993b03 100644 (file)
@@ -19,7 +19,7 @@ class ExampleHookTest extends \PHPUnit\Framework\TestCase implements HeadlessInt
     return \Civi\Test::headless()->apply();
   }
 
-  protected function setUp() {
+  protected function setUp(): void {
     $this->contact = \CRM_Core_DAO::createTestObject('CRM_Contact_DAO_Contact', [
       'contact_type' => 'Individual',
     ]);
@@ -27,7 +27,7 @@ class ExampleHookTest extends \PHPUnit\Framework\TestCase implements HeadlessInt
     $session->set('userID', $this->contact->id);
   }
 
-  protected function tearDown() {
+  protected function tearDown(): void {
     $this->contact->delete();
   }
 
index 17a93264e9c677441c7b219a23e221df7141c9c7..b147434aeab632490ef156f4402976246595a75a 100644 (file)
@@ -21,7 +21,7 @@ class ExampleTransactionalTest extends \PHPUnit\Framework\TestCase implements He
     return \Civi\Test::headless()->apply();
   }
 
-  protected function setUp() {
+  protected function setUp(): void {
     /** @var \CRM_Contact_DAO_Contact $contact */
     $contact = \CRM_Core_DAO::createTestObject('CRM_Contact_DAO_Contact', [
       'contact_type' => 'Individual',
@@ -60,9 +60,6 @@ class ExampleTransactionalTest extends \PHPUnit\Framework\TestCase implements He
     $this->assertTrue((bool) $dao->find());
   }
 
-  public function tearDown() {
-  }
-
   /**
    * Both testDummy1 and testDummy2 have been created at some point (as part of the test runs),
    * but all the data was rolled-back
index 00f6267c5c85086a71134f745497cb5e63da9520..16274b2ca1c6511d261b1a69f34c28a70e2f43e3 100644 (file)
@@ -30,8 +30,8 @@ use Civi\Api4\CustomGroup;
  */
 class ChainTest extends UnitTestCase {
 
-  public function tearDown() {
-    $result = CustomField::delete()
+  public function tearDown(): void {
+    CustomField::delete()
       ->setCheckPermissions(FALSE)
       ->addWhere('name', '=', 'FavPerson')
       ->addChain('group', CustomGroup::delete()->addWhere('name', '=', 'TestActCus'))
index fe5f4e8363642f66b87a57d51f96eac561be2086..ed140f87386aa09188f2c5ee230d4bcfa696df8a 100644 (file)
@@ -39,7 +39,7 @@ class FkJoinTest extends UnitTestCase {
     return parent::setUpHeadless();
   }
 
-  public function tearDown() {
+  public function tearDown(): void {
     $relatedTables = [
       'civicrm_activity',
       'civicrm_phone',
index c2571aa64e9695b7cfb17f3d96265f0bb57c37de..52c012252d19d03a467b9bf828937a75f01b8a39 100644 (file)
@@ -35,7 +35,7 @@ trait OptionCleanupTrait {
     $this->optionValueMaxId = \CRM_Core_DAO::singleValueQuery('SELECT MAX(id) FROM civicrm_option_value');
   }
 
-  public function tearDown() {
+  public function tearDown(): void {
     if ($this->optionValueMaxId) {
       \CRM_Core_DAO::executeQuery('DELETE FROM civicrm_option_value WHERE id > ' . $this->optionValueMaxId);
     }
index 067417b1d8bad4b999f896b80af9e8dc37230f00..9d91f0b582a8c1ce3b55f56b241ab7752cc49710 100644 (file)
@@ -48,15 +48,6 @@ class UnitTestCase extends \PHPUnit\Framework\TestCase implements HeadlessInterf
     return \Civi\Test::headless()->apply();
   }
 
-  /**
-   * Tears down the fixture, for example, closes a network connection.
-   *
-   * This method is called after a test is executed.
-   */
-  public function tearDown() {
-    parent::tearDown();
-  }
-
   /**
    * Quick clean by emptying tables created for the test.
    *