Fix node tests

finalhandler returns a request body that we do not expect. Use a simple 404 instead.

Author: brycekahle

package.json

@@ -37,7 +37,6 @@
     "envify": "^4.0.0",
     "eslint": "^7.6.0",
     "expect.js": "~0.3.1",
-    "finalhandler": "^1.1.2",
     "gulp": "^4.0.2",
     "gulp-header": "^2.0.9",
     "gulp-rename": "^2.0.0",

tests/support/sockjs_server.js

@@ -3,7 +3,6 @@
 
 var http = require('http');
 var serveStatic = require('serve-static');
-var finalhandler = require('finalhandler');
 var sockjs = require('./sockjs_app');
 var url = require('url');
 var path = require('path');
@@ -19,7 +18,7 @@ function startServer(port, config, prefix) {
     }
   };
 
-  var serve = serveStatic(path.join(__dirname, '../html'));
+  var serve = serveStatic(path.join(__dirname, '../html'), { fallthrough: false });
 
   var server = http.createServer();
   server.addListener('request', function(req, res) {
@@ -53,7 +52,10 @@ function startServer(port, config, prefix) {
       res.end('var clientOptions = ' +
               JSON.stringify(clientOptions) + ';');
     } else {
-      serve(req, res, finalhandler(req, res));
+      serve(req, res, function(err) {
+        var status = err ? err.statusCode : 404;
+        return res.writeHead(status).end();
+      });
     }
   });
   server.addListener('upgrade', function(req, res){