Make sure we backfilled at least one dead tag

This commit is contained in:
Joseph Schorr 2019-01-16 11:24:33 -05:00
parent a1caefcabe
commit d49fd1c727

View file

@ -39,6 +39,8 @@ def test_tagbackfillworker(clear_all_rows, initialized_db):
TagManifestToManifest.delete().execute() TagManifestToManifest.delete().execute()
Manifest.delete().execute() Manifest.delete().execute()
found_dead_tag = False
for repository_tag in list(RepositoryTag.select()): for repository_tag in list(RepositoryTag.select()):
# Backfill the tag. # Backfill the tag.
assert backfill_tag(repository_tag) assert backfill_tag(repository_tag)
@ -63,6 +65,7 @@ def test_tagbackfillworker(clear_all_rows, initialized_db):
assert tag.lifetime_end_ms is None assert tag.lifetime_end_ms is None
else: else:
assert tag.lifetime_end_ms == (repository_tag.lifetime_end_ts * 1000) assert tag.lifetime_end_ms == (repository_tag.lifetime_end_ts * 1000)
found_dead_tag = True
assert tag.manifest assert tag.manifest
@ -100,6 +103,8 @@ def test_tagbackfillworker(clear_all_rows, initialized_db):
in ManifestLabel.select().where(ManifestLabel.manifest == manifest_row)} in ManifestLabel.select().where(ManifestLabel.manifest == manifest_row)}
assert found_labels == expected_labels assert found_labels == expected_labels
assert found_dead_tag
def test_manifestbackfillworker_broken_manifest(clear_rows, initialized_db): def test_manifestbackfillworker_broken_manifest(clear_rows, initialized_db):
# Delete existing tag manifest so we can reuse the tag. # Delete existing tag manifest so we can reuse the tag.