Updated tests

.github/workflows/run-unittest.yml

@@ -5,7 +5,9 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.10", 3.11, 3.12]
+        python-version: ["3.10", 3.11, 3.12, 3.13]
+        # TODO
+        # python-version: [3.11, 3.12]
     steps:
       - uses: actions/checkout@v2
       - name: Set up Python ${{ matrix.python-version }}
@@ -15,7 +17,8 @@ jobs:
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
-          pip install M2Crypto
+          pip install M2Crypto  # TODO remove when tests are updated
+          pip install cryptography
           pip install -e .
       - name: Run tests
         run: python3 test_.py
@@ -26,12 +29,12 @@ jobs:
         run: pip uninstall python-magic -y
       - name: Test libmagic missing
         id: should_fail
-        run: python3 tests.py TestMime.test_libmagic
+        run: python3 test_.py TestMime.test_libmagic
         continue-on-error: true
       - name: Check on failures
         if: steps.should_fail.outcome != 'failure'
         run: exit 1
       - name: Install file-magic
         run: pip install file-magic
       - name: Test file-magic
-        run: python3 test_.py TestMime.test_libmagic
+        run: python3 test_.py TestMime.test_libmagic

envelope/address.py

@@ -2,6 +2,7 @@
 import logging
 from os import environ
 import re
+import sys
 from .utils import assure_list
 
 environ['PY3VE_IGNORE_UPDATER'] = '1'
@@ -11,6 +12,23 @@
 logger = logging.getLogger(__name__)
 
 
+def _getaddresses(*args):
+    # NOTE Python finally changed the old way of parsing wrong addresses.
+    # We might start using strict=True (default) in the future.
+    if sys.version_info <= (3, 11):
+        return getaddresses(*args)
+    return getaddresses(*args, strict=False)
+
+
+def _parseaddr(*args):
+    # NOTE Python finally changed the old way of parsing wrong addresses.
+    # We might start using strict=True (default) in the future.
+    # README should reflect that.
+    if sys.version_info <= (3, 11):
+        return parseaddr(*args)
+    return parseaddr(*args, strict=False)
+
+
 class Address(str):
     """
         You can safely access the `self.name` property to access the real name and `self.address` to access the e-mail address.
@@ -37,12 +55,14 @@ class Address(str):
 
     def __new__(cls, displayed_email=None, name=None, address=None):
         if displayed_email:
-            v = parseaddr(cls.remedy(displayed_email))
+            v = _parseaddr(cls.remedy(displayed_email))
             name, address = v[0] or name, v[1] or address
+
         if name:
             displayed_email = f"{name} <{address}>"
         else:
             displayed_email = address
+
         instance = super().__new__(cls, displayed_email or "")
         instance._name, instance._address = name or "", address or ""
         return instance
@@ -143,7 +163,7 @@ def parse(cls, email_or_list, single=False, allow_false=False):
         if allow_false and email_or_list is False:
             return False
 
-        addrs = getaddresses(cls.remedy(x) for x in assure_list(email_or_list))
+        addrs = _getaddresses(cls.remedy(x) for x in assure_list(email_or_list))
         addresses = [Address(name=real_name, address=address)
                      for real_name, address in addrs if not (real_name == address == "")]
         if single:
@@ -153,6 +173,8 @@ def parse(cls, email_or_list, single=False, allow_false=False):
             return addresses[0]
         # if len(addresses) == 0:
         #     raise ValueError(f"E-mail address cannot be parsed: {email_or_list}")
+        # if len(addresses) == 0:
+        #     return email_or_list
         return addresses
 
     @classmethod
@@ -162,6 +184,12 @@ def remedy(s):
             parsed as two distinguish addresses with getaddresses. Rename the at-sign in the display name
             to "person--AT--example.com <person@example2.com>" so that the result of getaddresses is less wrong.
             """
+
+            """
+            What happens when the string have more addresses?
+            It also needs to get the address from string like "person@example.com, <person@example.com>" so we need to 
+            take care of the comma and semicolon as well.
+            """
             if s.group(1).strip() == s.group(2).strip():
                 # Display name is the same as the e-mail in angle brackets
                 # Ex: "person@example.com <person@example.com>"

envelope/constants.py

@@ -5,7 +5,8 @@
 except ImportError:
     gnupg = None
 
-smime_import_error = "Cannot import M2Crypto. Run: `sudo apt install swig && pip3 install M2Crypto`"
+# smime_import_error = "Cannot import M2Crypto. Run: `sudo apt install swig && pip3 install M2Crypto`"
+smime_import_error = "Cannot import cryptography. Run: `sudo apt install cryptography`"
 CRLF = '\r\n'
 AUTO = "auto"
 PLAIN = "plain" # XX allow text/plain too?