Merge pull request #25439 from totten/master-test-tx

(NFC) Speed up random tests by 55-60%

tests/phpunit/CRM/Utils/API/MatchOptionTest.php

@@ -12,6 +12,7 @@ class CRM_Utils_API_MatchOptionTest extends CiviUnitTestCase {
   public $noise;
 
   public function setUp(): void {
+    $this->useTransaction();
     parent::setUp();
     $this->assertDBQuery(0, "SELECT count(*) FROM civicrm_contact WHERE first_name='Jeffrey' and last_name='Lebowski'");
 

tests/phpunit/CRM/Utils/API/ReloadOptionTest.php

@@ -12,6 +12,7 @@
 class CRM_Utils_API_ReloadOptionTest extends CiviUnitTestCase {
 
   public function setUp(): void {
+    $this->useTransaction();
     parent::setUp();
     CRM_Utils_Hook_UnitTests::singleton()->setHook('civicrm_post', [$this, 'onPost']);
   }

tests/phpunit/CRM/Utils/ArrayTest.php

@@ -6,6 +6,14 @@
  */
 class CRM_Utils_ArrayTest extends CiviUnitTestCase {
 
+  /**
+   * Set up for tests.
+   */
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   public function testAsColumns() {
     $rowsNum = [
       ['a' => 10, 'b' => 11],

tests/phpunit/CRM/Utils/Check/Component/EnvTest.php

@@ -8,6 +8,11 @@
  */
 class CRM_Utils_Check_Component_EnvTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * File check test should fail if reached maximum timeout.
    * @throws \GuzzleHttp\Exception\GuzzleException

tests/phpunit/CRM/Utils/Check/Component/OptionGroupsTest.php

@@ -8,6 +8,11 @@
  */
 class CRM_Utils_Check_Component_OptionGroupsTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   public function testCheckOptionGroupValues() {
     $optionGroup = $this->callAPISuccess('OptionGroup', 'create', [
       'name' => 'testGroup',

tests/phpunit/CRM/Utils/ColorTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_ColorTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @dataProvider contrastExamples
    */

tests/phpunit/CRM/Utils/DateTest.php

@@ -26,6 +26,14 @@
  */
 class CRM_Utils_DateTest extends CiviUnitTestCase {
 
+  /**
+   * Set up for tests.
+   */
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * Used by testGetFromTo
    */

tests/phpunit/CRM/Utils/GlobalStackTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_GlobalStackTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * Temporarily override global variables and ensure that the variable data.
    * is set as expected (before/during/after the override).

tests/phpunit/CRM/Utils/HTMLTest.php

@@ -15,6 +15,11 @@
  */
 class CRM_Utils_HTMLTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @return array
    */

tests/phpunit/CRM/Utils/HookTest.php

@@ -13,6 +13,7 @@ class CRM_Utils_HookTest extends CiviUnitTestCase {
   public $log;
 
   public function setUp(): void {
+    $this->useTransaction();
     parent::setUp();
     $this->fakeModules = [
       'hooktesta',

tests/phpunit/CRM/Utils/HtmlToTextTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_HtmlToTextTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @return array
    */

tests/phpunit/CRM/Utils/HttpClientTest.php

@@ -25,6 +25,7 @@ class CRM_Utils_HttpClientTest extends CiviUnitTestCase {
   protected $client;
 
   public function setUp(): void {
+    $this->useTransaction();
     parent::setUp();
 
     $this->tmpFile = $this->createTempDir() . '/example.txt';

tests/phpunit/CRM/Utils/ICalendarTest.php

@@ -15,6 +15,11 @@
  */
 class CRM_Utils_ICalendarTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @return array
    */

tests/phpunit/CRM/Utils/JSTest.php

@@ -15,6 +15,14 @@
  */
 class CRM_Utils_JSTest extends CiviUnitTestCase {
 
+  /**
+   * Set up for tests.
+   */
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @return array
    */

tests/phpunit/CRM/Utils/LazyArrayTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_LazyArrayTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   public function testAssoc() {
     $l = $this->createFruitBasket();
     $this->assertFalse($l->isLoaded());

tests/phpunit/CRM/Utils/Mail/FilteredPearMailerTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_Mail_FilteredPearMailerTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   public function testFilter() {
     $mock = new class() extends \Mail {
       public $buf = [];

tests/phpunit/CRM/Utils/Mail/IncomingTest.php

@@ -32,6 +32,7 @@ class CRM_Utils_Mail_IncomingTest extends CiviUnitTestCase {
   protected $name;
 
   public function setUp(): void {
+    $this->useTransaction();
     parent::setUp();
 
     $rand = rand(0, 1000);

tests/phpunit/CRM/Utils/MailTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_MailTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * Test case for add( )
    * test with empty params.

tests/phpunit/CRM/Utils/MoneyTest.php

@@ -7,6 +7,11 @@
  */
 class CRM_Utils_MoneyTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @dataProvider subtractCurrenciesDataProvider
    * @param string $leftOp

tests/phpunit/CRM/Utils/NumberTest.php

@@ -6,6 +6,11 @@
  */
 class CRM_Utils_NumberTest extends CiviUnitTestCase {
 
+  public function setUp(): void {
+    $this->useTransaction();
+    parent::setUp();
+  }
+
   /**
    * @return array
    */