Skip to content

Commit

Permalink
set explicit dry_run value
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisjsewell committed Sep 1, 2021
1 parent cccbcc9 commit e58d7e2
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion aiida/backends/general/migrations/duplicate_uuids.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ def _apply_new_uuid_mapping(table, mapping):
cursor.execute(query)


def deduplicate_uuids(table=None, dry_run=False):
def deduplicate_uuids(table=None, dry_run=True):
"""Detect and solve entities with duplicate UUIDs in a given database table.
Before aiida-core v1.0.0, there was no uniqueness constraint on the UUID column of the node table in the database
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ def setUpBeforeMigration(self):

# Now run the function responsible for solving duplicate UUIDs which would also be called by the user
# through the `verdi database integrity detect-duplicate-uuid` command
deduplicate_uuids(table='db_dbnode')
deduplicate_uuids(table='db_dbnode', dry_run=False)

def test_deduplicated_uuids(self):
"""Verify that after the migration, all expected nodes are still there with unique UUIDs."""
Expand Down

0 comments on commit e58d7e2

Please sign in to comment.