Deprecate Form_Event::testSubmit, migrate test to full clow
authorEileen McNaughton <emcnaughton@wikimedia.org>
Mon, 18 Sep 2023 02:03:26 +0000 (14:03 +1200)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Mon, 18 Sep 2023 04:54:26 +0000 (16:54 +1200)
commit5b5832e67563622fbc8834546cb2f3af4c2738d7
treea2907c93103d377baeffd633825324216b332af3
parent301eb61cc11a96f12ad11f2acaa525155ffe70fe
Deprecate Form_Event::testSubmit, migrate test to full clow

In the procss I looked at what the test is actually testing & the goal of the test is ensure the
doPayment contract is met, specifically regarding contactID
https://docs.civicrm.org/dev/en/latest/extensions/payment-processors/create/#getpaymentformfields

I updated the test to use the alterPaymentProcessorParams hook to actually check for that.
CRM/Core/Form.php
CRM/Event/Form/Registration/Confirm.php
tests/phpunit/CRM/Event/Form/Registration/ConfirmTest.php