@oneuptime/common 14.0.0 → 14.0.2

package/Tests/App/Dashboard/CreateWorkflowModal.test.tsx

@@ -52,10 +52,14 @@ import CreateWorkflowModal from "../../../../App/FeatureSet/Dashboard/src/Compon
 import Workflow from "../../../Models/DatabaseModels/Workflow";
 import WorkflowVariable from "../../../Models/DatabaseModels/WorkflowVariable";
 import ObjectID from "../../../Types/ObjectID";
+import { JSONObject } from "../../../Types/JSON";
 import {
   WorkflowTemplate,
+  WorkflowTemplateCategories,
+  WorkflowTemplateCategory,
   WorkflowTemplateVariable,
   getWorkflowTemplate,
+  getWorkflowTemplatesByCategory,
 } from "../../../Types/Workflow/Templates";
 import ModelAPI from "../../../UI/Utils/ModelAPI/ModelAPI";
 import ProjectUtil from "../../../UI/Utils/Project";
@@ -72,6 +76,95 @@ const SLACK_TEMPLATE_ID: string = "incident-created-slack";
 const EMAIL_TEMPLATE_ID: string = "scheduled-email-digest";
 const ZERO_CONFIG_TEMPLATE_ID: string = "manual-log";
 
+/*
+ * Every Jira template comes in an incident and an alert version. Alerts have
+ * no public notes, so the alert set has no public-note template and is one
+ * shorter.
+ */
+interface JiraKind {
+  /** The record, as the template ids and help text name it. */
+  noun: string;
+  /** The other kind's record, which nothing of this kind should mention. */
+  otherNoun: string;
+  /** In the order the picker shows them. */
+  templateIds: Array<string>;
+  createIssueTemplateId: string;
+  /** Calls Jira, so asks for the site URL and the token and nothing else. */
+  createFromIssueTemplateId: string;
+  /** Receive a Jira webhook and write to OneUptime only: nothing to configure. */
+  webhookOnlyTemplateIds: Array<string>;
+}
+
+const INCIDENT_JIRA_KIND: JiraKind = {
+  noun: "incident",
+  otherNoun: "alert",
+  templateIds: [
+    "jira-create-issue-for-incident",
+    "jira-transition-issue-on-incident-state",
+    "jira-comment-from-incident-private-note",
+    "jira-comment-from-incident-public-note",
+    "jira-comment-on-incident-update",
+    "jira-declare-incident-from-issue",
+    "jira-status-to-incident-state",
+    "jira-comment-to-incident-private-note",
+    "jira-issue-changes-to-incident-private-note",
+  ],
+  createIssueTemplateId: "jira-create-issue-for-incident",
+  createFromIssueTemplateId: "jira-declare-incident-from-issue",
+  webhookOnlyTemplateIds: [
+    "jira-status-to-incident-state",
+    "jira-issue-changes-to-incident-private-note",
+  ],
+};
+
+const ALERT_JIRA_KIND: JiraKind = {
+  noun: "alert",
+  otherNoun: "incident",
+  templateIds: [
+    "jira-create-issue-for-alert",
+    "jira-transition-issue-on-alert-state",
+    "jira-comment-from-alert-private-note",
+    "jira-comment-on-alert-update",
+    "jira-create-alert-from-issue",
+    "jira-status-to-alert-state",
+    "jira-comment-to-alert-private-note",
+    "jira-issue-changes-to-alert-private-note",
+  ],
+  createIssueTemplateId: "jira-create-issue-for-alert",
+  createFromIssueTemplateId: "jira-create-alert-from-issue",
+  webhookOnlyTemplateIds: [
+    "jira-status-to-alert-state",
+    "jira-issue-changes-to-alert-private-note",
+  ],
+};
+
+const JIRA_KINDS: Array<JiraKind> = [INCIDENT_JIRA_KIND, ALERT_JIRA_KIND];
+
+/** Every Jira template, in the order the picker shows them: incidents, then alerts. */
+const JIRA_TEMPLATE_IDS: Array<string> = [
+  ...INCIDENT_JIRA_KIND.templateIds,
+  ...ALERT_JIRA_KIND.templateIds,
+];
+
+const JIRA_WEBHOOK_ONLY_TEMPLATE_IDS: Array<string> = [
+  ...INCIDENT_JIRA_KIND.webhookOnlyTemplateIds,
+  ...ALERT_JIRA_KIND.webhookOnlyTemplateIds,
+];
+
+const JIRA_TOKEN_VARIABLE: string = "jiraBasicAuthToken";
+
+/** The one setting whose help text names the record: it is what the issue links back to. */
+const ONEUPTIME_URL_VARIABLE: string = "oneuptimeUrl";
+
+/** What someone setting the Jira templates up would type, in the order the wizard asks. */
+const JIRA_VALUES: Record<string, string> = {
+  jiraBaseUrl: "https://acme.atlassian.net",
+  jiraBasicAuthToken: "cHJpeWFAYWNtZS5jb206QVRBVFQzeEZmR0YwUzNjcjN0",
+  jiraProjectKey: "OPS",
+  jiraIssueType: "Task",
+  oneuptimeUrl: "https://oneuptime.com",
+};
+
 interface ModelCreateArguments {
   model: Workflow | WorkflowVariable;
   modelType: typeof Workflow | typeof WorkflowVariable;
@@ -211,6 +304,93 @@ const createdWorkflow: CreatedWorkflowFunction = (): Workflow => {
   return workflow;
 };
 
+type SearchTemplatesFunction = (query: string) => void;
+
+const searchTemplates: SearchTemplatesFunction = (query: string): void => {
+  fireEvent.change(screen.getByTestId("workflow-template-search"), {
+    target: { value: query },
+  });
+};
+
+type GetStepContentFunction = () => HTMLElement;
+
+const getStepContent: GetStepContentFunction = (): HTMLElement => {
+  return screen.getByTestId("workflow-wizard-step-content");
+};
+
+const TEMPLATE_CARD_TEST_ID_PREFIX: string = "workflow-template-card-";
+const VARIABLE_INPUT_TEST_ID_PREFIX: string = "workflow-variable-";
+
+type TestIdSuffixesFunction = (
+  container: HTMLElement,
+  prefix: string,
+) => Array<string>;
+
+/** What follows the prefix on each matching test id, in document order. */
+const testIdSuffixes: TestIdSuffixesFunction = (
+  container: HTMLElement,
+  prefix: string,
+): Array<string> => {
+  return within(container)
+    .queryAllByTestId((testId: string) => {
+      return testId.startsWith(prefix);
+    })
+    .map((element: HTMLElement): string => {
+      return (element.getAttribute("data-testid") || "").slice(prefix.length);
+    });
+};
+
+type SectionHeadingsFunction = () => Array<string>;
+
+/** The section headings of the pick step, in the order they show. */
+const sectionHeadings: SectionHeadingsFunction = (): Array<string> => {
+  return within(getStepContent())
+    .queryAllByRole("heading")
+    .map((heading: HTMLElement): string => {
+      return heading.textContent || "";
+    });
+};
+
+type SectionOfFunction = (heading: string) => HTMLElement;
+
+/** A pick-step section: its heading and the cards under it. */
+const sectionOf: SectionOfFunction = (heading: string): HTMLElement => {
+  return within(getStepContent()).getByRole("heading", { name: heading })
+    .parentElement as HTMLElement;
+};
+
+type TemplateNamesFunction = (templateIds: Array<string>) => Array<string>;
+
+const templateNames: TemplateNamesFunction = (
+  templateIds: Array<string>,
+): Array<string> => {
+  return templateIds.map((templateId: string): string => {
+    return getTemplate(templateId).name;
+  });
+};
+
+type ArgumentsOfComponentFunction = (
+  graph: JSONObject,
+  componentId: string,
+) => JSONObject;
+
+const argumentsOfComponent: ArgumentsOfComponentFunction = (
+  graph: JSONObject,
+  componentId: string,
+): JSONObject => {
+  const node: JSONObject | undefined = (
+    graph["nodes"] as Array<JSONObject>
+  ).find((candidate: JSONObject) => {
+    return (candidate["data"] as JSONObject)["id"] === componentId;
+  });
+
+  if (!node) {
+    throw new Error(`The graph has no component "${componentId}".`);
+  }
+
+  return (node["data"] as JSONObject)["arguments"] as JSONObject;
+};
+
 beforeEach(() => {
   jest.clearAllMocks();
   mockGetCurrentProjectId.mockReturnValue(PROJECT_ID);
@@ -559,3 +739,476 @@ describe("CreateWorkflowModal creation orchestration", () => {
     );
   });
 });
+
+type CardNamesInFunction = (heading: string) => Array<string>;
+
+/** The template names on the cards under one heading, in the order they show. */
+const cardNamesIn: CardNamesInFunction = (heading: string): Array<string> => {
+  return testIdSuffixes(sectionOf(heading), TEMPLATE_CARD_TEST_ID_PREFIX);
+};
+
+type AcceptEveryCreateFunction = (created: Workflow) => void;
+
+/** The API accepts the workflow and every variable row the wizard sends after it. */
+const acceptEveryCreate: AcceptEveryCreateFunction = (
+  created: Workflow,
+): void => {
+  mockCreate.mockImplementation(
+    async (
+      data: ModelCreateArguments,
+    ): Promise<{ data: Workflow | WorkflowVariable }> => {
+      if (data.modelType === Workflow) {
+        return { data: created };
+      }
+
+      return { data: data.model };
+    },
+  );
+};
+
+type CreateCallsFunction = () => Array<ModelCreateArguments>;
+
+/** Everything the wizard sent to the API to create, in the order it sent it. */
+const createCalls: CreateCallsFunction = (): Array<ModelCreateArguments> => {
+  return mockCreate.mock.calls.map((call: Array<unknown>) => {
+    return call[0] as ModelCreateArguments;
+  });
+};
+
+interface WrittenRow {
+  name: string | undefined;
+  content: string | undefined;
+  isSecret: boolean;
+  workflowId: string | undefined;
+  projectId: string | undefined;
+}
+
+type WrittenRowsFunction = (
+  calls: Array<ModelCreateArguments>,
+) => Array<WrittenRow>;
+
+/** The variable rows among those calls, as the API received them. */
+const writtenRows: WrittenRowsFunction = (
+  calls: Array<ModelCreateArguments>,
+): Array<WrittenRow> => {
+  return calls.map((data: ModelCreateArguments): WrittenRow => {
+    expect(data.modelType).toBe(WorkflowVariable);
+
+    const variable: WorkflowVariable = data.model as WorkflowVariable;
+
+    return {
+      name: variable.name,
+      content: variable.content,
+      isSecret: (variable as unknown as { isSecret: boolean }).isSecret,
+      workflowId: variable.workflowId?.toString(),
+      projectId: variable.projectId?.toString(),
+    };
+  });
+};
+
+type ExpectedRowFunction = (name: string) => WrittenRow;
+
+/** The row a Jira setting should become: only the token is secret. */
+const expectedJiraRow: ExpectedRowFunction = (name: string): WrittenRow => {
+  return {
+    name: name,
+    content: JIRA_VALUES[name],
+    isSecret: name === JIRA_TOKEN_VARIABLE,
+    workflowId: WORKFLOW_ID.toString(),
+    projectId: PROJECT_ID.toString(),
+  };
+};
+
+describe("CreateWorkflowModal Jira templates", () => {
+  test("the picker has a Jira section holding the seventeen Jira templates, the incident ones first", () => {
+    renderModal();
+
+    expect(JIRA_TEMPLATE_IDS).toHaveLength(17);
+    expect(cardNamesIn(WorkflowTemplateCategory.Jira)).toEqual(
+      templateNames(JIRA_TEMPLATE_IDS),
+    );
+  });
+
+  /*
+   * The picker walks WorkflowTemplateCategories, not the templates, so a
+   * category missing from that list would hide all of its templates with no
+   * error anywhere.
+   */
+  test("sections follow the declared category order, Jira included", () => {
+    renderModal();
+
+    expect(sectionHeadings()).toEqual([
+      "Blank",
+      ...WorkflowTemplateCategories.filter(
+        (category: WorkflowTemplateCategory) => {
+          return getWorkflowTemplatesByCategory(category).length > 0;
+        },
+      ),
+    ]);
+    expect(sectionHeadings()).toContain(WorkflowTemplateCategory.Jira);
+  });
+
+  /*
+   * Search reads the name, description, teaches line and category, trimmed
+   * and case-insensitively. The category alone brings every Jira template
+   * back, and nothing outside Jira mentions it, so the result is exactly the
+   * Jira section — with no "Start from scratch" card above it.
+   */
+  test("searching for jira shows exactly the seventeen Jira templates", () => {
+    renderModal();
+
+    for (const query of ["jira", "JIRA", "  Jira  "]) {
+      searchTemplates(query);
+
+      expect({
+        query: query,
+        headings: sectionHeadings(),
+        cards: testIdSuffixes(getStepContent(), TEMPLATE_CARD_TEST_ID_PREFIX),
+      }).toEqual({
+        query: query,
+        headings: [WorkflowTemplateCategory.Jira],
+        cards: templateNames(JIRA_TEMPLATE_IDS),
+      });
+    }
+  });
+
+  /*
+   * Someone looking for what OneUptime can do with alerts finds the Jira
+   * versions next to the alert templates they already know, under their own
+   * heading. The incident Jira templates never mention alerts, so they stay
+   * out of it.
+   */
+  test("searching for alert shows the alert Jira templates under Jira, after the Alerts templates", () => {
+    renderModal();
+
+    const alertCategoryTemplateIds: Array<string> =
+      getWorkflowTemplatesByCategory(WorkflowTemplateCategory.Alerts).map(
+        (template: WorkflowTemplate): string => {
+          return template.id;
+        },
+      );
+
+    expect(alertCategoryTemplateIds.length).toBeGreaterThan(0);
+
+    for (const query of ["alert", "ALERT", "  Alert  "]) {
+      searchTemplates(query);
+
+      expect({
+        query: query,
+        headings: sectionHeadings(),
+        alerts: cardNamesIn(WorkflowTemplateCategory.Alerts),
+        jira: cardNamesIn(WorkflowTemplateCategory.Jira),
+      }).toEqual({
+        query: query,
+        headings: [
+          WorkflowTemplateCategory.Alerts,
+          WorkflowTemplateCategory.Jira,
+        ],
+        alerts: templateNames(alertCategoryTemplateIds),
+        jira: templateNames(ALERT_JIRA_KIND.templateIds),
+      });
+    }
+  });
+
+  /*
+   * Each kind's text names only its own record, so searching for one never
+   * turns up the other's Jira templates — someone after the alert versions
+   * does not have to pick them out from among the incident ones.
+   */
+  test.each(JIRA_KINDS)(
+    "searching for $noun leaves only the $noun templates in the Jira section",
+    (kind: JiraKind) => {
+      renderModal();
+      searchTemplates(kind.noun);
+
+      expect(cardNamesIn(WorkflowTemplateCategory.Jira)).toEqual(
+        templateNames(kind.templateIds),
+      );
+    },
+  );
+
+  /*
+   * The badge is how someone can tell, before choosing, which Jira templates
+   * need a Jira API token. The four webhook-only ones need nothing at all.
+   */
+  test("each Jira card says how many settings it needs", () => {
+    renderModal();
+
+    const expectedBadges: Record<string, string | null> = {
+      "jira-create-issue-for-incident": "Needs 5 settings",
+      "jira-transition-issue-on-incident-state": "Needs 2 settings",
+      "jira-comment-from-incident-private-note": "Needs 2 settings",
+      "jira-comment-from-incident-public-note": "Needs 2 settings",
+      "jira-comment-on-incident-update": "Needs 2 settings",
+      "jira-declare-incident-from-issue": "Needs 2 settings",
+      "jira-status-to-incident-state": null,
+      "jira-comment-to-incident-private-note": "Needs 2 settings",
+      "jira-issue-changes-to-incident-private-note": null,
+      "jira-create-issue-for-alert": "Needs 5 settings",
+      "jira-transition-issue-on-alert-state": "Needs 2 settings",
+      "jira-comment-from-alert-private-note": "Needs 2 settings",
+      "jira-comment-on-alert-update": "Needs 2 settings",
+      "jira-create-alert-from-issue": "Needs 2 settings",
+      "jira-status-to-alert-state": null,
+      "jira-comment-to-alert-private-note": "Needs 2 settings",
+      "jira-issue-changes-to-alert-private-note": null,
+    };
+
+    expect(Object.keys(expectedBadges)).toEqual(JIRA_TEMPLATE_IDS);
+
+    for (const templateId of JIRA_TEMPLATE_IDS) {
+      const card: HTMLElement = screen.getByTestId(
+        `${TEMPLATE_CARD_TEST_ID_PREFIX}${getTemplate(templateId).name}`,
+      );
+
+      expect({
+        template: templateId,
+        badge: within(card).queryByText(/^Needs \d+ settings?$/)?.textContent,
+      }).toEqual({
+        template: templateId,
+        badge: expectedBadges[templateId] || undefined,
+      });
+    }
+  });
+
+  describe.each(JIRA_KINDS)(
+    "the $noun templates in the wizard",
+    (kind: JiraKind) => {
+      test("the create-issue template asks for its five settings and masks only the token", () => {
+        renderModal();
+        const template: WorkflowTemplate = goToConfigure(
+          kind.createIssueTemplateId,
+        );
+
+        expect(
+          testIdSuffixes(getStepContent(), VARIABLE_INPUT_TEST_ID_PREFIX),
+        ).toEqual(Object.keys(JIRA_VALUES));
+
+        const token: HTMLInputElement = getVariableInput(JIRA_TOKEN_VARIABLE);
+
+        expect(token).toHaveAttribute("type", "password");
+        expect(token).toHaveAttribute("autocomplete", "new-password");
+        expect(token).toHaveAttribute("spellcheck", "false");
+
+        /*
+         * The site URL is not a secret, and masking it would hide the typo that
+         * sends every request to the wrong host.
+         */
+        const baseUrl: HTMLInputElement = getVariableInput("jiraBaseUrl");
+
+        expect(baseUrl).toHaveAttribute("type", "text");
+        expect(baseUrl).not.toHaveAttribute("autocomplete");
+        expect(baseUrl).not.toHaveAttribute("spellcheck", "false");
+
+        for (const variable of template.variables) {
+          expect({
+            variable: variable.name,
+            type: getVariableInput(variable.name).getAttribute("type"),
+          }).toEqual({
+            variable: variable.name,
+            type: variable.name === JIRA_TOKEN_VARIABLE ? "password" : "text",
+          });
+        }
+      });
+
+      /*
+       * The OneUptime URL is what the new issue links back to, so its help text
+       * is the one place the wizard names the record. Someone setting up the
+       * alert version should not be told the link goes to an incident.
+       */
+      test(`the create-issue template's OneUptime URL help text says the issue links back to the ${kind.noun}`, () => {
+        renderModal();
+        const template: WorkflowTemplate = goToConfigure(
+          kind.createIssueTemplateId,
+        );
+
+        const variable: WorkflowTemplateVariable | undefined =
+          template.variables.find((candidate: WorkflowTemplateVariable) => {
+            return candidate.name === ONEUPTIME_URL_VARIABLE;
+          });
+
+        if (!variable) {
+          throw new Error(
+            `"${kind.createIssueTemplateId}" does not ask for the OneUptime URL.`,
+          );
+        }
+
+        const helpText: HTMLElement = within(getStepContent()).getByText(
+          variable.description,
+        );
+
+        /* Shown beside the OneUptime URL field, not beside some other one. */
+        expect(helpText.parentElement).toContainElement(
+          getVariableInput(ONEUPTIME_URL_VARIABLE),
+        );
+        expect(helpText).toHaveTextContent(
+          `link the Jira issue back to the ${kind.noun}`,
+        );
+        expect(helpText.textContent).not.toContain(kind.otherNoun);
+        expect(
+          within(getStepContent()).queryByText(
+            new RegExp(`back to the ${kind.otherNoun}`),
+          ),
+        ).not.toBeInTheDocument();
+      });
+
+      test("the create-issue template reports every missing setting and creates nothing", () => {
+        renderModal();
+        const template: WorkflowTemplate = goToConfigure(
+          kind.createIssueTemplateId,
+        );
+
+        submit();
+
+        expect(template.variables).toHaveLength(5);
+
+        for (const variable of template.variables) {
+          expect(
+            screen.getByText(`${variable.title} is required.`),
+          ).toBeInTheDocument();
+        }
+
+        expect(mockCreate).not.toHaveBeenCalled();
+      });
+
+      test("creating from the create-issue template writes a disabled workflow, then its five settings with only the token secret", async () => {
+        const created: Workflow = createdWorkflow();
+
+        acceptEveryCreate(created);
+
+        const harness: ModalHarness = renderModal();
+        const template: WorkflowTemplate = goToConfigure(
+          kind.createIssueTemplateId,
+        );
+
+        for (const [name, value] of Object.entries(JIRA_VALUES)) {
+          fillVariable(name, value);
+        }
+
+        submit();
+
+        await waitFor(() => {
+          expect(harness.onCreated).toHaveBeenCalledWith(created);
+        });
+
+        const calls: Array<ModelCreateArguments> = createCalls();
+
+        expect(calls).toHaveLength(6);
+        expect(calls[0]?.modelType).toBe(Workflow);
+
+        /*
+         * The workflow goes out switched off, under the name the template
+         * suggests — which differs between the two kinds, because a project
+         * cannot hold two workflows of the same name — and carrying only the
+         * reference: the token itself is written to the secret variable row
+         * and nowhere else.
+         */
+        const workflow: Workflow = calls[0]?.model as Workflow;
+        const graph: JSONObject = workflow.graph as JSONObject;
+
+        expect(workflow.isEnabled).toBe(false);
+        expect(workflow.name).toBe(template.workflowName);
+        expect(workflow.name).toContain(kind.noun);
+        expect(
+          argumentsOfComponent(graph, "create-issue-1")["request-headers"],
+        ).toEqual({
+          Authorization: "Basic {{local.variables.jiraBasicAuthToken}}",
+        });
+        expect(JSON.stringify(graph)).not.toContain(
+          JIRA_VALUES[JIRA_TOKEN_VARIABLE],
+        );
+
+        expect(writtenRows(calls.slice(1))).toEqual(
+          Object.keys(JIRA_VALUES).map(expectedJiraRow),
+        );
+        expect(mockDeleteItem).not.toHaveBeenCalled();
+      });
+
+      /*
+       * The other Jira-calling templates ask only where Jira is and as whom.
+       * The token is just as much a credential there, so it is written secret
+       * whichever kind of record the workflow is for.
+       */
+      test("creating from the create-from-issue template writes the site URL and a secret token", async () => {
+        const created: Workflow = createdWorkflow();
+
+        acceptEveryCreate(created);
+
+        const harness: ModalHarness = renderModal();
+        goToConfigure(kind.createFromIssueTemplateId);
+
+        expect(
+          testIdSuffixes(getStepContent(), VARIABLE_INPUT_TEST_ID_PREFIX),
+        ).toEqual(["jiraBaseUrl", JIRA_TOKEN_VARIABLE]);
+        expect(getVariableInput(JIRA_TOKEN_VARIABLE)).toHaveAttribute(
+          "type",
+          "password",
+        );
+
+        fillVariable("jiraBaseUrl", JIRA_VALUES["jiraBaseUrl"] as string);
+        fillVariable(
+          JIRA_TOKEN_VARIABLE,
+          JIRA_VALUES[JIRA_TOKEN_VARIABLE] as string,
+        );
+
+        submit();
+
+        await waitFor(() => {
+          expect(harness.onCreated).toHaveBeenCalledWith(created);
+        });
+
+        const calls: Array<ModelCreateArguments> = createCalls();
+
+        expect(calls[0]?.modelType).toBe(Workflow);
+        expect((calls[0]?.model as Workflow).isEnabled).toBe(false);
+        expect(writtenRows(calls.slice(1))).toEqual([
+          expectedJiraRow("jiraBaseUrl"),
+          expectedJiraRow(JIRA_TOKEN_VARIABLE),
+        ]);
+        expect(mockDeleteItem).not.toHaveBeenCalled();
+      });
+    },
+  );
+
+  /*
+   * A template that declares nothing must not show an empty Configure step,
+   * and must not write any variable rows: the webhook URL it needs is the
+   * one the created workflow's trigger shows.
+   */
+  test.each(JIRA_WEBHOOK_ONLY_TEMPLATE_IDS)(
+    "the webhook-only %s skips Configure and creates only the workflow",
+    async (templateId: string) => {
+      const created: Workflow = createdWorkflow();
+
+      mockCreate.mockResolvedValue({ data: created });
+
+      const harness: ModalHarness = renderModal();
+      const template: WorkflowTemplate = selectTemplate(templateId);
+
+      expect(template.variables).toEqual([]);
+      expectActiveStep("Name");
+      expect(
+        within(getProgress()).queryByText("Configure"),
+      ).not.toBeInTheDocument();
+      expect(
+        screen.getByTestId("modal-footer-submit-button"),
+      ).toHaveTextContent("Create Workflow");
+
+      submit();
+
+      await waitFor(() => {
+        expect(harness.onCreated).toHaveBeenCalledWith(created);
+      });
+
+      expect(mockCreate).toHaveBeenCalledTimes(1);
+
+      const data: ModelCreateArguments = mockCreate.mock
+        .calls[0]?.[0] as ModelCreateArguments;
+      const workflow: Workflow = data.model as Workflow;
+
+      expect(data.modelType).toBe(Workflow);
+      expect(workflow.name).toBe(template.workflowName);
+      expect(workflow.isEnabled).toBe(false);
+    },
+  );
+});