Remove Express 3.x middleware error stubs

closes #3217

Author: wesleytodd

History.md

@@ -3,6 +3,8 @@
 
 This incorporates all changes after 4.14.1 up to 4.15.0.
 
+  * remove:
+    - Remove Express 3.x middleware error stubs
   * deps: router@~1.3.0
     - Add `next("router")` to exit from router
     - Fix case where `router.use` skipped requests routes did not

lib/express.js

@@ -74,37 +74,3 @@ exports.Router = Router;
  */
 
 exports.static = require('serve-static');
-
-/**
- * Replace removed middleware with an appropriate error message.
- */
-
-[
-  'json',
-  'urlencoded',
-  'bodyParser',
-  'compress',
-  'cookieSession',
-  'session',
-  'logger',
-  'cookieParser',
-  'favicon',
-  'responseTime',
-  'errorHandler',
-  'timeout',
-  'methodOverride',
-  'vhost',
-  'csrf',
-  'directory',
-  'limit',
-  'multipart',
-  'staticCache',
-  'query',
-].forEach(function (name) {
-  Object.defineProperty(exports, name, {
-    get: function () {
-      throw new Error('Most middleware (like ' + name + ') is no longer bundled with Express and must be installed separately. Please see https://github.com/senchalabs/connect#middleware.');
-    },
-    configurable: true
-  });
-});

test/exports.js

@@ -50,12 +50,4 @@ describe('exports', function(){
     .get('/')
     .expect('bar', done);
   })
-
-  it('should throw on old middlewares', function(){
-    var error;
-    try { express.bodyParser; } catch (e) { error = e; }
-    should(error).have.property('message');
-    error.message.should.containEql('middleware');
-    error.message.should.containEql('bodyParser');
-  })
 })