summaryrefslogtreecommitdiff
path: root/tests/test_runner
diff options
context:
space:
mode:
authorTim Graham <timograham@gmail.com>2016-06-28 11:21:26 -0400
committerGitHub <noreply@github.com>2016-06-28 11:21:26 -0400
commitc9ae09addffb839403312131d4251e9d8b454508 (patch)
tree74913fbe2e90d88e094f8594947ebf313ec5f12f /tests/test_runner
parentc1b6f554e405fe733e8d80f7e3d77c277810e707 (diff)
Replaced use of TestCase.fail() with assertRaises().
Also removed try/except/fail antipattern that hides exceptions.
Diffstat (limited to 'tests/test_runner')
-rw-r--r--tests/test_runner/tests.py21
1 files changed, 5 insertions, 16 deletions
diff --git a/tests/test_runner/tests.py b/tests/test_runner/tests.py
index fd0b895789..7fa54f0d87 100644
--- a/tests/test_runner/tests.py
+++ b/tests/test_runner/tests.py
@@ -240,12 +240,8 @@ class DummyBackendTest(unittest.TestCase):
tested_connections = db.ConnectionHandler({})
with mock.patch('django.test.runner.connections', new=tested_connections):
runner_instance = DiscoverRunner(verbosity=0)
- try:
- old_config = runner_instance.setup_databases()
- runner_instance.teardown_databases(old_config)
- except Exception as e:
- self.fail("setup_databases/teardown_databases unexpectedly raised "
- "an error: %s" % e)
+ old_config = runner_instance.setup_databases()
+ runner_instance.teardown_databases(old_config)
class AliasedDefaultTestSetupTest(unittest.TestCase):
@@ -263,12 +259,8 @@ class AliasedDefaultTestSetupTest(unittest.TestCase):
})
with mock.patch('django.test.runner.connections', new=tested_connections):
runner_instance = DiscoverRunner(verbosity=0)
- try:
- old_config = runner_instance.setup_databases()
- runner_instance.teardown_databases(old_config)
- except Exception as e:
- self.fail("setup_databases/teardown_databases unexpectedly raised "
- "an error: %s" % e)
+ old_config = runner_instance.setup_databases()
+ runner_instance.teardown_databases(old_config)
class SetupDatabasesTests(unittest.TestCase):
@@ -366,7 +358,4 @@ class EmptyDefaultDatabaseTest(unittest.TestCase):
testcases.connections = db.ConnectionHandler({'default': {}})
connection = testcases.connections[db.utils.DEFAULT_DB_ALIAS]
self.assertEqual(connection.settings_dict['ENGINE'], 'django.db.backends.dummy')
- try:
- connections_support_transactions()
- except Exception as e:
- self.fail("connections_support_transactions() unexpectedly raised an error: %s" % e)
+ connections_support_transactions()