From c3710aeb2ac85bfd87b99e693dd599a42cf46a5a Mon Sep 17 00:00:00 2001
From: jasquat <jasquat@users.noreply.github.com>
Date: Tue, 23 May 2023 15:15:30 -0400
Subject: [PATCH] allow running all process model unit tests from the show page
 w/ burnettk

---
 .../src/components/ProcessModelTestRun.tsx    | 56 ++++++++++++++-----
 .../src/routes/ProcessModelShow.tsx           | 19 ++++++-
 2 files changed, 60 insertions(+), 15 deletions(-)

diff --git a/spiffworkflow-frontend/src/components/ProcessModelTestRun.tsx b/spiffworkflow-frontend/src/components/ProcessModelTestRun.tsx
index fe82167e3..9938431a0 100644
--- a/spiffworkflow-frontend/src/components/ProcessModelTestRun.tsx
+++ b/spiffworkflow-frontend/src/components/ProcessModelTestRun.tsx
@@ -10,10 +10,14 @@ import {
 } from './ErrorDisplay';
 
 type OwnProps = {
-  processModelFile: ProcessFile;
+  processModelFile?: ProcessFile;
+  buttonType?: string;
 };
 
-export default function ProcessModelTestRun({ processModelFile }: OwnProps) {
+export default function ProcessModelTestRun({
+  processModelFile,
+  buttonType = 'icon',
+}: OwnProps) {
   const [testCaseResults, setTestCaseResults] =
     useState<TestCaseResults | null>(null);
   const [showTestCaseResultsModal, setShowTestCaseResultsModal] =
@@ -32,7 +36,7 @@ export default function ProcessModelTestRun({ processModelFile }: OwnProps) {
             kind="ghost"
             className="green-icon"
             renderIcon={Checkmark}
-            iconDescription="PASS"
+            iconDescription="All BPMN unit tests passed"
             hasIconOnly
             size="lg"
             onClick={() => setShowTestCaseResultsModal(true)}
@@ -44,7 +48,7 @@ export default function ProcessModelTestRun({ processModelFile }: OwnProps) {
           kind="ghost"
           className="red-icon"
           renderIcon={Close}
-          iconDescription="FAILS"
+          iconDescription="BPMN unit tests failed"
           hasIconOnly
           size="lg"
           onClick={() => setShowTestCaseResultsModal(true)}
@@ -54,12 +58,17 @@ export default function ProcessModelTestRun({ processModelFile }: OwnProps) {
     return null;
   };
 
-  const onProcessModelTestRun = (fileName: string) => {
+  const onProcessModelTestRun = () => {
     const httpMethod = 'POST';
     setTestCaseResults(null);
 
+    let queryParams = '';
+    if (processModelFile) {
+      queryParams = `?test_case_file=${processModelFile.name}`;
+    }
+
     HttpService.makeCallToBackend({
-      path: `${targetUris.processModelTestsPath}?test_case_file=${fileName}`,
+      path: `${targetUris.processModelTestsPath}${queryParams}`,
       successCallback: onProcessModelTestRunSuccess,
       httpMethod,
     });
@@ -143,17 +152,36 @@ export default function ProcessModelTestRun({ processModelFile }: OwnProps) {
     );
   };
 
+  const buttonElement = () => {
+    if (buttonType === 'icon') {
+      return (
+        <Button
+          kind="ghost"
+          renderIcon={PlayOutline}
+          iconDescription="Run BPMN unit tests defined in this file"
+          hasIconOnly
+          size="lg"
+          onClick={() => onProcessModelTestRun()}
+        />
+      );
+    }
+    if (buttonType === 'text') {
+      return (
+        <Button
+          onClick={() => onProcessModelTestRun()}
+          title="Run all BPMN unit tests for this process model"
+        >
+          Run Unit Tests
+        </Button>
+      );
+    }
+    return null;
+  };
+
   return (
     <>
       {testCaseResultsModal()}
-      <Button
-        kind="ghost"
-        renderIcon={PlayOutline}
-        iconDescription="Run Test"
-        hasIconOnly
-        size="lg"
-        onClick={() => onProcessModelTestRun(processModelFile.name)}
-      />
+      {buttonElement()}
       {processModelTestRunResultTag()}
     </>
   );
diff --git a/spiffworkflow-frontend/src/routes/ProcessModelShow.tsx b/spiffworkflow-frontend/src/routes/ProcessModelShow.tsx
index 14281ab1f..407c81151 100644
--- a/spiffworkflow-frontend/src/routes/ProcessModelShow.tsx
+++ b/spiffworkflow-frontend/src/routes/ProcessModelShow.tsx
@@ -83,6 +83,18 @@ export default function ProcessModelShow() {
     `${params.process_model_id}`
   );
 
+  let hasTestCaseFiles: boolean = false;
+
+  const isTestCaseFile = (processModelFile: ProcessFile) => {
+    return processModelFile.name.match(/^test_.*\.json$/);
+  };
+
+  if (processModel) {
+    hasTestCaseFiles = !!processModel.files.find(
+      (processModelFile: ProcessFile) => isTestCaseFile(processModelFile)
+    );
+  }
+
   useEffect(() => {
     const processResult = (result: ProcessModel) => {
       setProcessModel(result);
@@ -310,7 +322,7 @@ export default function ProcessModelShow() {
         </Can>
       );
     }
-    if (processModelFile.name.match(/^test_.*\.json$/)) {
+    if (isTestCaseFile(processModelFile)) {
       elements.push(
         <Can I="POST" a={targetUris.processModelTestsPath} ability={ability}>
           <ProcessModelTestRun processModelFile={processModelFile} />
@@ -656,6 +668,11 @@ export default function ProcessModelShow() {
               Publish Changes
             </Button>
           </Can>
+          <Can I="POST" a={targetUris.processModelTestsPath} ability={ability}>
+            {hasTestCaseFiles ? (
+              <ProcessModelTestRun buttonType="text" />
+            ) : null}
+          </Can>
         </Stack>
         {processModelFilesSection()}
         <Can I="GET" a={targetUris.processInstanceListPath} ability={ability}>