- The ``unicode_path_segments`` configuration variable and the

  ``BFG_UNICODE_PATH_SEGMENTS`` configuration variable have been
  removed.  Path segments are now always passed to model
  ``__getitem__`` methods as unicode.  "True" has been the default for
  this setting since 0.5.4, but changing this configuration setting to
  false allowed you to go back to passing raw path element strings to
  model ``__getitem__`` methods.  This services a speed goal (we get
  about +80 req/s by removing the check), and it's clearer just to
  always expect unicode path segments in model ``__getitem__``
  methods.

1 files changed, 0 insertions, 17 deletions

diff --git a/repoze/bfg/tests/test_settings.py b/repoze/bfg/tests/test_settings.py
index 8c1fe2c39..97fe9d026 100644
--- a/repoze/bfg/tests/test_settings.py
+++ b/repoze/bfg/tests/test_settings.py
@@ -14,14 +14,12 @@ class TestSettings(unittest.TestCase):
         self.assertEqual(settings.reload_templates, False)
         self.assertEqual(settings.debug_notfound, False)
         self.assertEqual(settings.debug_authorization, False)
-        self.assertEqual(settings.unicode_path_segments, True)
 
     def test_with_option(self):
         settings = self._makeOne(reload_templates=True)
         self.assertEqual(settings.reload_templates, True)
         self.assertEqual(settings.debug_notfound, False)
         self.assertEqual(settings.debug_authorization, False)
-        self.assertEqual(settings.unicode_path_segments, True)
 
 class TestGetOptions(unittest.TestCase):
     def _callFUT(self, *arg, **kw):
@@ -94,21 +92,6 @@ class TestGetOptions(unittest.TestCase):
         self.assertEqual(result['debug_notfound'], True)
         self.assertEqual(result['debug_authorization'], True)
 
-    def test_unicode_path_segments(self):
-        result = self._callFUT({})
-        self.assertEqual(result['unicode_path_segments'], True)
-        result = self._callFUT({'unicode_path_segments':'false'})
-        self.assertEqual(result['unicode_path_segments'], False)
-        result = self._callFUT({'unicode_path_segments':'t'})
-        self.assertEqual(result['unicode_path_segments'], True)
-        result = self._callFUT({'unicode_path_segments':'1'})
-        self.assertEqual(result['unicode_path_segments'], True)
-        result = self._callFUT({}, {'BFG_UNICODE_PATH_SEGMENTS':'1'})
-        self.assertEqual(result['unicode_path_segments'], True)
-        result = self._callFUT({'unicode_path_segments':'false'},
-                             {'BFG_UNICODE_PATH_SEGMENTS':'1'})
-        self.assertEqual(result['unicode_path_segments'], True)
-
     def test_originals_kept(self):
         result = self._callFUT({'a':'i am so a'})
         self.assertEqual(result['a'], 'i am so a')