registerTemplateRenderer -> registerDummyRenderer

2 files changed, 13 insertions, 17 deletions

diff --git a/repoze/bfg/testing.py b/repoze/bfg/testing.py
index 2a186cf26..d32934031 100644
--- a/repoze/bfg/testing.py
+++ b/repoze/bfg/testing.py
@@ -24,9 +24,9 @@ def registerDummySecurityPolicy(userid=None, groupids=(), permissive=True):
 def registerModels(models):
     """ Registers a dictionary of models.  This is most useful for
     testing code that wants to call the
-    ``repoze.bfg.traversal.find_model`` API.  This API is called with
-    a path as one of its arguments.  If the dictionary you register
-    when calling this method contains that path as a key
+    ``repoze.bfg.traversal.find_model`` API.  The ``find_model`` API
+    is called with a path as one of its arguments.  If the dictionary
+    you register when calling this method contains that path as a key
     (e.g. '/foo/bar' or 'foo'), the corresponding value will be
     returned to ``find_model`` (and thus to your code)."""
     traverser = make_traverser_factory(models)
@@ -47,20 +47,16 @@ def registerEventListener(event_iface=Interface):
     registerSubscriber(subscriber, event_iface)
     return L
 
-def registerTemplateRenderer(path, renderer=None, for_=None):
+def registerDummyRenderer(path):
     """ Create and register a dummy template renderer at ``path``
     (usually a relative filename ala ``templates/foo.pt``) and return
-    the renderer object.  If ``renderer`` is not ``None``, it will be
-    registered as the renderer and returned (no dummy renderer object
-    will be created).  This function is useful when testing code that
-    calls the ``render_template_to_response`` or any other
-    ``render_template*`` API of the built-in templating systems. """
-    if for_ is None:
-        from repoze.bfg.interfaces import ITestingTemplateRenderer
-        for_ = ITestingTemplateRenderer
-    if renderer is None:
-        renderer = DummyTemplateRenderer()
-    return registerUtility(renderer, for_, path)
+    the renderer object.  This function is useful when testing code
+    that calls the ``render_template_to_response`` or any other
+    ``render_template*`` API of any of the built-in templating
+    systems."""
+    from repoze.bfg.interfaces import ITestingTemplateRenderer
+    renderer = DummyTemplateRenderer()
+    return registerUtility(renderer, ITestingTemplateRenderer, path)
 
 def registerView(name, result='', view=None, for_=(Interface, Interface)):
     """ Registers ``repoze.bfg`` view function under the name
diff --git a/repoze/bfg/tests/test_testing.py b/repoze/bfg/tests/test_testing.py
index 03f5bd938..5ef66e759 100644
--- a/repoze/bfg/tests/test_testing.py
+++ b/repoze/bfg/tests/test_testing.py
@@ -47,9 +47,9 @@ class TestTestingFunctions(unittest.TestCase, PlacelessSetup):
         from repoze.bfg.traversal import find_model
         self.assertEqual(find_model(None, '/ob1'), ob1)
 
-    def test_registerTemplateRenderer(self):
+    def test_registerDummyRenderer(self):
         from repoze.bfg import testing
-        template = testing.registerTemplateRenderer('templates/foo')
+        template = testing.registerDummyRenderer('templates/foo')
         from repoze.bfg.testing import DummyTemplateRenderer
         self.failUnless(isinstance(template, DummyTemplateRenderer))
         from repoze.bfg.chameleon_zpt import render_template_to_response