- Issue created by @Anybody
- First commit to issue fork.
- 🇩🇪Germany Grevil
Failing test is related to the deprecated "createUser" call in EstimatorTest.php:
Calling createUser() with $values as the first parameter is deprecated in drupal:10.1.0 and will be removed from drupal:11.0.0. Use createUser(array $permissions = [], $name = NULL, $admin = FALSE, array $values = []) instead. See https://www.drupal.org/node/3330762 →
I'll create a follow-up issue for this.
- 🇩🇪Germany Grevil
Ok, no need to create a follow-up issue. We should fix this in 📌 Automated Drupal 11 compatibility fixes for commerce_cart_estimate Needs review .
- 🇩🇪Germany Anybody Porta Westfalica
Great work once again @grevil! I left some comments!
As we're using the same defaults as before, this needs no update path, I think. Will be just fine like that imho.
- 🇩🇪Germany Grevil
Tests succeed again, as 🐛 Tests: Fix incorrect "createUser" call Active was just merged.
- 🇩🇪Germany Anybody Porta Westfalica
Perfect, hope it can be merged soon :)
- 🇩🇪Germany Anybody Porta Westfalica
Current state as patch for composer-patches attached.