fix: remove double disable class

zabute · Jul 27, 2023 · 4304469 · 4304469
1 parent 1524afa
commit 4304469
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/src/FormsyRadioGroup.tsx b/src/FormsyRadioGroup.tsx
@@ -88,7 +88,6 @@ class FormsyRadioGroup extends Component<IFormsyRadioGroupProps> {
     };
 
     const fieldProps = {
-      disabled,
       width,
       error: !disabled && error,
     };

diff --git a/test/FormsyRadioGroup.spec.tsx b/test/FormsyRadioGroup.spec.tsx
@@ -114,18 +114,6 @@ describe('<RadioGroup/>', () => {
       expect(labelField).toHaveClass('error');
       expect(labelField).toHaveTextContent('LABEL');
     });
-
-    it('should be disabled', () => {
-      wrapper = renderTestForm({
-        label: 'LABEL',
-        disabled: true,
-      });
-
-      const labelField = wrapper.container.querySelector('.field:first-child');
-
-      expect(labelField).toHaveClass('disabled');
-      expect(labelField).toHaveTextContent('LABEL');
-    });
   });
 
   describe('Field', () => {
@@ -152,6 +140,19 @@ describe('<RadioGroup/>', () => {
       expect(fields).toHaveLength(3);
       Array.from(fields).forEach((field) => expect(field).toHaveClass('ten'));
     });
+
+    it('should not apply disabled on the field it self, prevent double disable', () => {
+      wrapper = renderTestForm({
+        disabled: true,
+      });
+
+      const fields = wrapper.container.querySelectorAll('.field');
+
+      expect(fields).toHaveLength(3);
+      Array.from(fields).forEach((field) =>
+        expect(field).not.toHaveClass('disabled')
+      );
+    });
   });
 
   describe('Field.Group', () => {