From 08803e8958b76c9368607a52e5b4aa10682842b2 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Mar 02 2015 12:04:07 +0000 Subject: Document why we cannot tests a line of code in notify_new_issue --- diff --git a/progit/lib/notify.py b/progit/lib/notify.py index 93f22d9..3a7f6dc 100644 --- a/progit/lib/notify.py +++ b/progit/lib/notify.py @@ -117,6 +117,9 @@ New issue: ) mail_to = set([cmt.user.emails[0].email for cmt in issue.comments]) mail_to.add(issue.project.user.emails[0].email) + # I will keeps this here as things might change in the future, but + # basically we should never enter this `if` as currently an issue cannot + # be created assigned to someone, it is always a 2 steps process. if issue.assignee: mail_to.add(issue.assignee.emails[0].email) for prouser in issue.project.users: