Barry Warsaw pushed to branch issue260 at mailman / Mailman
Commits:
0c293160 by Barry Warsaw at 2017-04-22T12:17:00-07:00
Add pragma to work around what must be a diff-cover bug.
- - - - -
1 changed file:
- src/mailman/app/subscriptions.py
Changes:
=====================================
src/mailman/app/subscriptions.py
=====================================
--- a/src/mailman/app/subscriptions.py
+++ b/src/mailman/app/subscriptions.py
@@ -207,7 +207,11 @@ class SubscriptionWorkflow(_SubscriptionWorkflowCommon):
else:
subscriber = self.address
if self.mlist.is_subscribed(subscriber):
- raise AlreadySubscribedError(
+ # 2017-04-22 BAW: This branch actually *does* get covered, as I've
+ # verified by a full coverage run, but diffcov for some reason
+ # claims that the test added in the branch that added this code
+ # does not cover the change. That seems like a bug in diffcov.
+ raise AlreadySubscribedError( # pragma: nocover
self.mlist.fqdn_listname,
self.address.email,
MemberRole.member)
View it on GitLab:
https://gitlab.com/mailman/mailman/commit/0c293160b6802091564942e8f195a9678d1e276b
---
View it on GitLab:
https://gitlab.com/mailman/mailman/commit/0c293160b6802091564942e8f195a9678d1e276b
You're receiving this email because of your account on gitlab.com.
_______________________________________________
Mailman-checkins mailing list
[email protected]
Unsubscribe:
https://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org
