Remove 'cause' pattern for errors.

Fixes googleapis#56.

Author: jmuk

lib/api_callable.js

@@ -212,13 +212,12 @@ function retryable(aFunc, retry, otherArgs) {
         }
         cancelFunc = null;
         if (retry.retryCodes.indexOf(err.code) < 0) {
-          error = new Error('Exception occurred in retry method that was ' +
-              'not classified as transient');
-          error.cause = err;
-          callback(error);
+          err.note = 'Exception occurred in retry method that was ' +
+              'not classified as transient';
+          callback(err);
         } else {
-          error = new Error('Retry total timeout exceeded with exception');
-          error.cause = err;
+          error = err;
+          error.note = 'Retry total timeout exceeded with exception';
           var toSleep = Math.random() * delay;
           timeoutId = setTimeout(function() {
             now = new Date();

package.json

@@ -1,6 +1,6 @@
 {
   "name": "google-gax",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Google API Extensions",
   "main": "index.js",
   "files": [

test/api_callable.js

@@ -371,9 +371,9 @@ describe('retryable', function() {
     var spy = sinon.spy(fail);
     var apiCall = createApiCall(spy, settings);
     apiCall(null, null, function(err, resp) {
-      expect(err).to.be.ok;
-      expect(err.cause).to.be.an('error');
-      expect(err.cause.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err).to.be.an('error');
+      expect(err.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err.note).to.be.ok;
       expect(spy.callCount).to.eq(1);
       done();
     });
@@ -382,9 +382,9 @@ describe('retryable', function() {
   it('aborts retries', function(done) {
     var apiCall = createApiCall(fail, settings);
     apiCall(null, null, function(err, resp) {
-      expect(err).to.be.ok;
       expect(err).to.be.an('error');
-      expect(err.cause.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err.note).to.be.ok;
       done();
     });
   });
@@ -394,9 +394,9 @@ describe('retryable', function() {
     var spy = sinon.spy(fail);
     var apiCall = createApiCall(spy, settings);
     apiCall(null, null, function(err, resp) {
-      expect(err).to.be.ok;
-      expect(err.cause).to.be.an('error');
-      expect(err.cause.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err).to.be.an('error');
+      expect(err.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err.note).to.be.ok;
       expect(spy.callCount).to.eq(toAttempt);
       done();
     });
@@ -411,9 +411,9 @@ describe('retryable', function() {
     var spy = sinon.spy(func);
     var apiCall = createApiCall(spy, settings);
     apiCall(null, null, function(err, resp) {
-      expect(err).to.be.ok;
-      expect(err.cause).to.be.an('error');
-      expect(err.cause.code).to.eq(FAKE_STATUS_CODE_2);
+      expect(err).to.be.an('error');
+      expect(err.code).to.eq(FAKE_STATUS_CODE_2);
+      expect(err.note).to.be.ok;
       expect(spy.callCount).to.eq(1);
       done();
     });
@@ -441,9 +441,9 @@ describe('retryable', function() {
         spy, new gax.CallSettings({timeout: 0, retry: retryOptions}));
 
     apiCall(null, null, function(err, resp) {
-      expect(err).to.be.ok;
-      expect(err.cause).to.be.an('error');
-      expect(err.cause.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err).to.be.an('error');
+      expect(err.code).to.eq(FAKE_STATUS_CODE_1);
+      expect(err.note).to.be.ok;
       var now = new Date();
       expect(now.getTime() - startTime.getTime()).to.be.at.least(
           backoff.totalTimeoutMillis);