Skip to content

Commit f6edc01

Browse files
authored
Merge pull request #7630 from readthedocs/humitos/notification-test-fix
Check only for override settings (part 2)
2 parents f412dba + da8679c commit f6edc01

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)