Skip to content

Commit

Permalink
fixes for Django 2.0 + Travis/Tox testing
Browse files Browse the repository at this point in the history
  • Loading branch information
PetrDlouhy committed Dec 4, 2017
1 parent 43d4286 commit e763a18
Show file tree
Hide file tree
Showing 8 changed files with 37 additions and 12 deletions.
16 changes: 13 additions & 3 deletions .travis.yml
Expand Up @@ -11,10 +11,23 @@ env:
- PACKAGES="Django>=1.9,<1.10 django-threadedcomments>=1.1"
- PACKAGES="Django>=1.10,<1.11"
- PACKAGES="Django>=1.10,<1.11 django-threadedcomments>=1.1"
- PACKAGES="Django>=1.11,<1.12"
- PACKAGES="Django>=1.11,<1.12 django-threadedcomments>=1.1"
- PACKAGES="Django>=2.0,<2.1"
- PACKAGES="Django>=2.0,<2.1 django-threadedcomments>=1.1"
- PACKAGES='https://github.com/django/django/archive/master.tar.gz'
matrix:
exclude:
- python: '2.7'
env: PACKAGES="Django>=2.0,<2.1"
- python: '2.7'
env: PACKAGES="Django>=2.0,<2.1 django-threadedcomments>=1.1"
- python: '2.7'
env: PACKAGES='https://github.com/django/django/archive/master.tar.gz'
allow_failures:
- env: PACKAGES='https://github.com/django/django/archive/master.tar.gz'
- python: '3.5'
env: PACKAGES="Django>=2.0,<2.1 django-threadedcomments>=1.1"
before_install:
- pip install codecov
install:
Expand All @@ -24,9 +37,6 @@ script:
- coverage run example/manage.py test
after_success:
- codecov
branches:
only:
- master
notifications:
irc:
channels:
Expand Down
6 changes: 5 additions & 1 deletion example/article/models.py
@@ -1,10 +1,14 @@
from django.core.urlresolvers import reverse
from django.db import models
from django.utils.six import python_2_unicode_compatible

from fluent_comments.moderation import moderate_model, comments_are_open, comments_are_moderated
from fluent_comments.models import get_comments_for_model, CommentsRelation

try:
from django.urls import reverse
except ImportError: # Django<2.0
from django.core.urlresolvers import reverse


@python_2_unicode_compatible
class Article(models.Model):
Expand Down
6 changes: 5 additions & 1 deletion example/article/tests/test_comments.py
Expand Up @@ -5,10 +5,14 @@

from article.tests import factories
from django.contrib.auth.models import User
from django.core.urlresolvers import reverse
from django.test import TestCase
from django_comments.forms import CommentForm

try:
from django.urls import reverse
except ImportError: # Django<2.0
from django.core.urlresolvers import reverse


class CommentsTests(TestCase):

Expand Down
6 changes: 5 additions & 1 deletion example/article/tests/test_moderation.py
@@ -1,7 +1,6 @@
from datetime import datetime, timedelta

from akismet import Akismet
from django.core.urlresolvers import reverse
from django.test import RequestFactory
from django.test import TestCase
from django.utils.timezone import now
Expand All @@ -13,6 +12,11 @@
from article.tests import factories
from .utils import override_appsettings

try:
from django.urls import reverse
except ImportError: # Django<2.0
from django.core.urlresolvers import reverse


class MockedResponse(object):
def __init__(self, result):
Expand Down
3 changes: 2 additions & 1 deletion example/settings.py
Expand Up @@ -61,13 +61,14 @@
]


MIDDLEWARE_CLASSES = (
MIDDLEWARE = (
'django.middleware.common.CommonMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
)
MIDDLEWARE_CLASSES = MIDDLEWARE

TEST_RUNNER = 'django.test.runner.DiscoverRunner'

Expand Down
2 changes: 1 addition & 1 deletion example/urls.py
Expand Up @@ -7,7 +7,7 @@
admin.autodiscover()

urlpatterns = [
url(r'^admin/', include(admin.site.urls)),
url(r'^admin/', admin.site.urls),

url(r'^comments/', include(fluent_comments.urls)),
url(r'^articles/', include(article.urls)),
Expand Down
4 changes: 2 additions & 2 deletions fluent_comments/views.py
Expand Up @@ -36,7 +36,7 @@ def post_comment_ajax(request, using=None):

# Fill out some initial data fields from an authenticated user, if present
data = request.POST.copy()
if request.user.is_authenticated():
if request.user.is_authenticated is True:
if not data.get('name', ''):
data["name"] = request.user.get_full_name() or request.user.username
if not data.get('email', ''):
Expand Down Expand Up @@ -81,7 +81,7 @@ def post_comment_ajax(request, using=None):
# Otherwise create the comment
comment = form.get_comment_object()
comment.ip_address = request.META.get("REMOTE_ADDR", None)
if request.user.is_authenticated():
if request.user.is_authenticated is True:
comment.user = request.user

# Signal that the comment is about to be saved
Expand Down
6 changes: 4 additions & 2 deletions tox.ini
@@ -1,8 +1,8 @@
[tox]
envlist=
py27-django{18,19,110,111},
py27-django{18,19,110,111,20},
py33-django{18},
py34-django{18,19,110,111},
py34-django{18,19,110,111,20},
py34-django{110}-tc,
# py33-django-dev,
docs,
Expand All @@ -13,7 +13,9 @@ deps =
django19: Django >= 1.9,<1.10
django110: Django >= 1.10,<1.11
django111: Django >= 1.11,<1.12
django20: Django >= 2.0,<2.1
tc: django-threadedcomments >= 1.1
django-dev: https://github.com/django/django/tarball/master
mock
commands=
./runtests.sh

0 comments on commit e763a18

Please sign in to comment.