Merge pull request #76 from DouglasNeuroInformatics/test-ids

fix: add testid to destructive-action-dialog

Author: joshunrau

src/components/Form/BooleanField/BooleanFieldRadio.tsx

@@ -49,31 +49,25 @@ export const BooleanFieldRadio = ({
 
   return (
     <FieldGroup name={name}>
-      <Label>{label}</Label>
+      <FieldGroup.Row>
+        <Label>{label}</Label>
+        <FieldGroup.Description description={description} />
+      </FieldGroup.Row>
       <RadioGroup
         disabled={disabled || readOnly}
         name={name}
         value={stringifyBoolean(value)}
         onValueChange={handleValueChange}
       >
-        <FieldGroup.Description description={description} />
         <FieldGroup.Row>
           <RadioGroup.Item id={`${name}-true`} value="true" />
-          <Label
-            aria-disabled={disabled || readOnly}
-            className="text-muted-foreground font-normal"
-            htmlFor={`${name}-true`}
-          >
+          <Label aria-disabled={disabled || readOnly} className="text-muted-foreground" htmlFor={`${name}-true`}>
             {options?.true ?? t('form.radioLabels.true')}
           </Label>
         </FieldGroup.Row>
         <FieldGroup.Row>
           <RadioGroup.Item id={`${name}-false`} value="false" />
-          <Label
-            aria-disabled={disabled || readOnly}
-            className="text-muted-foreground font-normal"
-            htmlFor={`${name}-false`}
-          >
+          <Label aria-disabled={disabled || readOnly} className="text-muted-foreground" htmlFor={`${name}-false`}>
             {options?.false ?? t('form.radioLabels.false')}
           </Label>
         </FieldGroup.Row>

src/components/Form/Form.stories.tsx

@@ -59,12 +59,14 @@ type SimpleExampleFormSchemaType = typeof $SimpleExampleFormData;
 
 const booleanFields: FormFields<Pick<ExampleFormData, 'booleanCheck' | 'booleanRadio'>> = {
   booleanRadio: {
+    description: 'This is a boolean radio field',
     disabled: DISABLED,
     kind: 'boolean',
     label: 'Radio',
     variant: 'radio'
   },
   booleanCheck: {
+    description: 'This is a boolean check field',
     disabled: DISABLED,
     kind: 'boolean',
     label: 'Checkbox',

src/providers/CoreProvider/DestructiveActionDialog.tsx

@@ -69,7 +69,7 @@ export const DestructiveActionDialog = () => {
 
   return (
     <Dialog open={current !== null} onOpenChange={handleOpenChange}>
-      <Dialog.Content onOpenAutoFocus={(event) => event.preventDefault()}>
+      <Dialog.Content data-testid="destructive-action-dialog" onOpenAutoFocus={(event) => event.preventDefault()}>
         <Dialog.Header>
           <Dialog.Title>{display.title}</Dialog.Title>
           <Dialog.Description>{display.description}</Dialog.Description>