From de8bb6834cb92adeddd4ce51d9316c34bad110ba Mon Sep 17 00:00:00 2001 From: Seamus Lee Date: Thu, 27 Jul 2017 10:31:52 +0000 Subject: [PATCH] Add in code comment about it being a PHP7.1 issue --- tests/phpunit/api/v3/SyntaxConformanceTest.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/phpunit/api/v3/SyntaxConformanceTest.php b/tests/phpunit/api/v3/SyntaxConformanceTest.php index 59be3fc924..a15da8e9e1 100644 --- a/tests/phpunit/api/v3/SyntaxConformanceTest.php +++ b/tests/phpunit/api/v3/SyntaxConformanceTest.php @@ -688,6 +688,7 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase { $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 */ } @@ -1403,6 +1404,7 @@ class api_v3_SyntaxConformanceTest extends CiviUnitTestCase { $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 */ } -- 2.25.1