Reorganise routes.handlers

conman/routes/handlers/base/__init__.py → conman/routes/handlers.py

@@ -30,3 +30,27 @@ def handle(self, request, path):
         """
         view, args, kwargs = resolve(path, urlconf=self.urlconf)
         return view(request, *args, route=self.route, **kwargs)
+
+
+class UnboundViewMeta(type):
+    """
+    Metaclass that wraps the `view` attribute with `staticmethod`.
+
+    This ensures that the method does not bind to the class unintentionally.
+    """
+    def __new__(cls, name, bases, attrs):
+        """Create the new class with a staticmethod view attribute."""
+        view = attrs.get('view')
+        if view:
+            attrs['view'] = staticmethod(view)
+        return super().__new__(cls, name, bases, attrs)
+
+
+class SimpleHandler(BaseHandler, metaclass=UnboundViewMeta):
+    """
+    Abstract handler for Routes that have one url: `/` relative to the Route.
+
+    Subclasses should define a view on the class as `view`. This will be
+    called if the `path` passed to `handle` is `/`.
+    """
+    urlconf = 'conman.routes.simple.urls'

conman/routes/tests/test_handlers.py

@@ -11,7 +11,7 @@ class BaseHandlerPathTest(TestCase):
     """Test BaseHandler.path()."""
     def test_path(self):
         """Test directly on base class."""
-        base_handler_path = 'conman.routes.handlers.base.BaseHandler'
+        base_handler_path = 'conman.routes.handlers.BaseHandler'
         self.assertEqual(BaseHandler.path(), base_handler_path)
 
     def test_path_on_subclass(self):