#11 Refactor: Disentangle QuestionList

2020-12-30 22:46:26 +01:00 · 2020-12-30 22:46:26 +01:00 · 0016e12acd
parent ff505226ab
commit 0016e12acd
6 changed files with 185 additions and 170 deletions
--- a/redaktions-app/src/backend/queries/question.mock.ts
+++ b/redaktions-app/src/backend/queries/question.mock.ts
@ -1,5 +1,11 @@
 import {MockedResponse} from "@apollo/client/testing";
-import {BasicQuestionResponse, GET_ALL_QUESTIONS, GetAllQuestionsResponse} from "./question";
+import {
+  BasicQuestionResponse,
+  GET_ALL_QUESTIONS,
+  GET_QUESTION_BY_ID,
+  GetAllQuestionsResponse,
+  GetQuestionByIdResponse
+} from "./question";


 export const questionNodesMock: Array<BasicQuestionResponse> = [{
@ -46,3 +52,32 @@ export const getAllQuestionsMock: Array<MockedResponse<GetAllQuestionsResponse>>
  },
 ]

+export const getQuestionByIdMock: Array<MockedResponse<GetQuestionByIdResponse>> = [
+  ...questionNodesMock.map(q => ({
+    request: {
+      query: GET_QUESTION_BY_ID,
+      variables: {
+        id: q.id,
+      },
+    },
+    result: {
+      data: {
+        question: q,
+      },
+    },
+  })),
+  {
+    request: {
+      query: GET_QUESTION_BY_ID,
+      variables: {
+        id: "",
+      },
+    },
+    result: {
+      data: {
+        question: null,
+      },
+    },
+  }
+]
+
--- a/redaktions-app/src/backend/queries/question.ts
+++ b/redaktions-app/src/backend/queries/question.ts
@ -52,3 +52,20 @@ export interface GetAllQuestionsResponse {
    __typename: "QuestionsConnection",
  }
 }
+
+export const GET_QUESTION_BY_ID = gql`
+  query GetQuestionById($id:ID!) {
+    question(id: $id) {
+      ...BasicQuestionFragment
+    }
+  }
+  ${BasicQuestionFragment}
+`
+
+export interface GetQuestionByIdResponse {
+  question: BasicQuestionResponse | null,
+}
+
+export interface GetQuestionByIdVariables {
+  id: string,
+}
--- a/redaktions-app/src/components/DialogChangeCategory.tsx
+++ b/redaktions-app/src/components/DialogChangeCategory.tsx
@ -42,7 +42,7 @@ export default function DialogChangeCategory() {
      setTitle(data.category?.title || "");
      setDetails(data.category?.description || "")
    })
-  })
+  });
  const [editCategory, {loading: editLoading}] = useMutation<EditCategoryResponse, EditCategoryVariables>(EDIT_CATEGORY, {
    onError: (e) => enqueueSnackbar(`Ein Fehler ist aufgetreten: ${e.message}`, {variant: "error"}),
    onCompleted: (response) => {
--- a/redaktions-app/src/components/DialogChangeQuestion.tsx
+++ b/redaktions-app/src/components/DialogChangeQuestion.tsx
@ -1,61 +1,134 @@
-import React from 'react';
+import React, {useState} from 'react';
 import Dialog from '@material-ui/core/Dialog';
 import DialogContent from '@material-ui/core/DialogContent';
 import DialogTitle from '@material-ui/core/DialogTitle';
-import {BasicCategoryResponse} from "../backend/queries/category";
-import CategorySelectionMenu from "./CategorySelectionMenu";
 import {DialogActionBar} from "./DialogActionBar";
 import {DialogTitleAndDetails} from "./DialogTitleAndDetails";
+import {makeVar, useMutation, useQuery, useReactiveVar} from "@apollo/client";
+import {useSnackbar} from "notistack";
+import {
+  ADD_QUESTION,
+  AddQuestionResponse,
+  AddQuestionVariables,
+  EDIT_QUESTION,
+  EditQuestionResponse,
+  EditQuestionVariables
+} from "../backend/mutations/question";
+import {
+  BasicQuestionFragment,
+  BasicQuestionResponse,
+  GET_QUESTION_BY_ID,
+  GetQuestionByIdResponse,
+  GetQuestionByIdVariables
+} from "../backend/queries/question";
+import CategorySelectionMenu from "./CategorySelectionMenu";
+import {GET_ALL_CATEGORIES, GetAllCategoriesResponse} from "../backend/queries/category";

+export const dialogChangeQuestionId = makeVar<string>("");
+export const dialogChangeQuestionOpen = makeVar<boolean>(false);

-export interface ChangeQuestionDialogContent {
-  id: string
-  title: string,
-  details: string | null,
-  categoryId: number | null,
-}
+export default function DialogChangeQuestion() {
+  const [addMode, setAddMode] = useState(true);
+  const [title, setTitle] = useState("");
+  const [details, setDetails] = useState("");
+  const [categoryRowId, setCategoryRowId] = useState<number | null>(null);
+  const questionId = useReactiveVar(dialogChangeQuestionId);
+  const open = useReactiveVar(dialogChangeQuestionOpen);
+  const {enqueueSnackbar} = useSnackbar();
+  useQuery<GetQuestionByIdResponse, GetQuestionByIdVariables>(GET_QUESTION_BY_ID, {
+    variables: {
+      id: questionId,
+    },
+    onCompleted: (data => {
+      setAddMode(!data.question && !questionId)
+      setTitle(data.question?.title || "");
+      setDetails(data.question?.description || "");
+      setCategoryRowId(data.question?.categoryByCategoryRowId?.rowId || null)
+    })
+  })
+  const categories = useQuery<GetAllCategoriesResponse, null>(GET_ALL_CATEGORIES).data?.allCategories.nodes;

-interface DialogChangeQuestionProps {
-  title: string,
-  confirmButtonText: string,
-  open: boolean,
-  content: ChangeQuestionDialogContent,
-  loading: boolean,
-  categories?: Array<BasicCategoryResponse>,
+  const [editQuestion, {loading: editLoading}] = useMutation<EditQuestionResponse, EditQuestionVariables>(EDIT_QUESTION, {
+    onError: (e) => enqueueSnackbar(`Ein Fehler ist aufgetreten: ${e.message}`, {variant: "error"}),
+    onCompleted: (response) => {
+      if (response.updateQuestion) {
+        enqueueSnackbar("Frage erfolgreich geändert.", {variant: "success"})
+        dialogChangeQuestionOpen(false);
+      } else {
+        enqueueSnackbar("Ein Fehler ist aufgetreten, versuche es erneut.", {variant: "error"})
+      }
+    }
+  });
+  const [addQuestion, {loading: addLoading}] = useMutation<AddQuestionResponse, AddQuestionVariables>(ADD_QUESTION, {
+    onError: (e) => enqueueSnackbar(`Ein Fehler ist aufgetreten: ${e.message}`, {variant: "error"}),
+    onCompleted: (response) => {
+      if (response.createQuestion) {
+        enqueueSnackbar("Frage erfolgreich hinzugefügt.", {variant: "success"})
+        dialogChangeQuestionOpen(false);
+      } else {
+        enqueueSnackbar("Ein Fehler ist aufgetreten, versuche es erneut.", {variant: "error"})
+      }
+    },
+    update: (cache, {data}) => {
+      cache.modify({
+        fields: {
+          allQuestions(existingQuestions = {nodes: []}) {
+            const newQuestionRef = cache.writeFragment<BasicQuestionResponse | undefined>({
+              data: data?.createQuestion?.question,
+              fragment: BasicQuestionFragment,
+              fragmentName: "BasicQuestionFragment",
+            });
+            return {nodes: [...existingQuestions.nodes, newQuestionRef]};
+          }
+        }
+      });
+    }
+  });

-  handleContentChange(content: ChangeQuestionDialogContent): void
+  const handleConfirmButtonClick = () => {
+    if (addMode) {
+      addQuestion({
+        variables: {
+          title,
+          description: details,
+          categoryRowId: categoryRowId,
+        }
+      })
+    } else {
+      editQuestion({
+        variables: {
+          id: questionId,
+          title: title,
+          description: details,
+          categoryRowId: categoryRowId,
+        }
+      })
+    }
+  }

-  handleConfirmButtonClick(): void,
-
-  handleClose(): void,
-}
-
-
-export default function DialogChangeQuestion(props: DialogChangeQuestionProps) {
  return (
-    <Dialog open={props.open} onClose={props.handleClose} aria-labelledby="form-dialog-title">
-      <DialogTitle id="form-dialog-title">{props.title}</DialogTitle>
+    <Dialog open={open} onClose={() => dialogChangeQuestionOpen(false)} aria-labelledby="form-dialog-title">
+      <DialogTitle id="form-dialog-title">
+        {addMode ? "Neue Frage erstellen" : "Frage bearbeiten"}
+      </DialogTitle>
      <DialogContent>
        <DialogTitleAndDetails
-          title={props.content.title}
-          details={props.content.details}
-          onTitleChange={newTitle => props.handleContentChange({...props.content, title: newTitle})}
-          onDetailsChange={newDetails => props.handleContentChange({...props.content, details: newDetails})}
+          title={title}
+          details={details}
+          onTitleChange={newTitle => setTitle(newTitle)}
+          onDetailsChange={newDetails => setDetails(newDetails)}
        />
        <CategorySelectionMenu
-          selectedCategoryId={props.content.categoryId}
-          categories={props.categories}
-          handleCategoryChange={(categoryId) => props.handleContentChange({
-            ...props.content,
-            categoryId: categoryId,
-          })}
+          selectedCategoryId={categoryRowId}
+          categories={categories}
+          handleCategoryChange={(categoryId) => setCategoryRowId(categoryId)}
        />
      </DialogContent>
      <DialogActionBar
-        loading={props.loading}
-        onClose={props.handleClose}
-        onConfirmButtonClick={props.handleConfirmButtonClick}
-        confirmButtonText={props.confirmButtonText}
+        onClose={() => dialogChangeQuestionOpen(false)}
+        onConfirmButtonClick={handleConfirmButtonClick}
+        confirmButtonText={addMode ? "Erstellen" : "Speichern"}
+        loading={addMode ? addLoading : editLoading}
      />
    </Dialog>
  );
--- a/redaktions-app/src/components/QuestionList.tsx
+++ b/redaktions-app/src/components/QuestionList.tsx
@ -1,27 +1,16 @@
 import {Paper, Typography} from "@material-ui/core";
-import React, {useState} from "react";
+import React from "react";
 import {makeStyles} from "@material-ui/core/styles";
-import {useMutation, useQuery} from "@apollo/client";
+import {useQuery} from "@apollo/client";
 import AddCard from "./AddCard";
 import AccordionWithEdit from "./AccordionWithEdit";
-import {
-  BasicQuestionFragment,
-  BasicQuestionResponse,
-  GET_ALL_QUESTIONS,
-  GetAllQuestionsResponse
-} from "../backend/queries/question";
-import DialogChangeQuestion, {ChangeQuestionDialogContent} from "./DialogChangeQuestion";
-import {GET_ALL_CATEGORIES, GetAllCategoriesResponse} from "../backend/queries/category";
-import {
-  ADD_QUESTION,
-  AddQuestionResponse,
-  AddQuestionVariables,
-  EDIT_QUESTION,
-  EditQuestionResponse,
-  EditQuestionVariables
-} from "../backend/mutations/question";
-import {useSnackbar} from 'notistack';
-import DialogDeleteQuestion, {dialogDeleteQuestionId, dialogDeleteQuestionOpen, dialogDeleteQuestionTitle} from "./DialogDeleteQuestion";
+import {BasicQuestionResponse, GET_ALL_QUESTIONS, GetAllQuestionsResponse} from "../backend/queries/question";
+import DialogChangeQuestion, {dialogChangeQuestionId, dialogChangeQuestionOpen} from "./DialogChangeQuestion";
+import DialogDeleteQuestion, {
+  dialogDeleteQuestionId,
+  dialogDeleteQuestionOpen,
+  dialogDeleteQuestionTitle
+} from "./DialogDeleteQuestion";

 const useStyles = makeStyles((theme) => ({
  root: {
@ -31,82 +20,18 @@ const useStyles = makeStyles((theme) => ({
  },
 }));

-const emptyChangeQuestionDialog: ChangeQuestionDialogContent = {
-  id: "",
-  title: "",
-  details: "",
-  categoryId: null,
-}
-
 export default function QuestionList() {
-  const [changeDialogOpen, setChangeDialogOpen] = useState(false);
-  const [dialogTitle, setDialogTitle] = useState("");
-  const [dialogConfirmButtonText, setDialogConfirmButtonText] = useState("");
-  const [changeDialogContent, setChangeDialogContent] = useState(emptyChangeQuestionDialog);
-  const {enqueueSnackbar} = useSnackbar();
  const questions = useQuery<GetAllQuestionsResponse, null>(GET_ALL_QUESTIONS).data?.allQuestions.nodes;
-  const categories = useQuery<GetAllCategoriesResponse, null>(GET_ALL_CATEGORIES).data?.allCategories.nodes;
-  const [editQuestion, {loading: editLoading}] = useMutation<EditQuestionResponse, EditQuestionVariables>(EDIT_QUESTION, {
-    onError: (e) => enqueueSnackbar(`Ein Fehler ist aufgetreten: ${e.message}`, {variant: "error"}),
-    onCompleted: (response) => {
-      if (response.updateQuestion) {
-        enqueueSnackbar("Frage erfolgreich geändert.", {variant: "success"})
-        setChangeDialogOpen(false);
-      } else {
-        enqueueSnackbar("Ein Fehler ist aufgetreten, versuche es erneut.", {variant: "error"})
-      }
-    }
-  });
-  const [addQuestion, {loading: addLoading}] = useMutation<AddQuestionResponse, AddQuestionVariables>(ADD_QUESTION, {
-    onError: (e) => enqueueSnackbar(`Ein Fehler ist aufgetreten: ${e.message}`, {variant: "error"}),
-    onCompleted: (response) => {
-      if (response.createQuestion) {
-        enqueueSnackbar("Frage erfolgreich hinzugefügt.", {variant: "success"})
-        setChangeDialogOpen(false);
-      } else {
-        enqueueSnackbar("Ein Fehler ist aufgetreten, versuche es erneut.", {variant: "error"})
-      }
-    },
-    update: (cache, {data}) => {
-      cache.modify({
-        fields: {
-          allQuestions(existingQuestions = {nodes: []}) {
-            const newQuestionRef = cache.writeFragment<BasicQuestionResponse | undefined>({
-              data: data?.createQuestion?.question,
-              fragment: BasicQuestionFragment,
-              fragmentName: "BasicQuestionFragment",
-            });
-            return {nodes: [...existingQuestions.nodes, newQuestionRef]};
-          }
-        }
-      });
-    }
-  });
  const classes = useStyles();

-  const loading = editLoading || addLoading;
-
-  const handleAddClick = () => {
-    setDialogTitle("Neue Frage erstellen");
-    setDialogConfirmButtonText("Erstellen");
-    if (changeDialogContent.id !== "") {
-      setChangeDialogContent(emptyChangeQuestionDialog);
-    }
-    setChangeDialogOpen(true);
+  const handleAddButtonClick = () => {
+    dialogChangeQuestionId("")
+    dialogChangeQuestionOpen(true)
  }

  const handleEditButtonClick = (question: BasicQuestionResponse) => {
-    setDialogTitle("Frage bearbeiten");
-    setDialogConfirmButtonText("Speichern")
-    if (changeDialogContent.id !== question.id) {
-      setChangeDialogContent({
-        id: question.id,
-        title: question.title,
-        details: question.description,
-        categoryId: question.categoryByCategoryRowId ? question.categoryByCategoryRowId.rowId : null,
-      })
-    }
-    setChangeDialogOpen(true);
+    dialogChangeQuestionId(question.id)
+    dialogChangeQuestionOpen(true)
  };

  const handleDeleteButtonClick = (question: BasicQuestionResponse) => {
@ -115,31 +40,6 @@ export default function QuestionList() {
    dialogDeleteQuestionOpen(true);
  }

-  const handleDialogContentChange = (content: ChangeQuestionDialogContent) => {
-    setChangeDialogContent(content)
-  }
-
-  const handleChangeConfirmButtonClick = () => {
-    if (changeDialogContent.id !== "") {
-      editQuestion({
-        variables: {
-          id: changeDialogContent.id,
-          title: changeDialogContent.title,
-          description: changeDialogContent.details,
-          categoryRowId: changeDialogContent.categoryId,
-        }
-      })
-    } else {
-      addQuestion({
-        variables: {
-          title: changeDialogContent.title,
-          description: changeDialogContent.details,
-          categoryRowId: changeDialogContent.categoryId,
-        }
-      })
-    }
-  };
-
  return (
    <Paper className={classes.root}>
      <Typography component={"h2"} variant="h6" color="primary" gutterBottom>Fragen</Typography>
@ -152,18 +52,8 @@ export default function QuestionList() {
          onDeleteButtonClick={() => handleDeleteButtonClick(question)}
        />
      )}
-      <AddCard handleClick={handleAddClick}/>
-      <DialogChangeQuestion
-        title={dialogTitle}
-        confirmButtonText={dialogConfirmButtonText}
-        open={changeDialogOpen}
-        content={changeDialogContent}
-        categories={categories}
-        loading={loading}
-        handleContentChange={handleDialogContentChange}
-        handleConfirmButtonClick={handleChangeConfirmButtonClick}
-        handleClose={() => setChangeDialogOpen(false)}
-      />
+      <AddCard handleClick={handleAddButtonClick}/>
+      <DialogChangeQuestion/>
      <DialogDeleteQuestion/>
    </Paper>
  )
--- a/redaktions-app/src/integration-tests/question-list.integration.test.tsx
+++ b/redaktions-app/src/integration-tests/question-list.integration.test.tsx
@ -4,7 +4,7 @@ import {MockedProvider, MockedResponse} from '@apollo/client/testing';
 import {MemoryRouter} from 'react-router-dom';
 import QuestionList from "../components/QuestionList";
 import {SnackbarProvider} from "notistack";
-import {getAllQuestionsMock, questionNodesMock} from "../backend/queries/question.mock";
+import {getAllQuestionsMock, getQuestionByIdMock, questionNodesMock} from "../backend/queries/question.mock";
 import {getAllCategoriesMock} from "../backend/queries/category.mock";
 import {addQuestionMock, deleteQuestionMock, editQuestionMock} from "../backend/mutations/question.mock";
 import {
@ -125,7 +125,7 @@ describe('The QuestionList', () => {
 });

 function renderQuestionList(additionalMocks?: Array<MockedResponse>) {
-  const initialMocks = [...getAllQuestionsMock, ...getAllCategoriesMock];
+  const initialMocks = [...getAllQuestionsMock, ...getQuestionByIdMock, ...getAllCategoriesMock];
  const allMocks = additionalMocks ? [...initialMocks, ...additionalMocks] : initialMocks
  return render(
    <MockedProvider mocks={allMocks}>