- Issue created by @eduardo morales alberti
- Merge request !62Issue #3397608: Argument #2 ($value) must be of type string, null given β (Closed) created by eduardo morales alberti
- Status changed to Needs work
over 1 year ago 11:06am 30 October 2023 - πͺπΈSpain eduardo morales alberti Spain, πͺπΊ
We are not sure why the PHPUnit tests are failing:
https://git.drupalcode.org/issue/reroute_email-3397608/-/jobs/249613There were 8 failures: 1) Drupal\Tests\reroute_email_symfony_mailer\Functional\SymfonyMailerTestEmailFormTest::testFormTestEmail with data set #0 (true, '', array('email@original-destination.com', 'iDnwja5g@not-allowed.com', 'wPZeBs2V@not-allowed.com', 'Test Reroute Email Test Email Form', 'Testing email rerouting and t...l form'), true) Test was run in child process and ended unexpectedly
- πͺπΈSpain eduardo morales alberti Spain, πͺπΊ
We created a new branch with no changes, only to compare with the 2.3.x and seems like the phpstan and the Symfony mailer tests are failing. https://git.drupalcode.org/issue/reroute_email-3397608/-/pipelines/41453
- πͺπΈSpain eduardo morales alberti Spain, πͺπΊ
Pending review Symfony Mailer tests on branch 2.3.x https://www.drupal.org/project/reroute_email/issues/3391116#comment-1529... π Add tests for Drupal Symfony Mailer Active
- First commit to issue fork.
- Status changed to Fixed
about 1 year ago 12:17pm 9 December 2023 - πΊπ¦Ukraine bohart Lutsk, Ukraine
@Eduardo Morales Alberti, thanks for your contribution!
It looks like the issue was fixed as a part of π Adapt ContactFormTest, CaseSensitivityTest, MailKeysTest, TestEmailFormTest tests for 'Reroute Email (Symfony Mailer support)' module. Fixed
Committed to 2.3.x dev branch and will be a part of the next 2.3 series releases.Please re-open the issue or raise a new one if any new problems occur.
Thanks! Automatically closed - issue fixed for 2 weeks with no activity.