django_lostplaces/lostplaces/models/place.py

@@ -98,9 +98,6 @@ class PlaceAsset(Submittable):
         null=True
     )
 
-class DummyAsset(PlaceAsset):
-    name = models.CharField(max_length=50)
-
 class PlaceImage(PlaceAsset):
     """
     PlaceImage defines an image file object that points to a file in uploads/.

django_lostplaces/lostplaces/templates/flat_form.html

@@ -1,4 +0,0 @@
-<form method={{form.method}}>
-	{{form}}
-	<input type="submit"/>
-</form>

django_lostplaces/lostplaces/tests/views/test_base_views.py

@@ -3,19 +3,14 @@
 
 from django.test import TestCase, RequestFactory, Client
 from django.urls import reverse_lazy
-from django.db import models
 from django.contrib.auth.models import User, AnonymousUser
 from django.contrib.messages.storage.fallback import FallbackStorage
 from django.utils import timezone
 from django.shortcuts import render
 
 from lostplaces.models import Place
-from lostplaces.views import (
-    IsAuthenticatedMixin,
-    PlaceAssetCreateView
-)
+from lostplaces.views import IsAuthenticatedMixin
 from lostplaces.tests.views import ViewTestCase
-from lostplaces.models import DummyAsset
 
 class TestIsAuthenticated(ViewTestCase):
     view = IsAuthenticatedMixin
@@ -27,6 +22,9 @@ class TestIsAuthenticated(ViewTestCase):
             password='Develop123'
         )
         
+    def setUp(self):
+        self.client = Client()
+
     def test_logged_in(self):
         request = RequestFactory().get('/')
         request.user = User.objects.get(id=1)
@@ -45,6 +43,7 @@ class TestIsAuthenticated(ViewTestCase):
         response = IsAuthenticatedMixin.as_view()(request)
         self.assertHttpRedirect(response, '?'.join([str(reverse_lazy('login')), 'next=/']))
         
+        response = self.client.get(response['Location']) 
         self.assertTrue(len(messages) > 0)
 
 class TestIsPlaceSubmitterMixin(TestCase):
@@ -90,50 +89,3 @@ class TestIsPlaceSubmitterMixin(TestCase):
         self.assertTrue(response.context['messages'])
         self.assertTrue(len(response.context['messages']) > 0)
     
-class TestPlaceAssetCreateView(ViewTestCase):
-    view = PlaceAssetCreateView
-        
-    class DummyAssetCreateView(PlaceAssetCreateView):
-        model = DummyAsset
-        fields = ['name']
-        template_name = 'flat_form.html'
-        
-    @classmethod
-    def setUpTestData(cls):
-        user = User.objects.create_user(
-            username='testpeter',
-            password='Develop123'
-        )
-        
-        place = Place.objects.create(
-            name='Im a place',
-            submitted_when=timezone.now(),
-            submitted_by=user.explorer,
-            location='Testtown',
-            latitude=50.5,
-            longitude=7.0,
-            description='This is just a test, do not worry'
-        )
-        
-    def test_create_asset(self):
-        request = RequestFactory().post(
-            '/',
-            {
-                'name': 'im a place'
-            }
-        )
-        request.user = User.objects.get(id=1)
-        
-        response = TestPlaceAssetCreateView.DummyAssetCreateView.as_view()(request, place_id=1)
-        
-        self.assertTrue(
-            DummyAsset.objects.all().count() >= 1,
-            msg='Expecting a dummy place asset to be created by the view'
-        )
-        
-        self.assertEqual(
-            DummyAsset.objects.get(id=1).name,
-            'im a place',
-            msg='Expecting the dummy place asset to have the correct data'
-        )
-    

django_lostplaces/lostplaces/tests/views/test_views.py

@@ -42,7 +42,7 @@ class TestHomeView(GlobalTemplateTestCaseMixin, ViewTestCase):
         self.assertGlobal(response)
         
     def test_place_list_authenticated(self):
-        """
+	    """
         Testing there is the place list containing the name,
         location and description of the latest place for 
         authenticated users.
@@ -59,7 +59,7 @@ class TestHomeView(GlobalTemplateTestCaseMixin, ViewTestCase):
         )
         
     def test_place_list_unauthenticated(self):
-        """
+	    """
         Testing there is a place list of the lates places
         containing the place names only for unauthenticated users.
         """
@@ -82,7 +82,7 @@ class TestHomeView(GlobalTemplateTestCaseMixin, ViewTestCase):
         )
         
     def test_map_authenticated(self):
-        """
+	    """
         Testing there is a map showing all the lates places
         on a map for authenticated users.
         """
@@ -98,7 +98,7 @@ class TestHomeView(GlobalTemplateTestCaseMixin, ViewTestCase):
         )
         
     def test_map_unauthenticated(self):
-        """
+	    """
         Testing there is no map for unauthenticated users.
         """
         response = self.client.get(reverse('lostplaces_home'))