CRM-17860 - SettingTest::testCreateInvalidURLSettings - Fail for the right reasons
authorTim Otten <totten@civicrm.org>
Fri, 22 Jan 2016 23:17:33 +0000 (15:17 -0800)
committerTim Otten <totten@civicrm.org>
Tue, 2 Feb 2016 04:56:23 +0000 (21:56 -0700)
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

index c1335ddb3238c358be1204b70574bc70b4a4eb45..2cf55cd6e66d325e69d5d5729f013c4f3b4afe9a 100644 (file)
@@ -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(