[Restify 7] Fix uncaught exceptions triggering route lookups

lib/server.js

@@ -1077,7 +1077,7 @@ Server.prototype._onHandlerError = function _onHandlerError(
     var self = this;
 
     // Call route by name
-    if (_.isString(err)) {
+    if (!isUncaught && _.isString(err)) {
         var routeName = err;
         var routeHandler = self.router.lookupByName(routeName, req, res);
 
@@ -1482,9 +1482,10 @@ function errEvtNameFromError(err) {
     } else if (err.name === 'InvalidVersionError') {
         // remap the name for router errors
         return 'VersionNotAllowed';
-    } else {
+    } else if (err.name) {
         return err.name.replace(/Error$/, '');
     }
+    return '';
 }
 
 /**

test/server.test.js

@@ -2477,6 +2477,36 @@ test('should emit error with multiple next calls with strictNext', function(t) {
     });
 });
 
+test('uncaughtException should not trigger named routeHandler', function(t) {
+    SERVER.get(
+        {
+            name: 'foo',
+            path: '/foo'
+        },
+        function(req, res, next) {
+            throw 'bar'; //eslint-disable-line no-throw-literal
+        }
+    );
+
+    SERVER.get(
+        {
+            name: 'bar',
+            path: '/bar'
+        },
+        function(req, res, next) {
+            // This code should not run, but we can test against the status code
+            res.send(200);
+            next();
+        }
+    );
+
+    CLIENT.get('/foo', function(err, _, res) {
+        t.ok(err);
+        t.equal(res.statusCode, 500);
+        t.end();
+    });
+});
+
 test('should have proxy event handlers as instance', function(t) {
     var server = restify.createServer({
         handleUpgrades: false