Civi\API\ExternalBatch - Improve test for variables_order/$_ENV
authorTim Otten <totten@civicrm.org>
Mon, 13 Mar 2017 04:45:00 +0000 (21:45 -0700)
committerTim Otten <totten@civicrm.org>
Mon, 13 Mar 2017 04:45:00 +0000 (21:45 -0700)
commit40406b562fff2686d65bf7b837b501c65c2ff3a6
tree4a309d8f84098736be54bcc65b00f2bb72d51538
parentdad2a90eefc9e6029b7fe6865b8f6c17ee0b95bd
Civi\API\ExternalBatch - Improve test for variables_order/$_ENV

The recent revision #9595 updated `ExternalBatch` to produce a more
informative error message.  However, in the test environment used by
`flexmailer`, there was exactly 1 value in `$_ENV` (even if
`variables_order` was misconfigured).

This should make the test a bit harder to trip-up by affirmatively checking
for the most common environment variable.

Review note: In the known universe, the only direct references to
`ExternalBatch` are in `CiviUnitTestCase`, so this shouldn't impact any
runtime logic.
Civi/API/ExternalBatch.php