Skip to content

Commit 4b241a9

Browse files
committed
Drop Django 1.3 compatibility code
1 parent 5172a70 commit 4b241a9

File tree

7 files changed

+6
-39
lines changed

7 files changed

+6
-39
lines changed

pytest_django/live_server_helper.py

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,6 @@
44

55
from .lazy_django import get_django_version
66

7-
8-
def supported():
9-
import django.test.testcases
10-
11-
return hasattr(django.test.testcases, 'LiveServerThread')
12-
13-
147
class LiveServer(object):
158
"""The liveserver fixture
169
@@ -20,11 +13,8 @@ class LiveServer(object):
2013
"""
2114

2215
def __init__(self, addr):
23-
try:
24-
from django.test.testcases import LiveServerThread
25-
except ImportError:
26-
pytest.skip('live_server tests not supported in Django < 1.4')
2716
from django.db import connections
17+
from django.test.testcases import LiveServerThread
2818

2919
connections_override = {}
3020
for conn in connections.all():

pytest_django_test/settings_base.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@
1212
STATIC_URL = '/static/'
1313
SECRET_KEY = 'foobar'
1414

15-
SITE_ID = 1234 # Needed for 1.3 compatibility
16-
1715
# Used to construct unique test database names to allow detox to run multiple
1816
# versions at the same time
1917
uid = os.getenv('UID', '')

pytest_django_test/urls.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,4 @@
1-
try:
2-
from django.conf.urls import patterns # Django >1.4
3-
except ImportError:
4-
from django.conf.urls.defaults import patterns # Django 1.3
1+
from django.conf.urls import patterns
52

63
urlpatterns = patterns(
74
'',

pytest_django_test/urls_overridden.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,4 @@
1-
try:
2-
from django.conf.urls import patterns, url # Django >1.4
3-
except ImportError:
4-
from django.conf.urls.defaults import patterns, url # Django 1.3
1+
from django.conf.urls import patterns, url
52

63
from django.http import HttpResponse
74

tests/conftest.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,6 @@ def django_testdir(request, testdir, monkeypatch):
6363
'tpkg.app',
6464
]
6565
SECRET_KEY = 'foobar'
66-
SITE_ID = 1234 # Needed for 1.3 compatibility
6766
6867
MIDDLEWARE_CLASSES = (
6968
'django.contrib.sessions.middleware.SessionMiddleware',

tests/test_environment.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,7 @@ def test_mail_again():
4040
""")
4141
def test_invalid_template_variable(django_testdir):
4242
django_testdir.create_app_file("""
43-
try:
44-
from django.conf.urls import patterns # Django >1.4
45-
except ImportError:
46-
from django.conf.urls.defaults import patterns # Django 1.3
43+
from django.conf.urls import patterns
4744
4845
urlpatterns = patterns(
4946
'',
@@ -90,10 +87,7 @@ def test_ignore(client):
9087
""")
9188
def test_invalid_template_variable_opt_in(django_testdir):
9289
django_testdir.create_app_file("""
93-
try:
94-
from django.conf.urls import patterns # Django >1.4
95-
except ImportError:
96-
from django.conf.urls.defaults import patterns # Django 1.3
90+
from django.conf.urls import patterns
9791
9892
urlpatterns = patterns(
9993
'',

tests/test_fixtures.py

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,6 @@ def test_deleted_again(self, settings):
8585

8686

8787
class TestLiveServer:
88-
pytestmark = [
89-
pytest.mark.skipif(get_django_version() < (1, 4),
90-
reason="Django > 1.3 required"),
91-
]
92-
9388
def test_url(self, live_server):
9489
assert live_server.url == force_text(live_server)
9590

@@ -227,10 +222,7 @@ class MyCustomUser(AbstractUser):
227222
USERNAME_FIELD = 'identifier'
228223
""", 'models.py')
229224
django_testdir.create_app_file("""
230-
try:
231-
from django.conf.urls import patterns # Django >1.4
232-
except ImportError:
233-
from django.conf.urls.defaults import patterns # Django 1.3
225+
from django.conf.urls import patterns
234226
235227
urlpatterns = patterns(
236228
'',

0 commit comments

Comments
 (0)