Add validate fn stubs

Signed-off-by: Tyler Ohlsen <ohltyler@amazon.com>

Author: ohltyler

common/interfaces.ts

@@ -53,7 +53,7 @@ export type TemplateEdge = {
 export type TemplateFlow = {
   user_params?: Map<string, any>;
   nodes: TemplateNode[];
-  edges: TemplateEdge[];
+  edges?: TemplateEdge[];
 };
 
 export type TemplateFlows = {

common/utils.ts

@@ -12,6 +12,7 @@ import {
   generateId,
   ReactFlowEdge,
   TemplateFlows,
+  WorkflowTemplate,
 } from './';
 
 // TODO: implement this and remove hardcoded return values
@@ -68,3 +69,25 @@ export function toWorkspaceFlow(
     edges: [] as ReactFlowEdge[],
   };
 }
+
+// TODO: implement this
+/**
+ * Validates the UI workflow state.
+ * Note we don't have to validate connections since that is done via input/output handlers.
+ */
+export function validateWorkspaceFlow(
+  workspaceFlow: WorkspaceFlowState
+): boolean {
+  return true;
+}
+
+// TODO: implement this
+/**
+ * Validates the backend template. May be used when parsing persisted templates on server-side,
+ * or when importing/exporting on the UI.
+ */
+export function validateWorkflowTemplate(
+  workflowTemplate: WorkflowTemplate
+): boolean {
+  return true;
+}

public/pages/workflow_detail/utils/utils.ts

@@ -8,6 +8,7 @@ import {
   Workflow,
   ReactFlowComponent,
   toTemplateFlows,
+  validateWorkspaceFlow,
 } from '../../../../common';
 
 export function saveWorkflow(workflow: Workflow, rfInstance: any): void {
@@ -18,7 +19,7 @@ export function saveWorkflow(workflow: Workflow, rfInstance: any): void {
     nodes: processNodes(curFlowState.nodes),
   };
 
-  const isValid = validateFlowState(curFlowState);
+  const isValid = validateWorkspaceFlow(curFlowState);
   if (isValid) {
     const updatedWorkflow = {
       ...workflow,
@@ -35,13 +36,6 @@ export function saveWorkflow(workflow: Workflow, rfInstance: any): void {
   }
 }
 
-// TODO: implement this. Need more info on UX side to finalize what we need
-// to persist, what validation to do, etc.
-// Note we don't have to validate connections since that is done via input/output handlers.
-function validateFlowState(flowState: WorkspaceFlowState): boolean {
-  return true;
-}
-
 // Process the raw ReactFlow nodes to only persist the fields we need
 function processNodes(nodes: ReactFlowComponent[]): ReactFlowComponent[] {
   return nodes

server/routes/flow_framework_routes_service.ts

@@ -21,6 +21,7 @@ import {
   GET_WORKFLOW_STATE_NODE_API_PATH,
   SEARCH_WORKFLOWS_NODE_API_PATH,
   WorkflowTemplate,
+  validateWorkflowTemplate,
 } from '../../common';
 import { generateCustomError, getWorkflowsFromResponses } from './helpers';
 
@@ -226,7 +227,9 @@ export class FlowFrameworkRoutesService {
         const workflowTemplate = JSON.parse(
           templateData.toString()
         ) as WorkflowTemplate;
-        workflowTemplates.push(workflowTemplate);
+        if (validateWorkflowTemplate(workflowTemplate)) {
+          workflowTemplates.push(workflowTemplate);
+        }
       });
 
       return res.ok({ body: { workflowTemplates } });