- Issue created by @Grevil
- Merge request !28Issue #3445439: Implement basic tests and fix schema errors β (Merged) created by Grevil
- Status changed to Needs review
about 2 months ago 10:57am 6 May 2024 - π©πͺGermany Grevil
Ok, should be fixed now.
I spent half an hour, trying to fix those migration tests as well, but to no avail.
Quite a weird error as well:
PHPUnit\Framework\Exception: PHP Fatal error: Uncaught AssertionError: The container was serialized. in /var/www/html/web/core/lib/Drupal/Core/DependencyInjection/ContainerBuilder.php:88
Which is thrown, when executing the migration. I don't think this is worthwhile. Please review!
- Assigned to Grevil
- Status changed to Needs work
about 2 months ago 11:01am 6 May 2024 - π©πͺGermany Anybody Porta Westfalica
@Grevil: Please just comment out the migration tests with a link to this issue. We shouldn't delete them. perhaps someone wants to fix them in the future.
- π©πͺGermany Grevil
Ok, commenting these tests out, will give a warning on phpunit, because there are no existent tests in the class, but I can simply create fake tests instead.
- Issue was unassigned.
- Status changed to Needs review
about 2 months ago 11:17am 6 May 2024 - π©πͺGermany Anybody Porta Westfalica
I think it would be better to just remove the "Test" suffix or add another "Disabled" suffix to the class but this solution is also okay for me here.
- Status changed to RTBC
about 2 months ago 11:54am 6 May 2024 -
Grevil β
committed 41befa28 on 2.x
Issue #3445439: Implement basic tests and fix schema errors
-
Grevil β
committed 41befa28 on 2.x
- Status changed to Fixed
about 2 months ago 12:08pm 6 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.