Skip to content

Commit

Permalink
fix: fixup previous commit
Browse files Browse the repository at this point in the history
  • Loading branch information
OmarIthawi committed Jan 9, 2025
1 parent 5d31a0a commit 6836c00
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions scripts/tests/test_release_project_sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -180,9 +180,6 @@ def test_translations_entry_update_translation():
assert updates == {
'reviewed': True,
}
assert current_translation.updates == {
'reviewed': True,
}


def test_translations_entry_more_recent_review():
Expand Down Expand Up @@ -210,7 +207,6 @@ def test_translations_entry_more_recent_review():

assert status == 'no-op'
assert not updates, 'updates should be empty'
assert not release_translation.updates, 'save() should not be called'


def test_translations_entry_dry_run():
Expand All @@ -235,8 +231,9 @@ def test_translations_entry_dry_run():
)

assert status == 'update-dry-run'
assert not updates, 'updates should be empty'
assert not current_translation.updates, 'save() should not be called in --dry-run mode'
assert updates == {
'reviewed': True,
}, 'Planned updates but never saved because of dry-run'


def test_translations_entry_different_translation():
Expand All @@ -260,5 +257,5 @@ def test_translations_entry_different_translation():
translation_from_old_project, current_translation
)

assert (status, updates) == ('no-op', {})
assert not current_translation.updates, 'save() should not be called in --dry-run mode'
assert status == 'no-op'
assert updates == {}, ''

0 comments on commit 6836c00

Please sign in to comment.