Added a unit test that asserts that HTTPFound(location=None) throws a ValueError.

1 files changed, 10 insertions, 0 deletions

diff --git a/pyramid/tests/test_httpexceptions.py b/pyramid/tests/test_httpexceptions.py
index 927d27733..d319a59a0 100644
--- a/pyramid/tests/test_httpexceptions.py
+++ b/pyramid/tests/test_httpexceptions.py
@@ -294,6 +294,16 @@ class Test_HTTPMove(unittest.TestCase):
         from pyramid.httpexceptions import _HTTPMove
         return _HTTPMove(*arg, **kw)
 
+    def test_it_location_none_valueerrors(self):
+        """ Constructing a HTTPMove instance with location=None should
+            throw a ValueError from __init__ so that a more-confusing
+            exception won't be thrown later from .prepare(environ) """
+        try:
+            exc = self._makeOne(location=None)
+            self.assertEqual(1, 0)
+        except ValueError, e:
+            pass
+
     def test_it_location_not_passed(self):
         exc = self._makeOne()
         self.assertEqual(exc.location, '')