Summary: | The DB upgrade process is not tested | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Test Suite | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | dcook, kyle, m.de.rooy, martin.renvoize, nick, pedro.amorim, tomascohen |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
See Also: |
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34276 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34516 |
||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 34684, 34685 | ||
Bug Blocks: | 34841 |
Description
Jonathan Druart
2023-09-01 07:37:21 UTC
This is not very nice, but it will be efficient enough for now. I am suggesting to run the DB upgrade twice, before we run the test suite on Jenkins. The plan here is: 1. Push bug 34684 and bug 34685 that are needed 2. Merge https://gitlab.com/koha-community/koha-misc4dev/-/issues/72 It includes the DB starting point (19.11.00) on which we will run the DB upgrade, twice. 3. Adjust the Jenkins job (there is a new flag --run-db-upgrade-only that will need to be used from, at least, MySQL 8). If we had that before, we would have caught bug 34276. I concur. This sounds like a good course of action! Ran the test plan, works as expected. This is a good addition! Worked well for me too, seems a decent course of action. Thanks for the feedback. I am going to merge the misc4dev's merge request! Koha_Master_My8/1257 run successfully for the first time using this. (In reply to Jonathan Druart from comment #7) > Koha_Master_My8/1257 run successfully for the first time using this. I moved the "run db upgrade" step before running the other tests (Issue #399: Run the DB upgrade before running the tests) and everything is broken now (see Koha_Master_My8/1258). I will have a look later today. |