Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Use copies of tracker settings dicts in tests so that changes don't leak between tests

Browse Source
This commit is contained in:
Calen Pennington
2016-06-01 16:29:25 -04:00
parent 95054e1a03
commit 3207311e50
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
common/djangoapps/track/tests/test_tracker.py
View File

@@ -61,7 +61,7 @@ class TestTrackerInstantiation(TestCase):
class TestTrackerDjangoInstantiation(TestCase):
"""Test if backends are initialized properly from Django settings."""
@override_settings(TRACKING_BACKENDS=SIMPLE_SETTINGS)
@override_settings(TRACKING_BACKENDS=SIMPLE_SETTINGS.copy())
def test_django_simple_settings(self):
"""Test configuration of a simple backend"""
@@ -73,7 +73,7 @@ class TestTrackerDjangoInstantiation(TestCase):
self.assertEqual(backends.values()[0].count, 1)
@override_settings(TRACKING_BACKENDS=MULTI_SETTINGS)
@override_settings(TRACKING_BACKENDS=MULTI_SETTINGS.copy())
def test_django_multi_settings(self):
"""Test if multiple backends can be configured properly."""
@@ -88,7 +88,7 @@ class TestTrackerDjangoInstantiation(TestCase):
self.assertEqual(backends[0].count, event_count)
self.assertEqual(backends[1].count, event_count)
@override_settings(TRACKING_BACKENDS=MULTI_SETTINGS)
@override_settings(TRACKING_BACKENDS=MULTI_SETTINGS.copy())
def test_django_remove_settings(self):
"""Test if a backend can be remove by setting it to None."""