Skip to content

Commit

Permalink
CRM-17860, CRM-18231 - JobProcessMailingTest - Re-remove require_once
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
totten authored and sluc23 committed Jan 10, 2018
1 parent 62210cc commit e6cc725
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions tests/phpunit/api/v3/JobProcessMailingTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit e6cc725

Please sign in to comment.