Kill deprecated stuff for v2.0

Author: LewisJEllis

index.js

@@ -2,11 +2,7 @@
 
 module.exports = require('./lib/client');
 module.exports.utils = require('./lib/utils');
-module.exports.middleware = {
-  connect: require('./lib/middleware/connect')
-};
-// friendly alias for "raven.middleware.express"
-module.exports.middleware.express = module.exports.middleware.connect;
+
 module.exports.transports = require('./lib/transports');
 module.exports.parsers = require('./lib/parsers');
 

lib/client.js

@@ -509,35 +509,6 @@ extend(Raven.prototype, {
   }
 });
 
-// Deprecations
-extend(Raven.prototype, {
-  getIdent: function getIdent(result) {
-    utils.consoleAlertOnce('getIdent has been deprecated and will be removed in v2.0');
-    return result;
-  },
-  captureError: function captureError() {
-    utils.consoleAlertOnce('captureError has been deprecated and will be removed in v2.0; use captureException instead');
-    return this.captureException.apply(this, arguments);
-  },
-  captureQuery: function captureQuery() {
-    utils.consoleAlertOnce('captureQuery has been deprecated and will be removed in v2.0');
-    return this;
-  },
-  setUserContext: function setUserContext() {
-    utils.consoleAlertOnce('setUserContext has been deprecated and will be removed in v2.0; use setContext instead');
-    return this;
-  },
-  setExtraContext: function setExtraContext() {
-    utils.consoleAlertOnce('setExtraContext has been deprecated and will be removed in v2.0; use setContext instead');
-    return this;
-  },
-  setTagsContext: function setTagsContext() {
-    utils.consoleAlertOnce('setTagsContext has been deprecated and will be removed in v2.0; use setContext instead');
-    return this;
-  },
-});
-Raven.prototype.get_ident = Raven.prototype.getIdent;
-
 // Maintain old API compat, need to make sure arguments length is preserved
 function Client(dsn, options) {
   if (dsn instanceof Client) return dsn;

lib/middleware/connect.js

@@ -145,13 +145,6 @@ describe('raven.Client', function () {
     delete process.env.SENTRY_ENVIRONMENT;
   });
 
-  describe('#getIdent()', function () {
-    it('should match', function () {
-      var result = 'c988bf5cb7db4653825c92f6864e7206';
-      client.getIdent(result).should.equal('c988bf5cb7db4653825c92f6864e7206');
-    });
-  });
-
   describe('#captureMessage()', function () {
     it('should send a plain text message to Sentry server', function (done) {
       var scope = nock('https://app.getsentry.com')
@@ -208,7 +201,7 @@ describe('raven.Client', function () {
     });
   });
 
-  describe('#captureError()', function () {
+  describe('#captureException()', function () {
     it('should send an Error to Sentry server', function (done) {
       var scope = nock('https://app.getsentry.com')
         .filteringRequestBody(/.*/, '*')
@@ -219,7 +212,7 @@ describe('raven.Client', function () {
         scope.done();
         done();
       });
-      client.captureError(new Error('wtf?'));
+      client.captureException(new Error('wtf?'));
     });
 
     it('should send a plain text "error" with a synthesized stack', function (done) {
@@ -248,7 +241,7 @@ describe('raven.Client', function () {
         scope.done();
         done();
       });
-      client.captureError(new Error('wtf?'));
+      client.captureException(new Error('wtf?'));
     });
 
     it('shouldn\'t choke on circular references', function (done) {
@@ -271,7 +264,7 @@ describe('raven.Client', function () {
         }
       };
       kwargs.extra.foo = kwargs;
-      client.captureError(new Error('wtf?'), kwargs);
+      client.captureException(new Error('wtf?'), kwargs);
     });
   });
 
@@ -738,7 +731,7 @@ describe('raven.Client', function () {
     client.on('logged', function () {
       scope.done();
     });
-    client.captureError(new Error('wtf?'));
+    client.captureException(new Error('wtf?'));
   });
 
   it('should capture extra data', function (done) {
@@ -1201,20 +1194,7 @@ describe('raven.Client', function () {
   });
 });
 
-describe('raven.middleware', function () {
-  it('should use an instance passed to it instead of making a new one', function () {
-    var client = new raven.Client(dsn);
-    raven.middleware.express.getClient(client).should.equal(client);
-  });
-
-  it('should make a new instance when passed a DSN string', function () {
-    var client1 = new raven.Client(dsn);
-    var client2 = raven.middleware.express.getClient(dsn);
-    client2.should.not.equal(raven);
-    client2.should.not.equal(client1);
-    client2.should.be.an.instanceof(raven.constructor);
-  });
-
+describe('raven requestHandler/errorHandler middleware', function () {
   it('should explicitly add req and res to the domain', function (done) {
     var client = new raven.Client(dsn).install();
     var message = 'test breadcrumb';