Merge pull request #24 from abraham/never-error

Switch to NeverError subclass of Error
abraham · Aug 1, 2018 · 57b7592 · 57b7592
2 parents b391244 + c239cb6
commit 57b7592
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/index.test.ts b/src/index.test.ts
@@ -138,5 +138,5 @@ test('fold unknown', () => {
     successMock,
   );
 
-  expect(() => view(otherMock)).toThrowError('Unknown RemoteData type used');
+  expect(() => view(otherMock)).toThrowError('Unknown RemoteData state:');
 });
diff --git a/src/index.ts b/src/index.ts
@@ -20,7 +20,7 @@ export class Failure<E> {
 
   constructor(public error: E) {
     if (error === null || error === undefined) {
-      fail('Parameter "error" is required');
+      throw new Error('Parameter "error" is required');
     }
   }
 }
@@ -30,7 +30,7 @@ export class Success<D> {
 
   constructor(public data: D) {
     if (data === null || data === undefined) {
-      fail('Parameter "data" is required');
+      throw new Error('Parameter "data" is required');
     }
   }
 }
@@ -52,11 +52,13 @@ export function fold<T, E, D>(
       case Kinds.Success:
         return success(state.data);
       default:
-        return fail('Unknown RemoteData type used');
+        throw new NeverError(state);
     }
   }
 }
 
-function fail(error: string): never {
-  throw new Error(error);
+class NeverError extends Error {
+  constructor(value: never) {
+    super(`Unknown RemoteData state: ${value}`);
+  }
 }