f

Author: screspod

web/src/components/common/Checkbox.tsx

@@ -25,7 +25,6 @@ const Checkbox: React.FC<CheckboxProps> = ({
   labelClassName = '',
   id,
   label,
-  dataTestId,
   helpText,
 }) => {
   const { settings } = useSettings();
@@ -41,7 +40,7 @@ const Checkbox: React.FC<CheckboxProps> = ({
           onChange={onChange}
           disabled={disabled}
           className={`h-4 w-4 focus:ring-offset-2 border-gray-300 rounded ${className}`}
-          data-testid={dataTestId}
+          data-testid={`bool-input-${id}`}
           style={{
             color: themeColor,
             '--tw-ring-color': themeColor,

web/src/components/common/Input.tsx

@@ -35,7 +35,6 @@ const Input: React.FC<InputProps> = ({
   labelClassName = '',
   id,
   label,
-  dataTestId,
   helpText,
 }) => {
   const { settings } = useSettings();
@@ -71,7 +70,7 @@ const Input: React.FC<InputProps> = ({
             '--tw-ring-color': themeColor,
             '--tw-ring-offset-color': themeColor,
           } as React.CSSProperties}
-          data-testid={dataTestId}
+          data-testid={`text-input-${id}`}
         />
       </div>
       {error && <p className="mt-1 text-sm text-red-500">{error}</p>}

web/src/components/common/Radio.tsx

@@ -28,7 +28,6 @@ const Radio: React.FC<RadioProps> = ({
    labelClassName = '',
    id,
    label,
-   dataTestId,
    helpText,
 }) => {
   const { settings } = useSettings();
@@ -51,7 +50,7 @@ const Radio: React.FC<RadioProps> = ({
               onChange={onChange}
               disabled={disabled}
               className={`h-4 w-4 focus:ring-offset-2 border-gray-300 ${className}`}
-              data-testid={dataTestId ? `${dataTestId}-${option.name}` : undefined}
+              data-testid={`radio-input-${option.name}`}
               style={{
                 color: themeColor,
                 '--tw-ring-color': themeColor,

web/src/components/common/Textarea.tsx

@@ -31,7 +31,6 @@ const Textarea = ({
   labelClassName = '',
   id,
   label,
-  dataTestId,
   helpText,
 }: TextareaProps) => {
   const { settings } = useSettings();
@@ -60,7 +59,7 @@ const Textarea = ({
           '--tw-ring-color': themeColor,
           '--tw-ring-offset-color': themeColor,
         } as CSSProperties}
-        data-testid={dataTestId}
+        data-testid={`textarea-input-${id}`}
       />
       {error && <p className="mt-1 text-sm text-red-500">{error}</p>}
       {helpText && !error && <p className="mt-1 text-sm text-gray-500">{helpText}</p>}

web/src/components/wizard/config/ConfigurationStep.tsx

@@ -167,41 +167,37 @@ const ConfigurationStep: React.FC<ConfigurationStepProps> = ({ onNext }) => {
     switch (item.type) {
       case 'text':
         return (
-            <Input
-              value={getDisplayValue(item)}
-              onChange={handleInputChange}
-              dataTestId={`text-input-${item.name}`}
-            />
+          <Input
+            value={getDisplayValue(item)}
+            onChange={handleInputChange}
+          />
         );
 
       case 'textarea':
         return (
-            <Textarea
-              value={getDisplayValue(item)}
-              onChange={handleInputChange}
-              dataTestId={`textarea-input-${item.name}`}
-            />
+          <Textarea
+            value={getDisplayValue(item)}
+            onChange={handleInputChange}
+          />
 
         );
 
       case 'bool':
         return (
-            <Checkbox
-              checked={getEffectiveValue(item) === '1'}
-              onChange={handleCheckboxChange}
-              dataTestId={`bool-input-${item.name}`}
-            />
+          <Checkbox
+            checked={getEffectiveValue(item) === '1'}
+            onChange={handleCheckboxChange}
+          />
         );
 
       case 'radio':
         if (item.items) {
           return (
-              <Radio
-                value={getEffectiveValue(item)}
-                options={item.items}
-                onChange={e => handleRadioChange(item.name, e)}
-                dataTestId={`radio-input-${item.name}`}
-              />
+            <Radio
+              value={getEffectiveValue(item)}
+              options={item.items}
+              onChange={e => handleRadioChange(item.name, e)}
+            />
           );
         }
         return null;
@@ -219,19 +215,19 @@ const ConfigurationStep: React.FC<ConfigurationStepProps> = ({ onNext }) => {
         {group.description && (
           <p className="text-gray-600 mb-4">{group.description}</p>
         )}
-          {group.items.map(item => {
-            const renderedItem = renderConfigItem(item);
-            if (!renderedItem) return null;
-            return (
-              <ConfigItem
-                id={item.name}
-                label={item.title}
-                helpText={item.help_text}
-              >
-                {renderedItem}
-              </ConfigItem>
-            );
-          })}
+        {group.items.map(item => {
+          const renderedItem = renderConfigItem(item);
+          if (!renderedItem) return null;
+          return (
+            <ConfigItem
+              id={item.name}
+              label={item.title}
+              helpText={item.help_text}
+            >
+              {renderedItem}
+            </ConfigItem>
+          );
+        })}
       </div>
     );
   };