Merge 58dec4d into f9a7ca3

masonite/request.py

@@ -667,7 +667,7 @@ def _get_named_route(self, name, params):
             if route.named_route == name:
                 return self.compile_route_to_url(route.route_url, params)
 
-        return None
+        raise RouteException("Could not find the route with the name of '{}'".format(name))
 
     def _get_route_from_controller(self, controller):
         """Get the route using the controller.

tests/test_requests.py

@@ -1,5 +1,10 @@
+import unittest
+from cgi import MiniFieldStorage
+from pydoc import locate
 import pytest
 
+
+
 from config import application, providers
 from pydoc import locate
 from app.http.test_controllers.TestController import TestController
@@ -218,6 +223,21 @@ def test_redirections_reset(self):
 
         assert request.redirect_url is False
 
+    def test_redirect_to_throws_exception_when_no_routes_found(self):
+        app = App()
+        app.bind('Request', self.request)
+        app.bind('WebRoutes', WEB_ROUTES)
+        request = app.make('Request').load_app(app)
+
+        request.redirect_to('test')
+        request.redirect(name='test')
+
+        with pytest.raises(RouteException):
+            request.redirect_to('notavailable')
+
+        with pytest.raises(RouteException):
+            request.redirect(name='notavailable')
+
     def test_request_has_subdomain_returns_bool(self):
         app = App()
         app.bind('Request', self.request)