Opt out financial checking on membership_payment test & acl test

civicrm · Jul 26, 2021 · 27355f0 · 27355f0
1 parent 6e87fe1
commit 27355f0
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/tests/phpunit/api/v3/ACLPermissionTest.php b/tests/phpunit/api/v3/ACLPermissionTest.php
@@ -25,6 +25,16 @@ class api_v3_ACLPermissionTest extends CiviUnitTestCase {
 
   use Civi\Test\ACLPermissionTrait;
 
+  /**
+   * Should financials be checked after the test but before tear down.
+   *
+   * The setup methodology in this class bypasses valid financial creation
+   * so we don't check.
+   *
+   * @var bool
+   */
+  protected $isValidateFinancialsOnPostAssert = FALSE;
+
   public $DBResetRequired = FALSE;
   protected $_entity;
 

diff --git a/tests/phpunit/api/v3/MembershipPaymentTest.php b/tests/phpunit/api/v3/MembershipPaymentTest.php
@@ -23,6 +23,17 @@ class api_v3_MembershipPaymentTest extends CiviUnitTestCase {
   protected $_membershipStatusID;
   protected $_contribution = [];
 
+  /**
+   * Should financials be checked after the test but before tear down.
+   *
+   * This test class is opted out as this method should not be called outside
+   * of the LineItem::create function and the test is artificial & not creating
+   * valid financials.
+   *
+   * @var bool
+   */
+  protected $isValidateFinancialsOnPostAssert = FALSE;
+
   public function setUp(): void {
     parent::setUp();
     $this->useTransaction(TRUE);