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

Add framework for migration tests in SQLAlchemy #2392

Conversation

giovannipizzi
Copy link
Member

This fixes #2391. In particular it adds a framework for testing migrations
in SQLAlchemy/Alembic, and ports one test from Django, also to act as an
example and to verify that the migration testing is properly working.

I've also removed old files that were running some tests but actually were just
running on a set of example migrations and not on those of AiiDA.

@giovannipizzi giovannipizzi requested a review from sphuber January 15, 2019 18:33
@giovannipizzi giovannipizzi force-pushed the fix_2391_sqla_migrations_tests branch from 363e03e to 6f95622 Compare January 15, 2019 18:53
@coveralls
Copy link

coveralls commented Jan 15, 2019

Coverage Status

Coverage decreased (-0.003%) to 68.718% when pulling 50a58d7 on giovannipizzi:fix_2391_sqla_migrations_tests into a2456ab on aiidateam:provenance_redesign.

@giovannipizzi giovannipizzi force-pushed the fix_2391_sqla_migrations_tests branch from 6f95622 to 23856e3 Compare January 15, 2019 21:36
This fixes aiidateam#2391. In particular it adds a framework for testing migrations
in SQLAlchemy/Alembic, and ports one test from Django, also to act as an
example and to verify that the migration testing is properly working.

I've also removed old files that were running some tests but actually were just
running on a set of example migrations and not on those of AiiDA.
@giovannipizzi giovannipizzi force-pushed the fix_2391_sqla_migrations_tests branch from 23856e3 to 50a58d7 Compare January 15, 2019 21:46
@sphuber sphuber merged commit 487c112 into aiidateam:provenance_redesign Jan 16, 2019
@giovannipizzi giovannipizzi deleted the fix_2391_sqla_migrations_tests branch April 16, 2019 10:43
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