Rename generic Errors to IMAPErrors

This commit is contained in:
Evan Morikawa 2016-11-22 18:24:46 -08:00
parent 65fafc0390
commit 3ff98f45a9
4 changed files with 9 additions and 9 deletions

View file

@ -7,7 +7,7 @@ const Serialization = require('../serialization');
const {
IMAPConnection,
Provider,
Errors,
IMAPErrors,
} = require('nylas-core');
const DefaultSyncPolicy = require('../default-sync-policy')
const LocalDatabaseConnector = require('../../shared/local-database-connector')
@ -171,7 +171,7 @@ module.exports = (server) => {
reply(Serialization.jsonStringify(response));
})
.catch((err) => {
const code = err instanceof Errors.IMAPAuthenticationError ? 401 : 400
const code = err instanceof IMAPErrors.IMAPAuthenticationError ? 401 : 400
reply({message: err.message, type: "api_error"}).code(code);
})
},

View file

@ -1,7 +1,7 @@
const {
IMAPConnection,
MessageTypes,
Errors,
IMAPErrors,
PromiseUtils,
} = require('nylas-core');
const LocalDatabaseConnector = require('../shared/local-database-connector')
@ -211,7 +211,7 @@ class SyncWorker {
})
// Continue to retry if it was a network error
if (error instanceof Errors.RetryableError) {
if (error instanceof IMAPErrors.RetryableError) {
return Promise.resolve()
}

View file

@ -1,5 +1,5 @@
/**
* Errors may come from:
* IMAPErrors may come from:
*
* 1. Underlying IMAP provider (Fastmail, Yahoo, etc)
* 2. Node IMAP
@ -60,13 +60,13 @@ function convertImapError(imapError) {
}
/**
* An abstract base class that can be used to indicate errors that may fix
* themselves when retried
* An abstract base class that can be used to indicate IMAPErrors that may
* fix themselves when retried
*/
class RetryableError extends Error { }
/**
* Errors that originate from NodeIMAP. See `convertImapError` for
* IMAPErrors that originate from NodeIMAP. See `convertImapError` for
* documentation on underlying causes
*/
class IMAPSocketError extends RetryableError { }

View file

@ -7,6 +7,6 @@ module.exports = {
IMAPConnection: require('./imap-connection'),
MessageTypes: require('./message-types'),
Logger: require('./logger'),
Errors: require('./imap-errors'),
IMAPErrors: require('./imap-errors'),
PromiseUtils: require('./promise-utils'),
}