Merge pull request #126 from FlowTestAI/update-canvas-store

fix: update canvas store correctly to render react flow with latest changes

Author: jsajal

packages/flowtest-cli/graph/Graph.js

@@ -206,15 +206,21 @@ class Graph {
       }
     } catch (err) {
       console.log(chalk.red(`Flow failed at: ${JSON.stringify(node.data)} due to ${err}`));
-      this.logger.add(LogLevel.ERROR, `Flow failed due to ${err}`, node);
+      this.logger.add(LogLevel.ERROR, `Flow failed due to ${err}`, {
+        type: 'errorNode',
+        data: node.data,
+      });
       return {
         status: 'Failed',
       };
     }
 
     if (result === undefined) {
       console.log(chalk.red(`Flow failed due to failure to evaluate result at node: ${node.data}`));
-      this.logger.add(LogLevel.ERROR, 'Flow failed due to failure to evaluate result', node);
+      this.logger.add(LogLevel.ERROR, 'Flow failed due to failure to evaluate result', {
+        type: 'errorNode',
+        data: node.data,
+      });
       return {
         status: 'Failed',
       };

src/components/atoms/Editor.js

@@ -46,7 +46,7 @@ const hideScrollbar = EditorView.theme({
 export const Editor = ({ ...props }) => {
   const editor = useRef();
   const [view, setView] = useState(null);
-  const [dynamicOptions, setDynamicOptions] = useState([]);
+  const [dynamicOptions, setDynamicOptions] = useState(props.completionOptions || []);
 
   if (view) {
     if (!isEqual(dynamicOptions, props.completionOptions)) {

src/components/atoms/common/TextEditor.js

@@ -100,7 +100,7 @@ const highlightStyle = EditorView.baseTheme({
 export const TextEditor = ({ placeHolder, onChangeHandler, value, disableState, completionOptions, styles }) => {
   const editor1 = useRef();
   const [view, setView] = useState(null);
-  const [dynamicOptions, setDynamicOptions] = useState([]);
+  const [dynamicOptions, setDynamicOptions] = useState(completionOptions);
 
   if (view) {
     if (!isEqual(dynamicOptions, completionOptions)) {

src/components/molecules/flow/graph/Graph.js

@@ -183,14 +183,20 @@ class Graph {
         throw Error(`Timeout of ${this.timeout} ms exceeded, stopping graph run`);
       }
     } catch (err) {
-      this.logger.add(LogLevel.ERROR, `Flow failed due to ${err}`, node);
+      this.logger.add(LogLevel.ERROR, `Flow failed due to ${err}`, {
+        type: 'errorNode',
+        data: node.data,
+      });
       return {
         status: 'Failed',
       };
     }
 
     if (result === undefined) {
-      this.logger.add(LogLevel.ERROR, 'Flow failed due to failure to evaluate result', node);
+      this.logger.add(LogLevel.ERROR, 'Flow failed due to failure to evaluate result', {
+        type: 'errorNode',
+        data: node.data,
+      });
       return {
         status: 'Failed',
       };

src/components/molecules/flow/graph/compute/requestnode.js

@@ -32,7 +32,7 @@ class requestNode extends Node {
 
     const res = await this.runHttpRequest(options);
 
-    if (this.nodeData.requestBody.type === 'form-data') {
+    if (this.nodeData?.requestBody?.type === 'form-data') {
       options.data.value = '<BASE64_ENCODED_FILE_DATA>';
     }
 
@@ -111,7 +111,7 @@ class requestNode extends Node {
       data: requestData,
     };
 
-    if (this.auth && this.auth.type === 'basic-auth') {
+    if (this.auth && this.auth?.type === 'basic-auth') {
       options.auth = {};
       options.auth.username = this.auth.username;
       options.auth.password = this.auth.password;

src/components/molecules/flow/utils.js

@@ -119,7 +119,9 @@ export const initFlowData = {
       id: '1',
       type: 'authNode',
       description: 'Define authentication for the requests',
-      data: {},
+      data: {
+        type: 'no-auth',
+      },
       position: {
         x: 400,
         y: 150,