From a552196023a2bb3ad29c10300d39acdafae29fed Mon Sep 17 00:00:00 2001 From: Tim Otten Date: Fri, 22 Jan 2016 15:17:33 -0800 Subject: [PATCH] CRM-17860 - SettingTest::testCreateInvalidURLSettings - Fail for the right reasons The test checks that invalid URLs return an error. The URLs given where invalid -- but for the wrong reason. In unit-tests, `CIVICRM_UF_BASEURL` defaulted to the invalid `http://FIX ME`. After unifying `civicrm.settings.php`, the `CIVICRM_UF_BASEURL` was valid. The intention in the test was to include a weird character (`*`) to make the test fail, but we need a weirder/more-incorrect character. --- tests/phpunit/api/v3/SettingTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/phpunit/api/v3/SettingTest.php b/tests/phpunit/api/v3/SettingTest.php index c1335ddb32..2cf55cd6e6 100644 --- a/tests/phpunit/api/v3/SettingTest.php +++ b/tests/phpunit/api/v3/SettingTest.php @@ -235,7 +235,7 @@ class api_v3_SettingTest extends CiviUnitTestCase { public function testCreateInvalidURLSettings() { $params = array( 'domain_id' => $this->_domainID2, - 'userFrameworkResourceURL' => 'dfhkd*hfd', + 'userFrameworkResourceURL' => 'dfhkd hfd', ); $result = $this->callAPIFailure('setting', 'create', $params); $params = array( -- 2.25.1