- Assigned to fathershawn
- Status changed to Needs review
almost 2 years ago 12:11pm 1 February 2023 - 🇺🇸United States fathershawn New York
I love collaborating in community! Thanks so much for the patches! I'd like to figure out how two write a test for this case.
-
FatherShawn →
committed c217b29e on 4.0.x
Issue #3328368 by zerbash, Evaldas Užkuras, FatherShawn: Can't update if...
-
FatherShawn →
committed c217b29e on 4.0.x
- Status changed to Fixed
almost 2 years ago 12:55pm 3 February 2023 - 🇺🇸United States fathershawn New York
Thank you both. I learned how to write a test for this based on
\Drupal\Tests\system\Functional\UpdateSystem\UpdatePostUpdateTest
. Once the test was working it caught the reported error. Evaldas' fix was simpler and in line with the typing so I used that, the test passed and I've issued an update. Automatically closed - issue fixed for 2 weeks with no activity.