Closed Bug 797005 Opened 12 years ago Closed 12 years ago

Bedrock tests fail due to missing MANAGERS setting

Categories

(www.mozilla.org :: Bedrock, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: osmose, Assigned: osmose)

Details

The following two tests fail on a default local setup of bedrock:

test_format_identifier_mismatch (bedrock.lib.l10n_utils.tests.test_dotlang.TestDotlang)
test_parse_utf8_error (bedrock.lib.l10n_utils.tests.test_dotlang.TestDotlang)

They fail because they check if an email was sent by code that calls mail.mail_managers, which does nothing if settings.MANAGERS is empty.

We should probably include manager@example.com or something in MANAGERS in settings/local.py-dist to allow the tests to pass. We should also probably mock the setting out during the test to better isolate it.
Assignee: nobody → mkelly
Well hey, this was fixed at some point. Yay!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
Whiteboard: u=developer c=tests p=1
You need to log in before you can comment on or make changes to this bug.