Add Post related views tests

Author: MaciejWiatr

blog/__pycache__/admin.cpython-38.pyc

@@ -11,13 +11,22 @@
 
 @receiver(post_delete, sender=Post)
 def clear_files(sender, instance, **kwargs):
-    media_path = MEDIA_ROOT + f'/{instance.__class__.__name__}/{instance.slug}/'
+    media_path = MEDIA_ROOT + f'/{instance.__class__.__name__}/{instance.title}/'
+    print(f'Trying to delete {media_path}')
     if os.path.exists(media_path):
         root = Path(BASE_DIR)
         directory = Path(media_path)
         if root in directory.parents:  # Extra safety option to guarantee safety to files outside django dir
             print(f'Deleting media folder: {media_path}')
-            shutil.rmtree(media_path)
+            try:
+                shutil.rmtree(media_path)
+                print('Successfully deleted media folder')
+            except Exception as e:
+                print(f'An error occured during folder deletion {e}')
+        else:
+            print('Folder is in incorrect path')
+    else:
+        print('Folder dont exist')
 
 
 @receiver(post_save, sender=Post)

blog/__pycache__/forms.cpython-38.pyc

@@ -1,10 +1,22 @@
-from django.test import SimpleTestCase
+from django.test import TestCase
 from django.urls import reverse, resolve
 from blog.views import *
 
 
-class TestUrls(SimpleTestCase):
+class TestUrls(TestCase):
 
     def test_index_url_is_resolved(self):
         url = reverse('blog:index')
         self.assertEquals(resolve(url).func, index)
+
+    def test_post_create_url_resolves(self):
+        url = reverse('blog:post_create')
+        self.assertEquals(resolve(url).func.view_class, PostCreate)
+
+    def test_post_update_url_resolves(self):
+        url = reverse('blog:post_update', args=['test_case'])
+        self.assertEquals(resolve(url).func.view_class, PostUpdate)
+
+    def test_post_delete_url_resolves(self):
+        url = reverse('blog:post_delete', args=['test_case'])
+        self.assertEquals(resolve(url).func, post_delete)

blog/__pycache__/managers.cpython-38.pyc

@@ -0,0 +1,105 @@
+from django.test import TestCase, Client
+from django.urls import reverse
+from blog.urls import *
+from blog.views import *
+from blog.models import *
+from django.contrib.auth.models import User
+
+
+class ViewsMixin(object):
+
+    def get_response(self, url, method):
+        # client = Client()
+        if method == 'GET':
+            resp = self.client.get(reverse(url[0], args=url[1]))
+        elif method == 'POST':
+            resp = self.client.post(reverse(url[0], args=url[1]))
+        else:
+            resp = ''
+        return resp
+
+    def is_callable(self, url, user=None, post=None, fail=None, accept_redir=None):
+        if user:
+            self.client.force_login(user, backend=None)
+        if post:
+            method = 'POST'
+        else:
+            method = 'GET'
+        if fail:
+            resp = self.get_response(url, method=method)
+            status = resp.status_code in [404, 302]
+            self.assertTrue(status)
+        else:
+            resp = self.get_response(url, method=method)
+            if accept_redir:
+                print(resp.status_code)
+                status = resp.status_code in [200, 302]
+                print(status)
+                self.assertTrue(status)
+            else:
+                self.assertEquals(resp.status_code, 200)
+
+
+class TestViews(ViewsMixin, TestCase):
+
+    def setUp(self):
+        self.client = Client()
+        post_args = ['test_case']
+        self.urls = {
+            'index': ['blog:index', []],
+            'post_create': ['blog:post_create', []],
+            'post_detail': ['blog:post_detail', ['test_case']],
+            'post_update': ['blog:post_update', ['test_case']],
+            'post_delete': ['blog:post_delete', ['test_case']]
+        }
+        self.admin = User.objects.create_superuser(
+            username='test_admin',
+            email='[email protected]',
+            password='test_admin'
+        )
+        self.admin.save()
+        self.test_post = Post.objects.create(
+            title='test_case',
+            text='test_case',
+            slug='test_case',
+            tags='test,case'
+        )
+
+    def test_index(self):
+        self.is_callable(url=self.urls['index'])
+        self.is_callable(url=self.urls['index'], post=True, accept_redir=True)
+
+    def test_post_create_callable_authorized(self):
+        # Test authorized get and post request
+        self.is_callable(url=self.urls['post_create'], user=self.admin)
+        self.is_callable(url=self.urls['post_create'], user=self.admin, post=True)
+
+    def test_post_create_callable_unauthorized(self):
+        # Test unauthorized get and post request
+        self.is_callable(url=self.urls['post_create'], user=None, fail=True)
+        self.is_callable(url=self.urls['post_create'], user=None, post=True, fail=True)
+
+    def test_post_detail_callable(self):
+        self.is_callable(url=self.urls['post_detail'])
+
+    def test_post_update_callable_authorized(self):
+        # Test authorized get and post request
+        self.is_callable(url=self.urls['post_update'], user=self.admin)
+        self.is_callable(url=self.urls['post_update'], user=self.admin, post=True)
+
+    def test_post_update_callable_unauthorized(self):
+        # Test unauthorized get and post request
+        self.is_callable(url=self.urls['post_update'], user=None, fail=True)
+        self.is_callable(url=self.urls['post_update'], user=None, post=True, fail=True)
+
+    def test_post_delete_callable_authorized(self):
+        # Test authorized get and post request
+        self.is_callable(url=self.urls['post_delete'], user=self.admin, accept_redir=True)
+
+    def test_post_delete_callable_authorized_POST(self):
+        self.is_callable(url=self.urls['post_delete'], user=self.admin, post=True, accept_redir=True)
+
+    def test_post_delete_callable_unauthorized(self):
+        # Test unauthorized get and post request
+        self.is_callable(url=self.urls['post_delete'], user=None, fail=True)
+        self.is_callable(url=self.urls['post_delete'], user=None, post=True, fail=True)

blog/__pycache__/models.cpython-38.pyc

@@ -8,7 +8,7 @@
     path('', views.index, name='index'),
     # post related routes
     path('post/detail/<slug:slug>/', views.post_detail, name='post_detail'),
-    path('post/delete/<slug:slug>/', views.delete_post, name='post_delete'),
+    path('post/delete/<slug:slug>/', views.post_delete, name='post_delete'),
     path('post/create/', staff_member_required(views.PostCreate.as_view()), name='post_create'),
     path('post/update/<slug:slug>/', staff_member_required(views.PostUpdate.as_view()), name='post_update'),
     # comments related routes

blog/__pycache__/signals.cpython-38.pyc

@@ -9,7 +9,7 @@
 def file_path_gen(instance, filename):
     file = filename
     class_name = instance.__class__.__name__
-    slug = instance.slug if instance.slug else instance.title if instance.title else instance.id
+    slug = instance.title
     file_type = "media" if file.endswith(('.png', '.jpg', '.jpeg')) else "files"
 
     return f'{class_name}/{slug}/{file_type}/{file}'
@@ -18,14 +18,16 @@ def file_path_gen(instance, filename):
 
 
 def compress_image(uploadedImage):
-    imageTemproary = Image.open(uploadedImage)
+    image_temporary = Image.open(uploadedImage)
     outputIoStream = BytesIO()
-    imageTemproaryResized = imageTemproary.resize((1020, 573))
-    try:
-        imageTemproary = pure_pil_alpha_to_color_v2(imageTemproary)
-    except:
-        pass
-    imageTemproary.save(outputIoStream, format='JPEG', quality=80)
+    image_temproary_resized = image_temporary.resize((1020, 573))
+    if image_temporary.mode != 'RGB':
+        try:
+            image_temporary = pure_pil_alpha_to_color_v2(image_temporary)
+        except IndexError:
+            pass
+        image_temporary = image_temporary.convert('RGB')
+    image_temporary.save(outputIoStream, format='JPEG', quality=80)
     outputIoStream.seek(0)
     uploadedImage = InMemoryUploadedFile(outputIoStream, 'ImageField', "%s.jpg" % uploadedImage.name.split('.')[0],
                                          'image/jpeg', sys.getsizeof(outputIoStream), None)

blog/__pycache__/urls.cpython-38.pyc

@@ -84,7 +84,7 @@ class PostUpdate(UpdateView):
 
 
 @staff_member_required
-def delete_post(request, slug):
+def post_delete(request, slug):
     post = get_object_or_404(Post, slug=slug)
     messages.success(request, f'Post "{post.title[:15]}" id:{post.id} został usunięty')
     post.delete()

blog/__pycache__/utils.cpython-38.pyc

@@ -0,0 +1,25 @@
+asgiref==3.2.5
+certifi==2019.11.28
+chardet==3.0.4
+coverage==5.0.4
+Django==3.0.4
+django-admin-interface==0.12.1
+django-ckeditor==5.9.0
+django-cleanup==4.0.0
+django-colorfield==0.2.1
+django-crispy-forms==1.9.0
+django-flat-responsive==2.0
+django-flat-theme==1.1.4
+django-image-optimizer==1.0.0
+django-js-asset==1.2.2
+django-taggit==1.2.0
+idna==2.9
+Pillow==7.0.0
+pipfile==0.0.2
+pytz==2019.3
+requests==2.23.0
+six==1.14.0
+sqlparse==0.3.1
+tinify==1.5.1
+toml==0.10.0
+urllib3==1.25.8