diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/mailman/commands/cli_lists.py | 4 | ||||
| -rw-r--r-- | src/mailman/testing/layers.py | 5 |
2 files changed, 5 insertions, 4 deletions
diff --git a/src/mailman/commands/cli_lists.py b/src/mailman/commands/cli_lists.py index 0b5973591..8acf0c272 100644 --- a/src/mailman/commands/cli_lists.py +++ b/src/mailman/commands/cli_lists.py @@ -196,8 +196,8 @@ class Create: # owners. I suppose we could subclass that exception though. if args.owners: validator = getUtility(IEmailValidator) - invalid_owners = (owner for owner in args.owners - if not validator.is_valid(owner)) + invalid_owners = [owner for owner in args.owners + if not validator.is_valid(owner)] if invalid_owners: invalid = COMMASPACE.join(sorted(invalid_owners)) self.parser.error(_('Illegal owner addresses: $invalid')) diff --git a/src/mailman/testing/layers.py b/src/mailman/testing/layers.py index 6ae9d195e..f1809451d 100644 --- a/src/mailman/testing/layers.py +++ b/src/mailman/testing/layers.py @@ -37,7 +37,7 @@ import datetime import tempfile from base64 import b64encode -from lazr.config import as_timedelta +from lazr.config import as_boolean, as_timedelta from pkg_resources import resource_string from textwrap import dedent from urllib2 import Request, URLError, urlopen @@ -303,4 +303,5 @@ def is_testing(): :return: True when in testing mode. :rtype: bool """ - return MockAndMonkeyLayer.testing_mode or config.devmode.testing + return (MockAndMonkeyLayer.testing_mode or + as_boolean(config.devmode.testing)) |