fix tests temporarily

adobe · Dec 13, 2024 · b9ac520 · b9ac520
1 parent 5eaad74
commit b9ac520
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/packages/@react-aria/test-utils/src/gridlist.ts b/packages/@react-aria/test-utils/src/gridlist.ts
@@ -89,7 +89,7 @@ export class GridListTester {
     let {
       row,
       needsLongPress,
-      checkboxSelection,
+      checkboxSelection = true,
       interactionType = this._interactionType
     } = opts;
 

diff --git a/packages/@react-aria/test-utils/src/tree.ts b/packages/@react-aria/test-utils/src/tree.ts
@@ -110,6 +110,11 @@ export class TreeTester {
 
     let rowCheckbox = within(row).queryByRole('checkbox');
 
+    // TODO: investigagte why pressElement on a disabled checkbox is still toggling row selection in single selection
+    if (rowCheckbox?.getAttribute('disabled') === '') {
+      return;
+    }
+
     // this would be better than the check to do nothing in events.ts
     // also, it'd be good to be able to trigger selection on the row instead of having to go to the checkbox directly
     if (interactionType === 'keyboard' && !checkboxSelection) {

diff --git a/packages/@react-aria/test-utils/src/types.ts b/packages/@react-aria/test-utils/src/types.ts
@@ -120,6 +120,7 @@ export interface ToggleGridRowOpts extends BaseGridRowInteractionOpts {
   needsLongPress?: boolean,
   /**
    * Whether the checkbox should be used to select the row. If false, will attempt to select the row via press.
+   * @default 'true'
    */
   checkboxSelection?: boolean
 }