Fix delete bulk
Add test
This commit is contained in:
1 parent
053c02d8bc
commit
39f4ee654a
3 files changed
+12
-4
No files matched your search
@@ -21,6 +21,7 @@ class AliasDeleteSerializer(serializers.ModelSerializer):
|
|||||||
class Meta:
|
class Meta:
|
||||||
model = Alias
|
model = Alias
|
||||||
validators = [] # No Unique togheter check on deletion
|
validators = [] # No Unique togheter check on deletion
|
||||||
|
fields = '__all__'
|
||||||
|
|
||||||
|
|
||||||
class DomainSerializer(serializers.ModelSerializer):
|
class DomainSerializer(serializers.ModelSerializer):
|
||||||
|
|||||||
+10
-3
@@ -1,5 +1,6 @@
|
|||||||
|
import json
|
||||||
|
|
||||||
from django.contrib.auth.models import User
|
from django.contrib.auth.models import User
|
||||||
from django.test import override_settings
|
|
||||||
from django.urls import reverse
|
from django.urls import reverse
|
||||||
from rest_framework.test import APITestCase
|
from rest_framework.test import APITestCase
|
||||||
|
|
||||||
@@ -11,9 +12,9 @@ class RegexTestCase(APITestCase):
|
|||||||
|
|
||||||
def setUp(self):
|
def setUp(self):
|
||||||
user = User.objects.create(username='yolo')
|
user = User.objects.create(username='yolo')
|
||||||
d = Domain.objects.create(name='example.com')
|
self.domain = Domain.objects.create(name='example.com')
|
||||||
for a in self.aliases:
|
for a in self.aliases:
|
||||||
Alias.objects.create(source=a, destination='ninja@example.com', domain=d)
|
Alias.objects.create(source=a, destination='ninja@example.com', domain=self.domain)
|
||||||
self.client.force_login(user)
|
self.client.force_login(user)
|
||||||
|
|
||||||
def test_get_all(self):
|
def test_get_all(self):
|
||||||
@@ -34,3 +35,9 @@ class RegexTestCase(APITestCase):
|
|||||||
params = {'source__regex': '^ops-|^ops@'}
|
params = {'source__regex': '^ops-|^ops@'}
|
||||||
res = self.client.get(reverse('alias-list'), data=params)
|
res = self.client.get(reverse('alias-list'), data=params)
|
||||||
self.assertEqual(len(res.data), 1)
|
self.assertEqual(len(res.data), 1)
|
||||||
|
|
||||||
|
def test_delete_bulk_single(self):
|
||||||
|
data = {'destination': 'ninja@example.com', 'source': 'ding@example.com', 'domain': self.domain.pk}
|
||||||
|
Alias.objects.create(source=data['source'], destination=data['destination'], domain=self.domain)
|
||||||
|
res = self.client.delete(reverse('alias-delete-bulk'), data=[data], format='json')
|
||||||
|
self.assertEqual(res.status_code, 204, res.data)
|
||||||
+1
-1
@@ -2,7 +2,7 @@ Django==1.11.6
|
|||||||
django-extensions==1.9.6
|
django-extensions==1.9.6
|
||||||
django-filter==1.1.0
|
django-filter==1.1.0
|
||||||
djangorestframework==3.7.1
|
djangorestframework==3.7.1
|
||||||
Fabric==1.14.0
|
Fabric3==1.13.1.post1
|
||||||
gunicorn==19.7.1
|
gunicorn==19.7.1
|
||||||
mysqlclient==1.3.12
|
mysqlclient==1.3.12
|
||||||
passlib==1.7.1
|
passlib==1.7.1
|
||||||
Reference in new issue
Block a user