Skip to content

Commit da8679c

Browse files
committed
Check only for override settings (part 2)
The same as #7628 for another test case that I missed :(
1 parent f412dba commit da8679c

File tree

1 file changed

+11
-8
lines changed

1 file changed

+11
-8
lines changed

readthedocs/rtd_tests/tests/test_notifications.py

+11-8
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,10 @@ class TestNotification(SiteNotification):
203203
self.assertNotContains(response, 'Test success message')
204204

205205

206-
@override_settings(PRODUCTION_DOMAIN='readthedocs.org')
206+
@override_settings(
207+
PRODUCTION_DOMAIN='readthedocs.org',
208+
SUPPORT_EMAIL='[email protected]',
209+
)
207210
class SiteNotificationTests(TestCase):
208211

209212
class TestSiteNotification(SiteNotification):
@@ -235,16 +238,16 @@ def test_context_data(self):
235238
'other': {'name': 'other name'},
236239

237240
# readthedocs_processor context
238-
'DASHBOARD_ANALYTICS_CODE': None,
239-
'DO_NOT_TRACK_ENABLED': False,
240-
'GLOBAL_ANALYTICS_CODE': None,
241+
'DASHBOARD_ANALYTICS_CODE': mock.ANY,
242+
'DO_NOT_TRACK_ENABLED': mock.ANY,
243+
'GLOBAL_ANALYTICS_CODE': mock.ANY,
241244
'PRODUCTION_DOMAIN': 'readthedocs.org',
242-
'PUBLIC_DOMAIN': None,
245+
'PUBLIC_DOMAIN': mock.ANY,
243246
'SITE_ROOT': mock.ANY,
244-
'SUPPORT_EMAIL': None,
247+
'SUPPORT_EMAIL': '[email protected]',
245248
'TEMPLATE_ROOT': mock.ANY,
246-
'USE_PROMOS': False,
247-
'USE_SUBDOMAIN': False,
249+
'USE_PROMOS': mock.ANY,
250+
'USE_SUBDOMAIN': mock.ANY,
248251
}
249252
self.assertEqual(self.n.get_context_data(), context)
250253

0 commit comments

Comments
 (0)