From 308b85dcb9c1611db1ed293f77bf7e7054aa5abc Mon Sep 17 00:00:00 2001 From: David Banham Date: Wed, 8 Mar 2017 13:02:18 +1100 Subject: [PATCH] s/fail/catch/g --- README.md | 6 +- lib/application.js | 10 +-- lib/entities/attachment.js | 2 +- lib/entity_helpers/journals.js | 2 +- lib/entity_helpers/organisations.js | 2 +- sample_app/sample_app.js | 8 +- sample_app/views/index.handlebars | 6 +- test/accountingtests.js | 110 ++++++++++++++-------------- test/payrolltests.js | 10 +-- 9 files changed, 78 insertions(+), 78 deletions(-) diff --git a/README.md b/README.md index 418b5082..80cf40f5 100644 --- a/README.md +++ b/README.md @@ -135,7 +135,7 @@ Print a count of invoices: xeroClient.core.invoices.getInvoices() .then(function(invoices) { console.log("Invoices: " + invoices.length); -}).fail(function(err) { +}).catch(function(err) { console.error(err); }); ``` @@ -151,7 +151,7 @@ xeroClient.core.contacts.getContacts({ contacts.forEach(function(contact) { console.log(contact.Name); }); -}).fail(function(err) { +}).catch(function(err) { console.error(err); }); ``` @@ -160,7 +160,7 @@ Efficient paging: ```javascript xeroClient.core.contacts.getContacts({ pager: {start:1 /* page number */, callback: onContacts}}) - .fail(function(err) { + .catch(function(err) { console.log('Oh no, an error'); }); diff --git a/lib/application.js b/lib/application.js index 41b25f7d..a183d745 100644 --- a/lib/application.js +++ b/lib/application.js @@ -143,7 +143,7 @@ _.extend(Application.prototype, { } }) - .fail(function(err) { + .catch(function(err) { logger.error(err); throw err; }) @@ -198,7 +198,7 @@ _.extend(Application.prototype, { deferred.resolve(ret); callback && callback(null, obj, res); }) - .fail(function(err) { + .catch(function(err) { logger.error(err); throw err; }) @@ -300,7 +300,7 @@ _.extend(Application.prototype, { callback && callback(null, obj, res); } }) - .fail(function(err) { + .catch(function(err) { logger.error(err); throw err; }) @@ -367,7 +367,7 @@ _.extend(Application.prototype, { if (ret && ret.response) return ret.response; }) - .fail(function(err) { + .catch(function(err) { logger.error(err); throw err; }) @@ -390,7 +390,7 @@ _.extend(Application.prototype, { if (ret && ret.response) return self.convertEntities(ret.response, clonedOptions); }) - .fail(function(err) { + .catch(function(err) { logger.error(err); throw err; }) diff --git a/lib/entities/attachment.js b/lib/entities/attachment.js index 3c219a92..e2300b5f 100644 --- a/lib/entities/attachment.js +++ b/lib/entities/attachment.js @@ -35,7 +35,7 @@ var Attachment = Entity.extend(AttachmentSchema, { console.log(ret); return ret.response.Attachments.Attachment; }) - .fail(function(err) { + .catch(function(err) { console.log(err); }) diff --git a/lib/entity_helpers/journals.js b/lib/entity_helpers/journals.js index 0c5294e6..20063d14 100644 --- a/lib/entity_helpers/journals.js +++ b/lib/entity_helpers/journals.js @@ -53,7 +53,7 @@ var Journals = EntityHelper.extend({ callback && callback(null, journals); return journals; }) - .fail(function(err) { + .catch(function(err) { callback && callback(err); throw err; }) diff --git a/lib/entity_helpers/organisations.js b/lib/entity_helpers/organisations.js index ca8de004..c2776d27 100644 --- a/lib/entity_helpers/organisations.js +++ b/lib/entity_helpers/organisations.js @@ -21,7 +21,7 @@ var Organisations = EntityHelper.extend({ callback && callback(null, _.first(organisations)); return _.first(organisations); }) - .fail(function(err) { + .catch(function(err) { callback && callback(err); throw err; }) diff --git a/sample_app/sample_app.js b/sample_app/sample_app.js index e0f1e069..b8d9d94b 100644 --- a/sample_app/sample_app.js +++ b/sample_app/sample_app.js @@ -155,7 +155,7 @@ app.get('/employees', function(req, res) { } }); }) - .fail(function(err) { + .catch(function(err) { console.log(err) res.render('employees', { error: err, @@ -306,7 +306,7 @@ app.get('/timesheets', function(req, res) { } }); }) - .fail(function(err) { + .catch(function(err) { console.log(err) res.render('timesheets', { error: err, @@ -337,7 +337,7 @@ app.use('/createtimesheet', function(req, res) { .then(function(ret) { res.render('createtimesheet', { timesheets: ret.entities }) }) - .fail(function(err) { + .catch(function(err) { res.render('createtimesheet', { err: err }) }) @@ -400,7 +400,7 @@ app.use('/createinvoice', function(req, res) { .then(function(ret) { res.render('createinvoice', { outcome: 'Invoice created', id: ret.entities[0].InvoiceID }) }) - .fail(function(err) { + .catch(function(err) { res.render('createinvoice', { outcome: 'Error', err: err }) }) diff --git a/sample_app/views/index.handlebars b/sample_app/views/index.handlebars index 4c77d9e4..22ed8391 100644 --- a/sample_app/views/index.handlebars +++ b/sample_app/views/index.handlebars @@ -113,7 +113,7 @@ privateApp.core.invoices.getInvoices() .then(function(invoices) { console.log("Invoices: " + invoices.length); -}).fail(function(err) { +}).catch(function(err) { console.error(err); }); @@ -126,13 +126,13 @@ privateApp.core.contacts.getContacts({ contacts.forEach(function(contact) { console.log(contact.Name); }); -}).fail(function(err) { +}).catch(function(err) { console.error(err); });

Efficient paging:

privateApp.core.contacts.getContacts({ pager: {start:1 /* page number */, callback:onContacts}})
-    .fail(function(err) {
+    .catch(function(err) {
         console.log('Oh no, an error');
     });
 
diff --git a/test/accountingtests.js b/test/accountingtests.js
index 0e326895..ac6080bc 100644
--- a/test/accountingtests.js
+++ b/test/accountingtests.js
@@ -76,7 +76,7 @@ describe('get access for public or partner application', function() {
                 })
                 .then(function() {
                     done();
-                }).fail(function(err) {
+                }).catch(function(err) {
                     done(wrapError(err));
                 });
         });
@@ -188,7 +188,7 @@ describe('get access for public or partner application', function() {
                     })
                     .then(function() {
                         done();
-                    }).fail(function(err) {
+                    }).catch(function(err) {
                         done(wrapError(err));
                     });
             });
@@ -268,7 +268,7 @@ describe('regression tests', function() {
 
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     done(wrapError(err));
                 })
         })
@@ -304,7 +304,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(err);
                     done(wrapError(err));
                 })
@@ -396,7 +396,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -437,7 +437,7 @@ describe('regression tests', function() {
 
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -461,7 +461,7 @@ describe('regression tests', function() {
                     expect(account.AccountID).to.not.equal("");
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -480,12 +480,12 @@ describe('regression tests', function() {
                             expect(thisAccount.Name).to.equal(testAccountData.Name);
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             console.log(util.inspect(err, null, null));
                             done(wrapError(err));
                         });
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -497,7 +497,7 @@ describe('regression tests', function() {
                     expect(response.Status).to.equal("OK");
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -537,7 +537,7 @@ describe('regression tests', function() {
 
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -555,7 +555,7 @@ describe('regression tests', function() {
 
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     done(wrapError(err));
                 })
         })
@@ -566,7 +566,7 @@ describe('regression tests', function() {
                     expect(invoice.InvoiceID).to.not.equal(undefined);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     done(wrapError(err));
                 })
         })
@@ -590,11 +590,11 @@ describe('regression tests', function() {
                             });
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             done(wrapError(err));
                         })
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     done(wrapError(err));
                 })
         })
@@ -656,7 +656,7 @@ describe('regression tests', function() {
                     expect(PaymentID).to.not.equal(undefined);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -673,7 +673,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -686,7 +686,7 @@ describe('regression tests', function() {
                     expect(payment.PaymentID).to.not.equal("");
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -705,7 +705,7 @@ describe('regression tests', function() {
                     expect(response.entities[0].Status).to.equal("DELETED");
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -747,7 +747,7 @@ describe('regression tests', function() {
                     sharedTransaction = response.entities[0].BankTransactionID;
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -763,7 +763,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -776,7 +776,7 @@ describe('regression tests', function() {
                     expect(bankTransaction.BankTransactionID).to.not.equal(undefined);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -805,7 +805,7 @@ describe('regression tests', function() {
                     sampleTransferID = response.entities[0].BankTransferID;
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -820,7 +820,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -834,7 +834,7 @@ describe('regression tests', function() {
                     expect(bankTransfer.BankTransferID).to.equal(sampleTransferID);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -860,7 +860,7 @@ describe('regression tests', function() {
                     sampleTrackingCategory = response.entities[0];
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -885,7 +885,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -909,12 +909,12 @@ describe('regression tests', function() {
                             expect(response.entities[0].Name).to.equal("left");
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             console.log(util.inspect(err, null, null));
                             done(wrapError(err));
                         })
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -926,7 +926,7 @@ describe('regression tests', function() {
                     //console.log(response);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -993,7 +993,7 @@ describe('regression tests', function() {
                         });
                         return currentApp.core.trackingCategories.deleteTrackingCategory(trackingCategoryID)
                     })
-                    .fail(function(err) {
+                    .catch(function(err) {
                         console.log(util.inspect(err, null, null));
                         done(wrapError(err));
                     })
@@ -1013,7 +1013,7 @@ describe('regression tests', function() {
                     expect(response.entities[0].Status).to.equal(sampleTrackingCategory.Status);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1047,7 +1047,7 @@ describe('regression tests', function() {
                     sampleItem.ItemID = response.entities[0].ItemID;
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -1062,7 +1062,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -1074,7 +1074,7 @@ describe('regression tests', function() {
                     expect(item.ItemID).to.equal(sampleItem.ItemID);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -1095,12 +1095,12 @@ describe('regression tests', function() {
                             expect(response.entities[0].Name).to.equal(randomName);
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             console.log(util.inspect(err, null, null));
                             done(wrapError(err));
                         })
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -1111,7 +1111,7 @@ describe('regression tests', function() {
                 .then(function() {
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -1140,7 +1140,7 @@ describe('regression tests', function() {
 
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(err)
                     done(wrapError(err));
                 })
@@ -1155,14 +1155,14 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
         })
         it('get (paging)', function(done) {
             currentApp.core.contacts.getContacts({ pager: { start: 1, callback: onContacts } })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1192,7 +1192,7 @@ describe('regression tests', function() {
                     expect(contact.ContactID).to.equal(sampleContact.ContactID);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1208,7 +1208,7 @@ describe('regression tests', function() {
                     done();
 
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1228,7 +1228,7 @@ describe('regression tests', function() {
                     });
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1266,12 +1266,12 @@ describe('regression tests', function() {
                             expect(updatedContact.response.Contacts.Contact.Name).to.equal(newName);
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             console.log(util.inspect(err, null, null));
                             done(wrapError(err));
                         })
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1285,12 +1285,12 @@ describe('regression tests', function() {
                             console.log(attachments);
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             console.log(util.inspect(err, null, null));
                             done(wrapError(err));
                         })
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 });
@@ -1302,7 +1302,7 @@ describe('regression tests', function() {
 
         it('get (paging with callback)', function(done) {
             currentApp.core.journals.getJournals({ pager: { start: 1, callback: onJournals } })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1337,7 +1337,7 @@ describe('regression tests', function() {
                     expect(journals).to.not.equal(undefined);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1353,7 +1353,7 @@ describe('regression tests', function() {
                     sampleJournalId = _.first(journals).JournalID;
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1366,7 +1366,7 @@ describe('regression tests', function() {
                     expect(journal.JournalID).to.equal(sampleJournalId);
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(util.inspect(err, null, null));
                     done(wrapError(err));
                 })
@@ -1380,7 +1380,7 @@ describe('regression tests', function() {
                 .then(function(users) {
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(err)
                     done(wrapError(err));
                 });
@@ -1425,14 +1425,14 @@ describe('regression tests', function() {
                             console.log(response);
                             done();
                         })
-                        .fail(function(err) {
+                        .catch(function(err) {
                             console.log(err)
                             done(wrapError(err));
                         })
 
                     done();
                 })
-                .fail(function(err) {
+                .catch(function(err) {
                     console.log(err)
                     done(wrapError(err));
                 })
diff --git a/test/payrolltests.js b/test/payrolltests.js
index a8f15937..15833471 100644
--- a/test/payrolltests.js
+++ b/test/payrolltests.js
@@ -5,7 +5,7 @@ describe.skip('payitems', function() {
                 console.log(payitems[0].EarningsTypes);
                 done();
             })
-            .fail(function(err) {
+            .catch(function(err) {
                 done(wrapError(err));
             })
     })
@@ -26,7 +26,7 @@ describe.skip('timesheets', function() {
             .then(function() {
                 done();
             })
-            .fail(function(err) {
+            .catch(function(err) {
                 done(wrapError(err));
             })
 
@@ -38,7 +38,7 @@ describe.skip('timesheets', function() {
                     console.log(util.inspect(timesheets[0].toObject(), null, null));
                 done();
             })
-            .fail(function(err) {
+            .catch(function(err) {
                 done(wrapError(err));
             })
     })
@@ -52,7 +52,7 @@ describe.skip('employees', function() {
                 console.log(ret[0].toObject());
                 done();
             })
-            .fail(function(err) {
+            .catch(function(err) {
                 done(wrapError(err));
             })
     })
@@ -63,7 +63,7 @@ describe.skip('employees', function() {
                 console.log(employee.toObject());
                 done();
             })
-            .fail(function(err) {
+            .catch(function(err) {
                 done(wrapError(err));
             })
     })