- A ``static`` helper class was added to the ``repoze.bfg.views``

     module.  Instances of this class are willing to act as BFG views
     which return static resources using files on disk.  See the
     :mod:`repoze.bfg.view` docs for more info.

3 files changed, 88 insertions, 12 deletions

diff --git a/repoze/bfg/tests/test_view.py b/repoze/bfg/tests/test_view.py
index 46687e904..389c00cc5 100644
--- a/repoze/bfg/tests/test_view.py
+++ b/repoze/bfg/tests/test_view.py
@@ -1,13 +1,13 @@
 import unittest
 
-from zope.component.testing import PlacelessSetup
+from zope.testing.cleanup import cleanUp
 
-class BaseTest(PlacelessSetup):
+class BaseTest(object):
     def setUp(self):
-        PlacelessSetup.setUp(self)
+        cleanUp()
 
     def tearDown(self):
-        PlacelessSetup.tearDown(self)
+        cleanUp()
 
     def _registerView(self, app, name, *for_):
         import zope.component
@@ -30,6 +30,7 @@ class BaseTest(PlacelessSetup):
     def _makeEnviron(self, **extras):
         environ = {
             'wsgi.url_scheme':'http',
+            'wsgi.version':(1,0),
             'SERVER_NAME':'localhost',
             'SERVER_PORT':'8080',
             'REQUEST_METHOD':'GET',
@@ -372,12 +373,12 @@ class TestIsResponse(unittest.TestCase):
         f = self._getFUT()
         self.assertEqual(f(response), False)
 
-class TestViewExecutionPermitted(unittest.TestCase, PlacelessSetup):
+class TestViewExecutionPermitted(unittest.TestCase):
     def setUp(self):
-        PlacelessSetup.setUp(self)
+        cleanUp()
 
     def tearDown(self):
-        PlacelessSetup.tearDown(self)
+        cleanUp()
 
     def _callFUT(self, *arg, **kw):
         from repoze.bfg.view import view_execution_permitted
@@ -433,6 +434,40 @@ class TestViewExecutionPermitted(unittest.TestCase, PlacelessSetup):
         result = self._callFUT(context, request, '')
         self.failUnless(result is True)
 
+class TestStaticView(unittest.TestCase, BaseTest):
+    def setUp(self):
+        cleanUp()
+
+    def tearDown(self):
+        cleanUp()
+
+    def _getTargetClass(self):
+        from repoze.bfg.view import static
+        return static
+
+    def _getStaticDir(self):
+        import os
+        here = os.path.abspath(os.path.normpath(os.path.dirname(__file__)))
+        fixtureapp = os.path.join(here, 'fixtureapp')
+        return fixtureapp
+
+    def _makeOne(self):
+        static_dir = self._getStaticDir()
+        return self._getTargetClass()(static_dir)
+        
+    def test_it(self):
+        view = self._makeOne()
+        context = DummyContext()
+        request = DummyRequest()
+        request.subpath = ['__init__.py']
+        request.environ = self._makeEnviron()
+        response = view(context, request)
+        result = ''.join(list(response.app_iter))
+        static_dir = self._getStaticDir()
+        import os
+        filedata = open(os.path.join(static_dir, '__init__.py')).read()
+        self.assertEqual(result, filedata)
+
 class DummyContext:
     pass
 
diff --git a/repoze/bfg/view.py b/repoze/bfg/view.py
index ae4f304f0..3729fe4e5 100644
--- a/repoze/bfg/view.py
+++ b/repoze/bfg/view.py
@@ -1,3 +1,6 @@
+from paste.urlparser import StaticURLParser
+from webob import Response
+
 from zope.component import queryMultiAdapter
 from zope.component import queryUtility
 
@@ -120,3 +123,41 @@ def is_response(ob):
             return True
     return False
 
+class static(object):
+    """ An instance of this class is a callable which can act as a BFG
+    view; this view will serve static files from a directory on disk
+    based on the ``root_dir`` you provide to its constructor.  The
+    directory may contain subdirectories (recursively); the static
+    view implementation will descend into these directories as
+    necessary based on the components of the URL in order to resolve a
+    path into a response."""
+
+    def __init__(self, root_dir, cache_max_age=3600):
+        """ Pass the absolute filesystem path to the directory
+        containing static files directory as ``root_dir``,
+        ``cache_max_age`` influences the Expires and Max-Age caching
+        headers (default is 3600 seconds or five minutes)."""
+        self.app = StaticURLParser(root_dir, cache_max_age=cache_max_age)
+
+    def __call__(self, context, request):
+        subpath = '/'.join(request.subpath)
+        caught = []
+        def catch_start_response(status, headers, exc_info=None):
+            caught[:] = (status, headers, exc_info)
+        ecopy = request.environ.copy()
+        # Fix up PATH_INFO to get rid of everything but the "subpath"
+        # (the actual path to the file relative to the root dir).
+        # Zero out SCRIPT_NAME for good measure.
+        ecopy['PATH_INFO'] = '/' + subpath
+        ecopy['SCRIPT_NAME'] = ''
+        body = self.app(ecopy, catch_start_response)
+        if caught: 
+            status, headers, exc_info = caught
+            response = Response()
+            response.app_iter = body
+            response.status = status
+            response.headerlist = headers
+            return response
+        else:
+            raise RuntimeError('WSGI start_response not called')
+
diff --git a/repoze/bfg/wsgi.py b/repoze/bfg/wsgi.py
index 2a23fe7aa..149273c27 100644
--- a/repoze/bfg/wsgi.py
+++ b/repoze/bfg/wsgi.py
@@ -19,11 +19,11 @@ def wsgiapp(wrapped):
 
     Allows the following view declaration to be made::
 
-       <bfg:view
-            view=".views.hello_world"
-            name="hello_world.txt"
-            context="*"
-            />
+       <view
+          view=".views.hello_world"
+          name="hello_world.txt"
+          context="*"
+        />
 
     The wsgiapp decorator will convert the result of the WSGI
     application to a Response and return it to repoze.bfg as if the