- Issue created by @spuky
- Merge request !115fix easy_breadcrumb.settings.yml to match the order expected by the tests β (Merged) created by spuky
- last update
7 months ago 6 pass, 2 fail - Issue was unassigned.
- Status changed to Needs review
7 months ago 4:16pm 12 June 2024 - π©πͺGermany spuky
fixes the testAdministration Test in the testsuite to have one more passing unittest by reordering the easy_breadcrumb.settings.yml to match the order expected by the test
- last update
7 months ago 6 pass, 2 fail - πΊπΈUnited States Greg Boggs Portland Oregon
Thanks for working on the tests Spuky. Looks like they are still failing:
1) Drupal\Tests\easy_breadcrumb\Functional\EasyBreadcrumbConfigureTest::test8006DefaultConfigurationUpdate
InvalidArgumentException: The configuration property capitalizator_ignored_words.0.0 doesn't exist. - π©πͺGermany spuky
yes 2 of 3 are still failing...
The testAdministration one is the one that i fixed..
The other two are testing Update code from 3 years ago...
and someone would need to create DB dumps of pre update DBs to import the old DB run the upgrade during the testsmy suggestion would be to get rid of those 2 have a working test suite (that still needs improvement...)
- πΊπΈUnited States Greg Boggs Portland Oregon
That sounds perfect. Please do if you have the time and energy!
- Open on Drupal.org βCore: 10.2.1 + Environment: PHP 8.1 & MySQL 8last update
7 months ago Not currently mergeable. - last update
7 months ago 6 pass - last update
7 months ago 7 pass - last update
7 months ago 7 pass - last update
7 months ago 7 pass - last update
7 months ago 7 pass - last update
7 months ago 7 pass - Status changed to Fixed
7 months ago 4:50pm 26 June 2024 Automatically closed - issue fixed for 2 weeks with no activity.