none attr cannot split

tests/test_email.py

@@ -218,31 +218,31 @@ def main():
         or u.get_ip()
         or None
     )
-    sender = conf.get("EMAIL_SENDER") or None
+    sender = conf.get("EMAIL_SENDER") or ""
     if not sender and Path("/etc/postfix/generic").exists():
         with open("/etc/postfix/generic") as sender_fo:
             sender = sender_fo.readline().split()[-1]
     domain = sender.split("@")[-1] or None
 
     # check that we are not an open relay
-    check_listen_warn, check_listen_err = check_listen()
-    warnings += check_listen_warn if check_listen_warn else warnings
-    errors += check_listen_err if check_listen_err else errors
+    check_warn, check_err = check_listen()
+    warnings += check_warn if check_warn else warnings
+    errors += check_err if check_err else errors
 
     # check that relayhost is correct
-    check_relay_warn, check_relay_err = check_relay(relay, domain)
-    warnings += check_relay_warn if check_relay_warn else warnings
-    errors += check_relay_err if check_relay_err else errors
+    check_warn, check_err = check_relay(relay, domain)
+    warnings += check_warn if check_warn else warnings
+    errors += check_err if check_err else errors
 
     # check that we can send emails
-    check_send_warn, check_send_err = check_send(sender)
-    warnings += check_send_warn if check_send_warn else warnings
-    errors += check_send_err if check_send_err else errors
+    check_warn, check_err = check_send(sender)
+    warnings += check_warn if check_warn else warnings
+    errors += check_err if check_err else errors
 
     # check that spf record is correct
-    check_spf_warn, check_spf_err = check_spf(ip_addr, sender, domain)
-    warnings += check_spf_warn if check_spf_warn else warnings
-    errors += check_spf_err if check_spf_err else errors
+    check_warn, check_err = check_spf(ip_addr, sender, domain)
+    warnings += check_warn if check_warn else warnings
+    errors += check_err if check_err else errors
 
     if errors:
         exit(1)