Merge pull request #997 from lucc/tests/comma

Add tests for reply header creation

1 files changed, 14 insertions, 13 deletions

diff --git a/alot/commands/thread.py b/alot/commands/thread.py
index 6dd4f248..79fb8415 100644
--- a/alot/commands/thread.py
+++ b/alot/commands/thread.py
@@ -231,19 +231,6 @@ class ReplyCommand(Command):
                         my_addresses, mail.get_all('Cc', []))
                     envelope.add('Cc', decode_header(', '.join(cc)))
 
-        def ensure_unique_address(recipients):
-            """
-            clean up a list of name,address pairs so that
-            no address appears multiple times.
-            """
-            res = dict()
-            for name, address in getaddresses(recipients):
-                res[address] = name
-            urecipients = ['"{}" <{}>'.format(n, a) if n != ''
-                           else a
-                           for a, n in res.iteritems()]
-            return sorted(urecipients)
-
         to = ', '.join(ensure_unique_address(recipients))
         logging.debug('reply to: %s', to)
 
@@ -310,6 +297,20 @@ class ReplyCommand(Command):
                     new_value.append(address)
         return new_value
 
+    @staticmethod
+    def ensure_unique_address(recipients):
+        """
+        clean up a list of name,address pairs so that
+        no address appears multiple times.
+        """
+        res = dict()
+        for name, address in getaddresses(recipients):
+            res[address] = name
+        urecipients = ['"{}" <{}>'.format(n, a) if n != ''
+                       else a
+                       for a, n in res.iteritems()]
+        return sorted(urecipients)
+
 
 @registerCommand(MODE, 'forward', arguments=[
     (['--attach'], {'action': 'store_true', 'help': 'attach original mail'}),