Made TestNoInitialDataLoading pass with MySQL (Refs django#15926)

audreyfeldroy · Jun 6, 2012 · f0664dc · f0664dc
1 parent 7edf231
commit f0664dc
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 31 deletions.
diff --git a/django/core/management/commands/flush.py b/django/core/management/commands/flush.py
@@ -81,9 +81,8 @@ def handle_noargs(self, **options):
             # Reinstall the initial_data fixture.
             kwargs = options.copy()
             kwargs['database'] = db
-            if options.get('load_initial_data', True):
-            # Reinstall the initial_data fixture.
-                from django.core.management import call_command
+            if options.get('load_initial_data'):
+                # Reinstall the initial_data fixture.
                 call_command('loaddata', 'initial_data', **options)
 
         else:

diff --git a/django/core/management/commands/syncdb.py b/django/core/management/commands/syncdb.py
@@ -2,6 +2,7 @@
 import traceback
 
 from django.conf import settings
+from django.core.management import call_command
 from django.core.management.base import NoArgsCommand
 from django.core.management.color import no_style
 from django.core.management.sql import custom_sql_for_model, emit_post_sync_signal
@@ -27,7 +28,7 @@ def handle_noargs(self, **options):
         verbosity = int(options.get('verbosity'))
         interactive = options.get('interactive')
         show_traceback = options.get('traceback')
-        load_initial_data = options.get('load_initial_data', True)
+        load_initial_data = options.get('load_initial_data')
 
         self.style = no_style()
 
@@ -158,6 +159,5 @@ def model_installed(model):
 
         # Load initial_data fixtures (unless that has been disabled)
         if load_initial_data:
-            from django.core.management import call_command
             call_command('loaddata', 'initial_data', verbosity=verbosity,
                          database=db, skip_validation=True)
diff --git a/tests/modeltests/fixtures_model_package/tests.py b/tests/modeltests/fixtures_model_package/tests.py
@@ -27,47 +27,22 @@ def test_syncdb(self):
         management.call_command(
             'syncdb',
             verbosity=0,
-            commit=False
-        )
-        self.assertQuerysetEqual(
-            Book.objects.all(), [
-                u'Achieving self-awareness of Python programs'
-            ],
-            lambda a: a.name
-        )
-
-        Book.objects.all().delete()
-
-        management.call_command(
-            'syncdb',
-            verbosity=0,
-            commit=False,
             load_initial_data=False
         )
         self.assertQuerysetEqual(Book.objects.all(), [])
 
     def test_flush(self):
-        Book.objects.all().delete()
-
-        management.call_command(
-            'flush',
-            verbosity=0,
-            interactive=False,
-            commit=False
-        )
+        # Test presence of fixture (flush called by TransactionTestCase)
         self.assertQuerysetEqual(
             Book.objects.all(), [
                 u'Achieving self-awareness of Python programs'
             ],
             lambda a: a.name
         )
 
-        Book.objects.all().delete()
-
         management.call_command(
             'flush',
             verbosity=0,
-            commit=False,
             interactive=False,
             load_initial_data=False
         )