- Issue created by @Grevil
- Merge request !28Issue #3445439: Implement basic tests and fix schema errors → (Merged) created by Grevil
- Status changed to Needs review
11 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
11 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
11 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
11 months ago 11:54am 6 May 2024 - Status changed to Fixed
11 months ago 12:08pm 6 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.