Add tests

Author: fmarco

.travis.yml

@@ -0,0 +1,25 @@
+sudo: false
+language: python
+python:
+- "2.7"
+- "3.4"
+- "3.5"
+- "3.6"
+
+cache:
+  directories:
+    - $HOME/.cache/pip
+    - $TRAVIS_BUILD_DIR/.tox
+
+install:
+  - pip install --upgrade pip wheel setuptools
+  - pip install tox-travis coveralls
+
+branches:
+  - develop
+  - master
+
+script: tox
+
+after_success:
+  - coveralls

README.md

@@ -1,7 +1,22 @@
 ## django-rest-invitations
 
+
+[![Build Status](https://travis-ci.org/fmarco/django-rest-invitations.svg?branch=master)](https://travis-ci.org/fmarco/django-rest-invitations)
+
+[![Coverage Status](https://coveralls.io/repos/fmarco/django-rest-invitations/badge.svg?branch=master&service=github)](https://coveralls.io/github/fmarco/django-rest-invitations?branch=master)
+
 Rest customizable extension for [django-invitations](https://github.com/bee-keeper/django-invitations)
 
+Supported Django versions:
+
+* 2.7
+* 3.4
+* 3.5
+* 3.6
+
+Supported Django versions:
+
+* Django 1.11
 
 ### Requirements
 
@@ -90,11 +105,6 @@ url(r'^', include('rest_invitations.urls'))
     - /invitations/accept-invite/`<key>`/ (get)
 
 
-### TODOS
-
-* Tests
-* ...
-
 
 ### Contributions
 

manage.py

@@ -0,0 +1,8 @@
+#!/usr/bin/env python
+import os
+os.environ['DJANGO_SETTINGS_MODULE'] = 'test_settings'
+
+from django.core import management
+
+if __name__ == "__main__":
+    management.execute_from_command_line()

requirements_test.txt

@@ -0,0 +1 @@
+coverage>=3.7.1

rest_invitations/views.py

@@ -7,7 +7,6 @@
                                        permission_classes)
 from rest_framework.permissions import AllowAny, IsAuthenticated
 from rest_framework.response import Response
-from rest_framework.reverse import reverse
 
 from .app_settings import (CREATE_AND_SEND_URL, SEND_MULTIPLE_URL, SEND_URL,
                            InvitationBulkWriteSerializer, InvitationModel,
@@ -24,7 +23,7 @@ class InvitationViewSet(
     permission_classes = [IsAuthenticated]
 
     def get_serializer_class(self):
-        if self.action in ['list', 'retrive']:
+        if self.action in ['list', 'retrieve']:
             return InvitationReadSerializer
         elif self.action == 'send_multiple':
             return InvitationBulkWriteSerializer

setup.py

@@ -12,9 +12,8 @@
     keywords=['django', 'invitation', 'django-allauth', 'invite', 'rest', 'django-rest-framework', 'drf', 'invitations'],
     zip_safe=False,
     install_requires=[
-        'Django>=1.8.0',
-        'djangorestframework>=3.1.0',
-        'django-invitations==1.9.1'
+        'djangorestframework>=3.7.7',
+        'django-invitations==1.9.2'
     ],
     include_package_data=True,
     classifiers=[

test_settings.py

@@ -0,0 +1,67 @@
+# -*- coding: utf-8 -*-
+import django
+
+SECRET_KEY = 'sosecret'
+SITE_ID = 1
+
+DATABASES = {
+    'default': {
+        'ENGINE': 'django.db.backends.sqlite3',
+        'NAME': ':memory:',
+    }
+}
+
+ROOT_URLCONF = 'test_urls'
+
+if django.VERSION >= (1, 8):
+    TEMPLATES = [
+        {
+            'BACKEND': 'django.template.backends.django.DjangoTemplates',
+            'DIRS': [],
+            'APP_DIRS': True,
+            'OPTIONS': {
+                'context_processors': [
+                    'django.template.context_processors.debug',
+                    'django.template.context_processors.request',
+                    'django.contrib.auth.context_processors.auth',
+                    'django.contrib.messages.context_processors.messages'
+                ],
+            },
+        },
+    ]
+else:
+    TEMPLATE_CONTEXT_PROCESSORS = (
+        "django.contrib.auth.context_processors.auth",
+        "django.core.context_processors.debug",
+        "django.core.context_processors.i18n",
+        "django.core.context_processors.media",
+        "django.core.context_processors.static",
+        "django.core.context_processors.request",
+        "django.contrib.messages.context_processors.messages",
+    )
+
+MIDDLEWARE_CLASSES = (
+    'django.contrib.sessions.middleware.SessionMiddleware',
+    'django.middleware.common.CommonMiddleware',
+    'django.middleware.csrf.CsrfViewMiddleware',
+    'django.contrib.auth.middleware.AuthenticationMiddleware',
+    'django.contrib.messages.middleware.MessageMiddleware',
+    'django.middleware.clickjacking.XFrameOptionsMiddleware',
+)
+
+EMAIL_BACKEND = 'django.core.mail.backends.locmem.EmailBackend'
+
+INSTALLED_APPS = (
+    'django.contrib.auth',
+    'django.contrib.admin',
+    'django.contrib.contenttypes',
+    'django.contrib.sessions',
+    'django.contrib.sites',
+    'django.contrib.messages',
+    'invitations',
+    'rest_framework',
+    'rest_invitations'
+)
+AUTHENTICATION_BACKENDS = (
+    'django.contrib.auth.backends.ModelBackend',
+)

test_urls.py

@@ -0,0 +1,9 @@
+# -*- coding: utf-8 -*-
+from django.conf.urls import include, url
+from django.contrib import admin
+
+
+urlpatterns = [
+    url(r'^admin/', admin.site.urls),
+    url(r'^', include('rest_invitations.urls'))
+]

tests/__init__.py

@@ -0,0 +1,118 @@
+from django.contrib.auth import get_user_model
+from django.core import mail
+from django.urls import reverse
+from django.utils import timezone
+from invitations.utils import get_invitation_model
+from rest_framework import status
+from rest_framework.test import APITestCase
+
+User = get_user_model()
+Invitation = get_invitation_model()
+
+
+class TestInvitationAPI(APITestCase):
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestInvitationAPI, cls).setUpClass()
+        cls.user = User.objects.create_user(
+            email='[email protected]', username='test_user', password='test'
+        )
+
+    def test_invitation_send(self):
+        self.assertEqual(len(mail.outbox), 0)
+        invitation = Invitation.create(email='[email protected]')
+        url = reverse('invitation-send', kwargs={'pk': invitation.pk})
+        response = self.client.post(url, format='json')
+        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
+        self.assertEqual(len(mail.outbox), 0)
+        self.client.login(username=self.user.username, password='test')
+        url = reverse('invitation-send', kwargs={'pk': invitation.pk})
+        response = self.client.post(url, format='json')
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(len(mail.outbox), 1)
+
+    def test_invitation_create_and_send(self):
+        self.assertEqual(len(mail.outbox), 0)
+        self.assertEqual(Invitation.objects.count(), 0)
+        url = reverse('invitation-create-and-send')
+        data = {}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
+        self.assertEqual(Invitation.objects.count(), 0)
+        self.client.login(username=self.user.username, password='test')
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(Invitation.objects.count(), 0)
+        self.assertEqual(len(mail.outbox), 0)
+        data = {'email': '[email protected]'}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(len(mail.outbox), 1)
+        self.assertEqual(Invitation.objects.count(), 1)
+
+    def test_invitation_send_multiple(self):
+        self.assertEqual(len(mail.outbox), 0)
+        self.assertEqual(Invitation.objects.count(), 0)
+        url = reverse('invitation-send-multiple')
+        data = {}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
+        self.assertEqual(Invitation.objects.count(), 0)
+        self.client.login(username=self.user.username, password='test')
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(Invitation.objects.count(), 0)
+        self.assertEqual(len(mail.outbox), 0)
+        data = {'email': []}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(len(mail.outbox), 0)
+        self.assertEqual(Invitation.objects.count(), 0)
+        data = {'email': ['[email protected]', '[email protected]']}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(len(mail.outbox), 2)
+        self.assertEqual(Invitation.objects.count(), 2)
+        data = {'email': ['[email protected]', '[email protected]']}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(len(mail.outbox), 2)
+        self.assertEqual(Invitation.objects.count(), 2)
+        data = {'email': ['[email protected]']}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(len(mail.outbox), 2)
+        self.assertEqual(Invitation.objects.count(), 2)
+        data = {'email': ['[email protected]']}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(len(mail.outbox), 2)
+        self.assertEqual(Invitation.objects.count(), 2)
+
+    def test_invitation_accept(self):
+        invitation = Invitation.create(
+            email='[email protected]',
+            inviter=self.user
+        )
+        invitation_pk = invitation.pk
+        invitation.sent = timezone.now()
+        invitation.save()
+        url = reverse('invitations:accept-invite', kwargs={'key': 'something'})
+        data = {}
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_410_GONE)
+        self.assertFalse(invitation.accepted)
+        url = reverse(
+            'invitations:accept-invite',
+            kwargs={'key': invitation.key}
+        )
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        invitation = Invitation.objects.get(pk=invitation_pk)
+        self.assertTrue(invitation.accepted)
+        response = self.client.post(url, data, format='json')
+        self.assertEqual(response.status_code, status.HTTP_410_GONE)
+
+    def tearDown(self):
+        self.client.logout()

tests/tests.py

@@ -0,0 +1,32 @@
+[tox]
+envlist =
+  py27-django111
+  py34-django111
+  py35-django111
+  py36-django111
+  flake8
+  isort
+
+[testenv]
+setenv =
+  PYTHONWARNINGS = all
+deps =
+  -rrequirements_test.txt
+  django111: Django>=1.11,<2.0
+commands =
+  python -V
+  coverage run manage.py test --settings=test_settings
+  coverage report
+
+[testenv:flake8]
+skip_install = True
+deps=flake8
+commands=flake8 rest_invitations tests --exclude=migrations
+
+[testenv:isort]
+deps = isort
+commands = isort -c -rc -df rest_invitations tests
+skip_install = true
+
+[coverage:run]
+include = rest_invitations*