fix: remove TODO comments and noop tests from test_notify

Signed-off-by: Kevin Morris <kevr@0cost.org>
This commit is contained in:
Kevin Morris 2021-11-29 16:20:36 -08:00
parent 436d742017
commit 44f2366675
No known key found for this signature in database
GPG key ID: F7E46DED420788F3

View file

@ -262,7 +262,6 @@ The package {pkgbase.Name} [1] was disowned by {user2.Username} [2].
def test_comaintainer_addition(user: User, pkgbases: List[PackageBase]):
# TODO: Add this in fastapi code!
pkgbase = pkgbases[0]
notif = notify.ComaintainerAddNotification(user.ID, pkgbase.ID)
notif.send()
@ -282,7 +281,6 @@ You were added to the co-maintainer list of {pkgbase.Name} [1].
def test_comaintainer_removal(user: User, pkgbases: List[PackageBase]):
# TODO: Add this in fastapi code!
pkgbase = pkgbases[0]
notif = notify.ComaintainerRemoveNotification(user.ID, pkgbase.ID)
notif.send()
@ -417,14 +415,9 @@ Request #{pkgreq.ID} has been rejected by {user2.Username} [1].
assert email.body == expected
def test_close_request_auto_accept():
pass
def test_close_request_comaintainer_cc(user: User, user2: User,
pkgreq: PackageRequest,
pkgbases: List[PackageBase]):
# TODO: Check this in fastapi code!
pkgbase = pkgbases[0]
with db.begin():
db.create(models.PackageComaintainer, PackageBase=pkgbase,