Concurrency Tests - Update ExternalBatch for symfony/proces v4.1+
authorTim Otten <totten@civicrm.org>
Tue, 15 Aug 2023 00:01:20 +0000 (17:01 -0700)
committerTim Otten <totten@civicrm.org>
Tue, 15 Aug 2023 00:13:05 +0000 (17:13 -0700)
commit8ac67afe11de436daf815a6dd834a43198e631de
treeba6fcbc6f0a3968de0ce4d4f9ca5b353f6b1eb60
parent647267c92c1f236ce10f4af70e6f937d74f5fada
Concurrency Tests - Update ExternalBatch for symfony/proces v4.1+

Before
------

`api_v3_JobProcessMailingTest::testConcurrency()` and `Civi\FlexMailer\ConcurrencyTest`
fail when executed on Drupal 10.

After
-----

Pass when executed on Drupal 10

Technical Details
-----------------

`symfony/process` v4.1 deprecated the usage of `new Process(string)` and
added `Process::fromShellCommandline(string)`. The `new Process(string)` was
dropped from v5.x and v6.x. So you get an unconditional failure on D10. (On
other platforms, it can also fail... depending on how you deal with
deprecations.)

In `civicrm-core:composer.json`, the minimum mversion of `symfony/process` is `4.4`.
So that means we can use the newer `Process::fromShellCommandline(string)`.
Civi/API/ExternalBatch.php