refactor: clean up variable names again

Parkreiner · Parkreiner · commit 1cc293519aee · 2023-10-09T18:28:32.000Z
diff --git a/site/src/pages/TemplatePage/TemplatePageHeader.tsx b/site/src/pages/TemplatePage/TemplatePageHeader.tsx
@@ -1,6 +1,6 @@
 import { type FC, useRef, useState } from "react";
 import { Link as RouterLink, useNavigate } from "react-router-dom";
-import { useDeletionPopoverState } from "./useDeletionPopoverState";
+import { useDeletionDialogState } from "./useDeletionDialogState";
 
 import { useQuery } from "react-query";
 import { workspacesByQuery } from "api/queries/workspaces";
@@ -123,7 +123,7 @@ export const TemplatePageHeader: FC<TemplatePageHeaderProps> = ({
   onDeleteTemplate,
 }) => {
   const navigate = useNavigate();
-  const popoverState = useDeletionPopoverState(template, onDeleteTemplate);
+  const dialogState = useDeletionDialogState(template, onDeleteTemplate);
 
   const queryText = `template:${template.name}`;
   const workspaceCountQuery = useQuery({
@@ -153,7 +153,7 @@ export const TemplatePageHeader: FC<TemplatePageHeaderProps> = ({
               <TemplateMenu
                 templateVersion={activeVersion.name}
                 templateName={template.name}
-                onDelete={popoverState.openDeleteConfirmation}
+                onDelete={dialogState.openDeleteConfirmation}
               />
             )}
           </>
@@ -184,9 +184,9 @@ export const TemplatePageHeader: FC<TemplatePageHeaderProps> = ({
 
       {safeToDeleteTemplate ? (
         <DeleteDialog
-          isOpen={popoverState.isDeleteDialogOpen}
-          onConfirm={popoverState.confirmDelete}
-          onCancel={popoverState.cancelDeleteConfirmation}
+          isOpen={dialogState.isDeleteDialogOpen}
+          onConfirm={dialogState.confirmDelete}
+          onCancel={dialogState.cancelDeleteConfirmation}
           entity="template"
           name={template.name}
         />
@@ -195,8 +195,8 @@ export const TemplatePageHeader: FC<TemplatePageHeaderProps> = ({
           type="info"
           title="Unable to delete"
           hideCancel={false}
-          open={popoverState.isDeleteDialogOpen}
-          onClose={popoverState.cancelDeleteConfirmation}
+          open={dialogState.isDeleteDialogOpen}
+          onClose={dialogState.cancelDeleteConfirmation}
           confirmText="See workspaces"
           confirmLoading={workspaceCountQuery.status !== "success"}
           onConfirm={() => {
diff --git a/site/src/pages/TemplatePage/useDeletionDialogState.test.ts b/site/src/pages/TemplatePage/useDeletionDialogState.test.ts
@@ -1,19 +1,19 @@
 import { act, renderHook, waitFor } from "@testing-library/react";
 import { MockTemplate } from "testHelpers/entities";
-import { useDeletionPopoverState } from "./useDeletionPopoverState";
+import { useDeletionDialogState } from "./useDeletionDialogState";
 import * as API from "api/api";
 
 test("delete dialog starts closed", () => {
   const { result } = renderHook(() =>
-    useDeletionPopoverState(MockTemplate, jest.fn()),
+    useDeletionDialogState(MockTemplate, jest.fn()),
   );
   expect(result.current.isDeleteDialogOpen).toBeFalsy();
 });
 
 test("confirm template deletion", async () => {
   const onDeleteTemplate = jest.fn();
   const { result } = renderHook(() =>
-    useDeletionPopoverState(MockTemplate, onDeleteTemplate),
+    useDeletionDialogState(MockTemplate, onDeleteTemplate),
   );
 
   //Open delete confirmation
@@ -31,7 +31,7 @@ test("confirm template deletion", async () => {
 
 test("cancel template deletion", () => {
   const { result } = renderHook(() =>
-    useDeletionPopoverState(MockTemplate, jest.fn()),
+    useDeletionDialogState(MockTemplate, jest.fn()),
   );
 
   //Open delete confirmation
diff --git a/site/src/pages/TemplatePage/useDeletionDialogState.ts b/site/src/pages/TemplatePage/useDeletionDialogState.ts
@@ -9,7 +9,7 @@ type DeleteTemplateState =
   | { status: "confirming" }
   | { status: "deleting" };
 
-export const useDeletionPopoverState = (
+export const useDeletionDialogState = (
   template: Template,
   onDelete: () => void,
 ) => {
