diff --git a/tests/fixtures/tests.py b/tests/fixtures/tests.py index fb807bb97c4..1e1e1a8cbdf 100644 --- a/tests/fixtures/tests.py +++ b/tests/fixtures/tests.py @@ -6,6 +6,7 @@ from django.contrib.sites.models import Site from django.core import management from django.db import connection, IntegrityError from django.test import TestCase, TransactionTestCase, skipUnlessDBFeature +from django.utils.encoding import force_text from django.utils import six from .models import Article, Book, Spy, Tag, Visa @@ -157,18 +158,6 @@ class FixtureLoadingTests(DumpDataAssertMixin, TestCase): '' ]) - # Loading a fixture that doesn't exist emits a warning - with warnings.catch_warnings(record=True) as w: - warnings.simplefilter("always") - management.call_command('loaddata', 'unknown.json', verbosity=0) - self.assertEqual(len(w), 1) - self.assertTrue(w[0].message, "No fixture named 'unknown' found.") - - # An attempt to load a nonexistent 'initial_data' fixture isn't an error - with warnings.catch_warnings(record=True) as w: - management.call_command('loaddata', 'initial_data.json', verbosity=0) - self.assertEqual(len(w), 0) - # object list is unaffected self.assertQuerysetEqual(Article.objects.all(), [ '', @@ -383,6 +372,35 @@ class FixtureLoadingTests(DumpDataAssertMixin, TestCase): News StoriesLatest news storiesPoker has no place on ESPN2006-06-16T12:00:00Time to reform copyright2006-06-16T13:00:00copyrightfixturesarticle3lawfixturesarticle3Django ReinhardtStephane GrappelliPrinceAchieving self-awareness of Python programs""", format='xml', natural_foreign_keys=True) +class NonExistentFixtureTests(TestCase): + """ + Custom class to limit fixture dirs. + """ + available_apps = ['django.contrib.auth', 'django.contrib.contenttypes'] + + def test_loaddata_not_existent_fixture_file(self): + stdout_output = six.StringIO() + with warnings.catch_warnings(record=True) as w: + warnings.simplefilter("always") + # With verbosity=2, we get both stdout output and a warning + management.call_command( + 'loaddata', + 'this_fixture_doesnt_exist', + verbosity=2, + stdout=stdout_output, + ) + self.assertIn("No fixture 'this_fixture_doesnt_exist' in", + force_text(stdout_output.getvalue())) + self.assertEqual(len(w), 1) + self.assertEqual(force_text(w[0].message), + "No fixture named 'this_fixture_doesnt_exist' found.") + + # An attempt to load a non-existent 'initial_data' fixture doesn't produce any warning + with warnings.catch_warnings(record=True) as w: + management.call_command('loaddata', 'initial_data.json', verbosity=0) + self.assertEqual(len(w), 0) + + class FixtureTransactionTests(DumpDataAssertMixin, TransactionTestCase): available_apps = [ diff --git a/tests/fixtures_regress/tests.py b/tests/fixtures_regress/tests.py index 400ca040361..d2c3eff4fa2 100644 --- a/tests/fixtures_regress/tests.py +++ b/tests/fixtures_regress/tests.py @@ -16,7 +16,6 @@ from django.db.models import signals from django.test import (TestCase, TransactionTestCase, skipIfDBFeature, skipUnlessDBFeature) from django.test import override_settings -from django.utils.encoding import force_text from django.utils._os import upath from django.utils import six from django.utils.six import PY3, StringIO @@ -459,18 +458,6 @@ class TestFixtures(TestCase): verbosity=0, ) - def test_loaddata_not_existant_fixture_file(self): - stdout_output = StringIO() - with warnings.catch_warnings(record=True): - management.call_command( - 'loaddata', - 'this_fixture_doesnt_exist', - verbosity=2, - stdout=stdout_output, - ) - self.assertTrue("No fixture 'this_fixture_doesnt_exist' in" in - force_text(stdout_output.getvalue())) - def test_ticket_20820(self): """ Regression for ticket #20820 -- loaddata on a model that inherits