Fix remaining lint issues

coder · bryphe-coder · Feb 23, 2022 · Feb 21, 2022 · Feb 21, 2022 · Feb 21, 2022
commit 1a490dfb33ef557dd74be9687e1cf765ec1a1c93
diff --git a/site/components/Form/FormCloseButton.tsx b/site/components/Form/FormCloseButton.tsx
@@ -23,7 +23,7 @@ export const FormCloseButton: React.FC<FormCloseButtonProps> = ({ onClose }) =>
     return () => {
       document.body.removeEventListener("keydown", handleKeyPress, false)
     }
-  }, [])
+  }, [onClose])
 
   return (
     <IconButton className={styles.closeButton} onClick={onClose} size="medium">

diff --git a/site/contexts/UserContext.tsx b/site/contexts/UserContext.tsx
@@ -25,21 +25,21 @@ const UserContext = React.createContext<UserContext>({
 
 export const useUser = (redirectOnError = false): UserContext => {
   const ctx = useContext(UserContext)
-  const router = useRouter()
+  const { push, asPath } = useRouter()
 
   const requestError = ctx.error
   useEffect(() => {
     if (redirectOnError && requestError) {
       // 'void' means we are ignoring handling the promise returned
       // from router.push (and lets the linter know we're OK with that!)
-      void router.push({
+      void push({
         pathname: "/login",
         query: {
-          redirect: router.asPath,
+          redirect: asPath,
         },
       })
     }
-  }, [redirectOnError, requestError])
+  }, [asPath, push, redirectOnError, requestError])
 
   return ctx
 }

diff --git a/site/jest.config.js b/site/jest.config.js
@@ -24,7 +24,15 @@ module.exports = {
       displayName: "lint",
       runner: "jest-runner-eslint",
       testMatch: ["<rootDir>/**/*.js", "<rootDir>/**/*.ts", "<rootDir>/**/*.tsx"],
-      testPathIgnorePatterns: ["/.next/", "/out/", "/_jest/", "jest.config.js", "jest-runner.*.js", "next.config.js"],
+      testPathIgnorePatterns: [
+        "/.next/",
+        "/out/",
+        "/_jest/",
+        "dev.ts",
+        "jest.config.js",
+        "jest-runner.*.js",
+        "next.config.js",
+      ],
     },
   ],
   collectCoverageFrom: [

diff --git a/site/pages/projects/[organization]/[project]/create.tsx b/site/pages/projects/[organization]/[project]/create.tsx
@@ -1,4 +1,4 @@
-import React from "react"
+import React, { useCallback } from "react"
 import { makeStyles } from "@material-ui/core/styles"
 import { useRouter } from "next/router"
 import useSWR from "swr"
@@ -10,14 +10,24 @@ import { FullScreenLoader } from "../../../../components/Loader/FullScreenLoader
 import { CreateWorkspaceForm } from "../../../../forms/CreateWorkspaceForm"
 
 const CreateWorkspacePage: React.FC = () => {
-  const router = useRouter()
+  const { push, query } = useRouter()
   const styles = useStyles()
   const { me } = useUser(/* redirectOnError */ true)
-  const { organization, project: projectName } = router.query
+  const { organization, project: projectName } = query
   const { data: project, error: projectError } = useSWR<API.Project, Error>(
     `/api/v2/projects/${organization}/${projectName}`,
   )
 
+  const onCancel = useCallback(async () => {
+    await push(`/projects/${organization}/${projectName}`)
+  }, [push, organization, projectName])
+
+  const onSubmit = async (req: API.CreateWorkspaceRequest) => {
+    const workspace = await API.Workspace.create(req)
+    await push(`/workspaces/me/${workspace.name}`)
+    return workspace
+  }
+
   if (projectError) {
     return <ErrorSummary error={projectError} />
   }
@@ -26,16 +36,6 @@ const CreateWorkspacePage: React.FC = () => {
     return <FullScreenLoader />
   }
 
-  const onCancel = async () => {
-    await router.push(`/projects/${organization}/${projectName}`)
-  }
-
-  const onSubmit = async (req: API.CreateWorkspaceRequest) => {
-    const workspace = await API.Workspace.create(req)
-    await router.push(`/workspaces/me/${workspace.name}`)
-    return workspace
-  }
-
   return (
     <div className={styles.root}>
       <CreateWorkspaceForm onCancel={onCancel} onSubmit={onSubmit} project={project} />

diff --git a/site/pages/projects/create.tsx b/site/pages/projects/create.tsx
diff --git a/site/pages/workspaces/[user]/[workspace].tsx b/site/pages/workspaces/[user]/[workspace].tsx
@@ -27,9 +27,12 @@ const WorkspacesPage: React.FC = () => {
     // So if the user is the same as 'me', use 'me' as the parameter
     const normalizedUserParam = me && userParam === me.id ? "me" : userParam
 
-    // The SWR API expects us to 'throw' if the query isn't ready yet, so these casts to `any` are OK
-    // because the API expects exceptions.
-    return `/api/v2/workspaces/${(normalizedUserParam as any).toString()}/${(workspaceParam as any).toString()}`
+    // The SWR API expects us to 'throw' if the query isn't ready yet:
+    if (normalizedUserParam === null || workspaceParam === null) {
+      throw "Data not yet available to make API call"
+    }
+
+    return `/api/v2/workspaces/${normalizedUserParam}/${workspaceParam}`
   })
 
   if (workspaceError) {