organize error constants

eshaham · May 24, 2019 · 540369a · 540369a
1 parent 3dbac70
commit 540369a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 13 deletions.
diff --git a/src/constants.js b/src/constants.js
@@ -27,7 +27,7 @@ export const LOGIN_RESULT = {
   UNKNOWN_ERROR: 'UNKNOWN_ERROR',
 };
 
-export const NAVIGATION_ERRORS = {
+export const ERRORS = {
   TIMEOUT: 'TIMEOUT',
   GENERIC: 'GENERIC',
 };

diff --git a/src/helpers/navigation.js b/src/helpers/navigation.js
@@ -1,10 +1,5 @@
 import waitUntil from './waiting';
 
-export const NAVIGATION_ERRORS = {
-  TIMEOUT: 'timeout',
-  GENERIC: 'generic',
-};
-
 export async function waitForNavigation(page, options) {
   await page.waitForNavigation(options);
 }

diff --git a/src/scrapers/base-scraper.js b/src/scrapers/base-scraper.js
@@ -1,7 +1,6 @@
 import { EventEmitter } from 'events';
 
-import { SCRAPE_PROGRESS_TYPES, LOGIN_RESULT } from '../constants';
-import { NAVIGATION_ERRORS } from '../helpers/navigation';
+import { SCRAPE_PROGRESS_TYPES, LOGIN_RESULT, ERRORS } from '../constants';
 
 const SCRAPE_PROGRESS = 'SCRAPE_PROGRESS';
 
@@ -14,11 +13,11 @@ function createErrorResult(errorType, errorMessage) {
 }
 
 function createTimeoutError(errorMessage) {
-  return createErrorResult(NAVIGATION_ERRORS.TIMEOUT, errorMessage);
+  return createErrorResult(ERRORS.TIMEOUT, errorMessage);
 }
 
-function createGenericNavigationError(errorMessage) {
-  return createErrorResult(NAVIGATION_ERRORS.GENERIC, errorMessage);
+function createGenericError(errorMessage) {
+  return createErrorResult(ERRORS.GENERIC, errorMessage);
 }
 
 class BaseScraper {
@@ -41,7 +40,7 @@ class BaseScraper {
     } catch (e) {
       loginResult = e.timeout ?
         createTimeoutError(e.message) :
-        createGenericNavigationError(e.message);
+        createGenericError(e.message);
     }
 
     let scrapeResult;
@@ -52,7 +51,7 @@ class BaseScraper {
         scrapeResult =
           e.timeout ?
             createTimeoutError(e.message) :
-            createGenericNavigationError(e.message);
+            createGenericError(e.message);
       }
     } else {
       scrapeResult = loginResult;