Make error matching exhaustive

Refs #388
PREreview · Oct 24, 2022 · f0e81a1 · f0e81a1
1 parent cd8d4d3
commit f0e81a1
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 8 deletions.
diff --git a/src/write-review/write-review-add-author.ts b/src/write-review/write-review-add-author.ts
@@ -38,7 +38,7 @@ export const writeReviewAddAuthor = flow(
       ),
       RM.filterOrElseW(
         ({ canAddAuthors }) => canAddAuthors,
-        () => 'not-found',
+        () => 'not-found' as const,
       ),
       RM.bindW(
         'form',
@@ -53,8 +53,12 @@ export const writeReviewAddAuthor = flow(
       ),
       RM.orElseW(error =>
         match(error)
+          .with(
+            'no-session',
+            fromMiddlewareK(() => seeOther(format(writeReviewMatch.formatter, { doi: preprint.doi }))),
+          )
           .with('not-found', () => notFound)
-          .otherwise(fromMiddlewareK(() => seeOther(format(writeReviewMatch.formatter, { doi: preprint.doi })))),
+          .exhaustive(),
       ),
     ),
   ),

diff --git a/src/write-review/write-review-change-author.ts b/src/write-review/write-review-change-author.ts
@@ -36,15 +36,15 @@ export const writeReviewChangeAuthor = (doi: PreprintId['doi'], index: number) =
         ),
         RM.filterOrElseW(
           ({ canAddAuthors }) => canAddAuthors,
-          () => 'not-found',
+          () => 'not-found' as const,
         ),
         RM.bindW(
           'form',
           RM.fromReaderTaskK(({ user }) => getForm(user.orcid, preprint.doi)),
         ),
         RM.bindW(
           'author',
-          fromOptionK(() => 'not-found')(
+          fromOptionK(() => 'not-found' as const)(
             flow(
               O.fromNullableK(({ form }) => form.otherAuthors),
               O.chain(RA.lookup(index)),
@@ -61,8 +61,12 @@ export const writeReviewChangeAuthor = (doi: PreprintId['doi'], index: number) =
         ),
         RM.orElseW(error =>
           match(error)
+            .with(
+              'no-session',
+              fromMiddlewareK(() => seeOther(format(writeReviewMatch.formatter, { doi: preprint.doi }))),
+            )
             .with('not-found', () => notFound)
-            .otherwise(fromMiddlewareK(() => seeOther(format(writeReviewMatch.formatter, { doi: preprint.doi })))),
+            .exhaustive(),
         ),
       ),
     ),

diff --git a/src/write-review/write-review-remove-author.ts b/src/write-review/write-review-remove-author.ts
@@ -40,15 +40,15 @@ export const writeReviewRemoveAuthor = (doi: PreprintId['doi'], index: number) =
         ),
         RM.filterOrElseW(
           ({ canAddAuthors }) => canAddAuthors,
-          () => 'not-found',
+          () => 'not-found' as const,
         ),
         RM.bindW(
           'form',
           RM.fromReaderTaskK(({ user }) => getForm(user.orcid, preprint.doi)),
         ),
         RM.bindW(
           'author',
-          fromOptionK(() => 'not-found')(
+          fromOptionK(() => 'not-found' as const)(
             flow(
               O.fromNullableK(({ form }) => form.otherAuthors),
               O.chain(RA.lookup(index)),
@@ -65,8 +65,12 @@ export const writeReviewRemoveAuthor = (doi: PreprintId['doi'], index: number) =
         ),
         RM.orElseW(error =>
           match(error)
+            .with(
+              'no-session',
+              fromMiddlewareK(() => seeOther(format(writeReviewMatch.formatter, { doi: preprint.doi }))),
+            )
             .with('not-found', () => notFound)
-            .otherwise(fromMiddlewareK(() => seeOther(format(writeReviewMatch.formatter, { doi: preprint.doi })))),
+            .exhaustive(),
         ),
       ),
     ),