- Issue created by @Grevil
- @grevil opened merge request.
- Status changed to Needs review
over 1 year ago 12:24pm 6 June 2023 - π©πͺGermany Grevil
Alright, the test output did not show any schema errors! Please review!
- π©πͺGermany Grevil
Weirdly enough, I found an uninstallation bug, while manually uninstalling the module. I have no idea, why this error is not displayed, when running the tests, but I could even replicate it, when resetting the Database and "only" having this module activated.
I created a separate issue for that: π Uninstalling the module throws an error Closed: cannot reproduce
- Status changed to RTBC
over 1 year ago 4:00pm 23 June 2023 - π©πͺGermany Anybody Porta Westfalica
Well done! This ensures the module installs and uninstalls without errors and allows us to see schema errors. Helpful addition @Grevil! :)
- Assigned to Grevil
- Status changed to Needs work
over 1 year ago 8:07am 26 June 2023 - π©πͺGermany Grevil
I'll quickly invest a few minutes on trying to replicate π Uninstalling the module throws an error Closed: cannot reproduce through tests.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 8:44am 26 June 2023 - π©πͺGermany Grevil
OK, I added two more tests. I couldn't replicate the issue described in π Uninstalling the module throws an error Closed: cannot reproduce any more (Neither through the provided test nor manually).
Please review the newly added tests!
- Status changed to RTBC
over 1 year ago 8:47am 26 June 2023 - π©πͺGermany Anybody Porta Westfalica
Well done @Grevil! RTBC!
I closed the other issue. - π©πͺGermany Anybody Porta Westfalica
@Maintainers: Could you please enable automated tests at: https://www.drupal.org/node/3039366/qa β
Thanks :)
- First commit to issue fork.
-
nginex β
committed 47f6e690 on 2.x authored by
Grevil β
Issue #3365072: Add basic installation tests
-
nginex β
committed 47f6e690 on 2.x authored by
Grevil β
- πΊπ¦Ukraine nginex
The tests are enabled for 2.x.
Thanks for the patch.
- Status changed to Fixed
over 1 year ago 7:09am 3 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.