lostplaces/lostplaces_app/models.py

@@ -54,7 +54,7 @@ class Taggable(models.Model):
         
     tags = TaggableManager(blank=True)
     
-class Mapable(models.Model):
+class MapablePoint(models.Model):
     '''
     This abstract model class represents an object that can be
     displayed on a map.
@@ -109,7 +109,7 @@ class Voucher(models.Model):
         return "Voucher " + str(self.code)
 
 
-class Place(Submittable, Taggable, Mapable):
+class Place(Submittable, Taggable, MapablePoint):
     """
     Place defines a lost place (location, name, description etc.).
     """

lostplaces/lostplaces_app/tests/models/test_abstract_models.py

@@ -6,7 +6,7 @@ from django.contrib.auth.models import User
 
 from lostplaces_app.models import (
     Taggable,
-    Mapable,
+    MapablePoint,
     Submittable
 )
 from lostplaces_app.tests.models import ModelTestCase
@@ -22,9 +22,9 @@ class TaggableTestCase(ModelTestCase):
         self.assertField('tags', TaggableManager)
 
 
-class MapableTestCase(ModelTestCase):
+class MapablePointTestCase(ModelTestCase):
 
-    model = Mapable
+    model = MapablePoint
 
     def test_name(self):
         self.assertCharField(

lostplaces/lostplaces_app/tests/views/__init__.py

@@ -1,6 +1,6 @@
 from django.test import TestCase
 
-from lostplaces_app.models import Taggable, Mapable
+from lostplaces_app.models import Taggable, MapablePoint
 
 from taggit.models import Tag
 
@@ -59,7 +59,7 @@ class ViewTestCase(TestCase):
         '''
         self.assertEqual(
             response.status_code, code,
-            msg='Expecting an HTTP %s response, but got HTTP %s' % (
+            "Expected an HTTP %s response, but got HTTP %s" % (
                 code,
                 response.status_code
             )
@@ -165,9 +165,9 @@ class TaggableViewTestCaseMixin:
             msg='Expecting delete_url_name to be of type string'
         )
         
-class MapableViewTestCaseMixin:
+class MapablePointViewTestCaseMixin:
     
-    def assertMapableContext(self, context):
+    def assertMapablePointContext(self, context):
         self.assertTrue(
             'all_points' in context,
             msg='Expecting the context for mapable point to contain \'all_points\' attribute'
@@ -175,9 +175,9 @@ class MapableViewTestCaseMixin:
         
         for point in context['all_points']:
             self.assertTrue(
-                isinstance(point, Mapable),
+                isinstance(point, MapablePoint),
                 msg='Expecting all entries to be an instance of %s, got %s' % (
-                    str(Mapable),
+                    str(MapablePoint),
                     str(type(point))
                 )
             )

lostplaces/lostplaces_app/tests/views/test_place_views.py

@@ -14,7 +14,7 @@ from lostplaces_app.forms import PlaceImageCreateForm, PlaceForm
 from lostplaces_app.tests.views import (
     ViewTestCase,
     TaggableViewTestCaseMixin,
-    MapableViewTestCaseMixin
+    MapablePointViewTestCaseMixin
 )
 
 
@@ -83,7 +83,7 @@ class TestPlaceListView(ViewTestCase):
         self.assertContext(response, 'mapping_config')
 
 
-class PlaceDetailViewTestCase(TaggableViewTestCaseMixin, MapableViewTestCaseMixin, ViewTestCase):
+class PlaceDetailViewTestCase(TaggableViewTestCaseMixin, MapablePointViewTestCaseMixin, ViewTestCase):
     view = PlaceDetailView
 
     @classmethod
@@ -123,4 +123,4 @@ class PlaceDetailViewTestCase(TaggableViewTestCaseMixin, MapableViewTestCaseMixi
                 str(self.view)
             )
         )
-        self.assertMapableContext(response.context['mapping_config'])
+        self.assertMapablePointContext(response.context['mapping_config'])