Merge pull request #14454 from eileenmcnaughton/php72

(NFC) Remove without param tests

tests/phpunit/api/v3/SyntaxConformanceTest.php

@@ -782,28 +782,6 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
     $this->assertRegExp('/API (.*) does not exist/', $result['error_message']);
   }
 
-  /**
-   * @dataProvider entities
-   * @param $Entity
-   */
-  public function testWithoutParam_get($Entity) {
-    // should get php complaining that a param is missing
-    try {
-      $result = civicrm_api($Entity, 'Get');
-      $this->fail('Expected an exception. No exception was thrown.');
-    }
-    // As of php7.1 a new Exception is thrown by PHP ArgumentCountError when not enough params are passed.
-    catch (ArgumentCountError $e) {
-      /* ok */
-    }
-    catch (PHPUnit\Framework\Error $e) {
-      /* ok */
-    }
-    catch (PHPUnit_Framework_Error $e) {
-      /* ok */
-    }
-  }
-
   /**
    * @dataProvider entities
    * @param $Entity
@@ -1594,28 +1572,6 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase {
     $this->assertContains(strtolower("API ($Entity, Delete) does not exist"), strtolower($result['error_message']));
   }
 
-  /**
-   * @dataProvider entities
-   * @param $Entity
-   */
-  public function testWithoutParam_delete($Entity) {
-    // should delete php complaining that a param is missing
-    try {
-      $result = civicrm_api($Entity, 'Delete');
-      $this->fail('Expected an exception. No exception was thrown.');
-    }
-    // As of php7.1 a new Exception is thrown by PHP ArgumentCountError when not enough params are passed.
-    catch (ArgumentCountError $e) {
-      /* ok */
-    }
-    catch (PHPUnit\Framework\Error $e) {
-      /* ok */
-    }
-    catch (PHPUnit_Framework_Error $e) {
-      /* ok */
-    }
-  }
-
   /**
    * @dataProvider entities_delete
    * @param $Entity