From 2fd826db63897ea5f2789f91d687c8b03bc152ed Mon Sep 17 00:00:00 2001 From: Tim Otten Date: Fri, 21 Feb 2020 00:34:40 -0800 Subject: [PATCH] Update recently-added PathUrlTest to be more representative Before ------ The test generates the URL for an example page (`civicrm/mailing/subscribe`) by calling `url()`. It does not explicitly set the `url(...$frontend...)` option, so it defaults to `FALSE`. By contrast, the other places which make URLs for `civicrm/mailing/subscribe` (eg `CRM/Mailing/BAO/Mailing.php` and `CRM/Utils/Token.php`) do set `$frontend` to `TRUE`. After ----- The test behaves the same way as other code. --- tests/phpunit/E2E/Core/PathUrlTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/phpunit/E2E/Core/PathUrlTest.php b/tests/phpunit/E2E/Core/PathUrlTest.php index 83b0b233419b..8855c559e142 100644 --- a/tests/phpunit/E2E/Core/PathUrlTest.php +++ b/tests/phpunit/E2E/Core/PathUrlTest.php @@ -18,7 +18,7 @@ class PathUrlTest extends \CiviEndToEndTestCase { */ public function testSystemRouter() { $this->assertUrlContentRegex(';class="CRM_Mailing_Form_Subscribe";', - \CRM_Utils_System::url('civicrm/mailing/subscribe', 'reset=1', TRUE, NULL, FALSE)); + \CRM_Utils_System::url('civicrm/mailing/subscribe', 'reset=1', TRUE, NULL, FALSE, TRUE)); } /**