Merge pull request noironetworks#289 from noironetworks/none_desc

Add a guard to handle the description is None case

Author: tbachman

aim/tests/unit/test_utils.py

@@ -39,6 +39,8 @@ def test_sanitize_display_name(self):
             utils.sanitize_display_name('some' * 15))
 
     def test_sanitize_description(self):
+        self.assertEqual(
+            '', utils.sanitize_description(None))
         self.assertEqual(
             '_eF5\\!#$%()*,-./:;@ _{|}~?&+_',
             utils.sanitize_description('\'eF5\\!#$%()*,-./:;@ _{|}~?&+\"'))

aim/utils.py

@@ -21,5 +21,7 @@ def sanitize_display_name(display_name):
 
 
 def sanitize_description(description):
+    if description is None:
+        return ''
     return re.sub(r'[^a-zA-Z0-9\\!#$%()*,-./:;@ _{|}~?&+]',
                   '_', description[:124])