Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testupgrade #1993

Open
wants to merge 3 commits into
base: maintenance/gramps60
Choose a base branch
from

Conversation

prculley
Copy link
Contributor

This adds some tests of the db upgrade process. I have added several small dbs created with older versions of Gramps.
The imports_test code was updated to allow zips of these dbs as inputs to the test. I chose to modify imports_test as there was substantial code reuse if I created a new module.

As usual the test compares the extracted and loaded db with the matching Gramps XML file and reports any differences. This should fail if the upgrade code crashes or misses something important.

I've tested this on top of PR#1992 and it seems to work.

@Nick-Hall Nick-Hall added this to the v6.0 milestone Feb 19, 2025
@emyoulation
Copy link
Contributor

The imports_test code was updated to allow zips of these dbs as inputs to the test.

Nice! I assume that this accepts the backups automatically generated during a Convert schema?

@prculley
Copy link
Contributor Author

It does not accept zips created during upgrade directly. You have to massage the names of the zip and the db directory inside the zip to says something like "imp_???????" in order for the test files to be recognized. And of course, you need a .gramps file with the same name to compare the db with.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants