From 20ec56b64faa4e5ccbf190db914851cc7aed0ceb Mon Sep 17 00:00:00 2001 From: Tim Otten Date: Tue, 7 Nov 2017 22:47:19 -0800 Subject: [PATCH] CRM-17860, CRM-18231 - JobProcessMailingTest - Re-remove require_once Overview -------- Remove an unnecessary line from test suite. Before ------ The test worked when executed as part of the `api_v3` suite, but it failed when run individually (because the `require_once` doesn't work). Moreover, the tests for `org.civicrm.flexmailer` failed After ----- The tests work on their own. Comment ------- This change deals entirely with the packaging/framing of a test-case. It neither (a) involves the runtime behavior of the application nor (b) the semantics/behavior of the test. Consequently, it should be safe to merge as long as the test-runner remains happy. --- tests/phpunit/api/v3/JobProcessMailingTest.php | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/phpunit/api/v3/JobProcessMailingTest.php b/tests/phpunit/api/v3/JobProcessMailingTest.php index 08d4c2600d..202a575e3e 100644 --- a/tests/phpunit/api/v3/JobProcessMailingTest.php +++ b/tests/phpunit/api/v3/JobProcessMailingTest.php @@ -36,9 +36,6 @@ * */ -//@todo - why doesn't class loader find these (I tried renaming) -require_once 'CiviTest/CiviMailUtils.php'; - /** * Class api_v3_JobTest * @group headless -- 2.25.1