Add more tests

Author: cuducos

.gitignore

@@ -5,4 +5,5 @@
 .env
 .ropeproject/
 __pycache__/
+htmlcov/
 static/

InternetSemLimites/api/tests/test_edit_view.py

@@ -34,6 +34,12 @@ def setUp(self):
         self.resp = self.client.post(resolve_url('api:provider', self.provider.pk), self.data)
         self.edited_provider = Provider.objects.last()
 
+    def test_not_allowed_methods(self):
+        url = resolve_url('api:provider', self.provider.pk)
+        for r in (self.client.delete(url), self.client.patch(url, self.data)):
+            with self.subTest():
+                self.assertEqual(405, r.status_code)
+
     def test_post(self):
         self.assertRedirects(self.resp, resolve_url('api:provider', self.edited_provider.pk))
 

InternetSemLimites/core/mixins.py

@@ -27,8 +27,10 @@ def send_mail(self):
     def get_email_template_name(self):
         if self.email_template_name:
             return self.email_template_name
-        return '{}/{}_email.txt'.format(self.object._meta.app_label,
-                                        self.object._meta.model_name)
+
+        app = self.object._meta.app_label
+        model = self.object._meta.model_name
+        return f'{app}/{model}_email.txt'
 
     def get_email_context_data(self, **kwargs):
         context = dict(kwargs)

InternetSemLimites/core/tests/test_new_mail.py

@@ -2,6 +2,8 @@
 from django.core import mail
 from django.shortcuts import resolve_url
 from django.test import TestCase
+
+from InternetSemLimites.core.mixins import EmailAdminCreateMixin
 from InternetSemLimites.core.models import Provider, State
 
 
@@ -37,3 +39,25 @@ def test_email_body(self):
         for content in contents:
             with self.subTest():
                 self.assertIn(content, self.email.body)
+
+
+class TestMixindminCreateMixin(TestCase):
+
+    def setUp(self):
+        self.mixin = EmailAdminCreateMixin()
+        self.mixin.object = Provider
+
+    def test_get_unset_email_template_name(self):
+        self.assertEqual('core/provider_email.txt',
+                         self.mixin.get_email_template_name())
+
+    def test_get_set_email_template_name(self):
+        self.mixin.email_template_name = '42'
+        self.assertEqual('42', self.mixin.get_email_template_name())
+
+    def test_get_unset_email_context_name(self):
+        self.assertEqual('provider', self.mixin.get_email_context_name())
+
+    def test_get_set_email_context_name(self):
+        self.mixin.email_context_name = '42'
+        self.assertEqual('42', self.mixin.get_email_context_name())

InternetSemLimites/core/tests/test_provider_model.py

@@ -15,6 +15,9 @@ def setUp(self):
         self.provider = Provider.objects.create(**props)
         self.provider.coverage = [sc, go]
 
+    def test_str(self):
+        self.assertEqual('Xpto', str(self.provider))
+
     def test_create(self):
         self.assertTrue(Provider.objects.exists())