Skip to content

Commit ba5f99c

Browse files
committed
Check only override settings
Since we are using this tests in -corporate as well, we need to check only for settings wee are override in the test to avoid passing in .org and failing in .com
1 parent 010a7b5 commit ba5f99c

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

readthedocs/rtd_tests/tests/test_notifications.py

+8-7
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
'readthedocs.notifications.backends.SiteBackend',
3333
],
3434
PRODUCTION_DOMAIN='readthedocs.org',
35+
SUPPORT_EMAIL='[email protected]',
3536
)
3637
@mock.patch('readthedocs.notifications.notification.render_to_string')
3738
@mock.patch.object(Notification, 'send')
@@ -69,16 +70,16 @@ class TestNotification(Notification):
6970
'request': req,
7071

7172
# readthedocs_processor context
72-
'DASHBOARD_ANALYTICS_CODE': None,
73-
'DO_NOT_TRACK_ENABLED': False,
74-
'GLOBAL_ANALYTICS_CODE': None,
73+
'DASHBOARD_ANALYTICS_CODE': mock.ANY,
74+
'DO_NOT_TRACK_ENABLED': mock.ANY,
75+
'GLOBAL_ANALYTICS_CODE': mock.ANY,
7576
'PRODUCTION_DOMAIN': 'readthedocs.org',
76-
'PUBLIC_DOMAIN': None,
77+
'PUBLIC_DOMAIN': mock.ANY,
7778
'SITE_ROOT': mock.ANY,
78-
'SUPPORT_EMAIL': None,
79+
'SUPPORT_EMAIL': '[email protected]',
7980
'TEMPLATE_ROOT': mock.ANY,
80-
'USE_PROMOS': False,
81-
'USE_SUBDOMAIN': False,
81+
'USE_PROMOS': mock.ANY,
82+
'USE_SUBDOMAIN': mock.ANY,
8283
},
8384
)
8485

0 commit comments

Comments
 (0)