From 3207311e507a8c9ef9f83a5e634ce156058bd209 Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Wed, 1 Jun 2016 16:29:25 -0400 Subject: [PATCH] Use copies of tracker settings dicts in tests so that changes don't leak between tests --- common/djangoapps/track/tests/test_tracker.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/common/djangoapps/track/tests/test_tracker.py b/common/djangoapps/track/tests/test_tracker.py index 230a6b939e..4914503d16 100644 --- a/common/djangoapps/track/tests/test_tracker.py +++ b/common/djangoapps/track/tests/test_tracker.py @@ -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."""