From c5754c916059ab867f106dbe20bf292d4c93afbe Mon Sep 17 00:00:00 2001
From: Eric Tuvesson <eric.tuvesson@gmail.com>
Date: Thu, 27 Jun 2024 15:42:25 +0200
Subject: [PATCH] feat(runtime): Add "data-testId" attribute to Columns node
 (#44)

---
 .../src/components/visual/Columns/Columns.tsx        |  3 +++
 .../noodl-viewer-react/src/nodes/visual/columns.js   | 12 +++++++++++-
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/packages/noodl-viewer-react/src/components/visual/Columns/Columns.tsx b/packages/noodl-viewer-react/src/components/visual/Columns/Columns.tsx
index 3ac6235..81ec7bf 100644
--- a/packages/noodl-viewer-react/src/components/visual/Columns/Columns.tsx
+++ b/packages/noodl-viewer-react/src/components/visual/Columns/Columns.tsx
@@ -7,6 +7,8 @@ export interface ColumnsProps extends Noodl.ReactProps {
   marginX: string;
   marginY: string;
 
+  attrs: React.Attributes;
+
   justifyContent: 'flex-start' | 'flex-end' | 'center';
   direction: 'row' | 'column';
   minWidth: string;
@@ -123,6 +125,7 @@ export function Columns(props: ColumnsProps) {
 
   return (
     <div
+      {...props.attrs}
       className={['columns-container', props.className].join(' ')}
       ref={containerRef}
       style={{
diff --git a/packages/noodl-viewer-react/src/nodes/visual/columns.js b/packages/noodl-viewer-react/src/nodes/visual/columns.js
index 706cb1f..cb13c33 100644
--- a/packages/noodl-viewer-react/src/nodes/visual/columns.js
+++ b/packages/noodl-viewer-react/src/nodes/visual/columns.js
@@ -23,7 +23,7 @@ const ColumnsNode = {
     ]
   },
 
-  initialize() {
+  initialize() {    this.props.attrs = {};
     this.props.layoutString = '1 2 1';
     this.props.minWidth = 0;
     this.props.marginX = 16;
@@ -62,6 +62,16 @@ const ColumnsNode = {
           );
         }
 
+        this.forceUpdate();
+      }
+    },
+    testId: {
+      index: 100009,
+      displayName: 'Test ID Attribute',
+      group: 'Advanced HTML',
+      type: 'string',
+      set(value) {
+        this.props.attrs["data-testid"] = value;
         this.forceUpdate();
       }
     }