Stop forcing apiVersion to 3 in setUp
authorEileen McNaughton <emcnaughton@wikimedia.org>
Sat, 2 Dec 2023 04:33:47 +0000 (17:33 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Sun, 3 Dec 2023 02:01:44 +0000 (15:01 +1300)
It should be possible to override it at a class level...

tests/phpunit/CiviTest/CiviUnitTestCase.php
tests/phpunit/api/v3/ActivityContactTest.php

index fd65764ef6b3a69242619736398e51350f55713e..b643348dc93d338bc524c8250d94ea00b0acec31 100644 (file)
@@ -295,8 +295,6 @@ class CiviUnitTestCase extends PHPUnit\Framework\TestCase {
     parent::setUp();
     CRM_Core_Session::singleton()->set('userID');
 
-    $this->_apiversion = 3;
-
     //  Use a temporary file for STDIN
     $GLOBALS['stdin'] = tmpfile();
     if ($GLOBALS['stdin'] === FALSE) {
index a848a6896cd73a817e86f444906f0eb67c530ab7..b7de1795c4adbc5630b64217f5add8df1c6d3edf 100644 (file)
@@ -18,7 +18,7 @@
  * @group headless
  */
 class api_v3_ActivityContactTest extends CiviUnitTestCase {
-  protected $_apiversion;
+  protected $_apiversion = 3;
   protected $_contactID;
   protected $_activityID;
   protected $_params;
@@ -42,9 +42,8 @@ class api_v3_ActivityContactTest extends CiviUnitTestCase {
    * @param int $version
    *
    * @dataProvider versionThreeAndFour
-   * @throws \CRM_Core_Exception
    */
-  public function testCreateActivityContact($version) {
+  public function testCreateActivityContact(int $version): void {
     $this->_apiversion = $version;
 
     $result = $this->callAPISuccess('ActivityContact', 'create', $this->_params);
@@ -61,9 +60,8 @@ class api_v3_ActivityContactTest extends CiviUnitTestCase {
    * @param int $version
    *
    * @dataProvider versionThreeAndFour
-   * @throws \CRM_Core_Exception
    */
-  public function testDeleteActivityContact($version) {
+  public function testDeleteActivityContact(int $version): void {
     $this->_apiversion = $version;
     //create one
     $create = $this->callAPISuccess('activity_contact', 'create', $this->_params);