@jmruthers/pace-core 0.5.191 → 0.6.1

package/src/components/DataTable/components/__tests__/DataTableModals.test.tsx

@@ -62,7 +62,7 @@ interface TestData extends Record<string, unknown> {
 }
 
 describe('[component] DataTableModals', () => {
-  const defaultProps = {
+  const baseProps = {
     showImportModal: false,
     onCloseImportModal: vi.fn(),
     onImport: vi.fn(),
@@ -78,14 +78,14 @@ describe('[component] DataTableModals', () => {
 
   describe('Rendering', () => {
     it('renders nothing when no modals are open', () => {
-      const { container } = render(<DataTableModals {...defaultProps} />);
+      const { container } = render(<DataTableModals {...baseProps} />);
       expect(container.firstChild).toBeNull();
     });
 
     it('renders ImportModal when showImportModal is true', () => {
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
         />
       );
@@ -94,7 +94,7 @@ describe('[component] DataTableModals', () => {
     });
 
     it('does not render ImportModal when showImportModal is false', () => {
-      render(<DataTableModals {...defaultProps} />);
+      render(<DataTableModals {...baseProps} />);
 
       expect(screen.queryByTestId('import-modal')).not.toBeInTheDocument();
     });
@@ -104,7 +104,7 @@ describe('[component] DataTableModals', () => {
     it('passes isOpen prop to ImportModal', () => {
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
         />
       );
@@ -118,7 +118,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onCloseImportModal={onCloseImportModal}
         />
@@ -138,7 +138,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           importModalConfig={config}
         />
@@ -159,7 +159,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
           columns={columns}
@@ -188,7 +188,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
           columns={columns}
@@ -209,7 +209,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
         />
@@ -230,7 +230,7 @@ describe('[component] DataTableModals', () => {
 
       const { rerender } = render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={false}
           onStoreFocus={onStoreFocus}
         />
@@ -238,7 +238,7 @@ describe('[component] DataTableModals', () => {
 
       rerender(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onStoreFocus={onStoreFocus}
         />
@@ -252,7 +252,7 @@ describe('[component] DataTableModals', () => {
 
       const { rerender } = render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onRestoreFocus={onRestoreFocus}
         />
@@ -260,7 +260,7 @@ describe('[component] DataTableModals', () => {
 
       rerender(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={false}
           onRestoreFocus={onRestoreFocus}
         />
@@ -274,7 +274,7 @@ describe('[component] DataTableModals', () => {
     it('does not call focus handlers when not provided', () => {
       const { rerender } = render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={false}
         />
       );
@@ -282,7 +282,7 @@ describe('[component] DataTableModals', () => {
       expect(() => {
         rerender(
           <DataTableModals
-            {...defaultProps}
+            {...baseProps}
             showImportModal={true}
           />
         );
@@ -300,7 +300,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
         />
@@ -330,7 +330,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
         />
@@ -361,7 +361,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
           columns={[]}
@@ -386,7 +386,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
           columns={columns}
@@ -406,7 +406,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={undefined as any}
         />
@@ -432,7 +432,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
           columns={columns}
@@ -456,7 +456,7 @@ describe('[component] DataTableModals', () => {
 
       render(
         <DataTableModals
-          {...defaultProps}
+          {...baseProps}
           showImportModal={true}
           onImport={onImport}
           columns={columns}

package/src/components/DataTable/components/__tests__/EditableRow.test.tsx

@@ -179,7 +179,7 @@ describe('[component] EditableRow', () => {
     actions: [],
   });
   
-  const defaultProps = getDefaultProps();
+  const baseProps = getDefaultProps();
 
   beforeEach(() => {
     vi.clearAllMocks();
@@ -191,7 +191,7 @@ describe('[component] EditableRow', () => {
 
   describe('Rendering', () => {
     it('renders editable row with table row structure', () => {
-      render(<EditableRow {...defaultProps} />);
+      render(<EditableRow {...baseProps} />);
 
       const row = screen.getByRole('row');
       expect(row).toBeInTheDocument();
@@ -234,7 +234,7 @@ describe('[component] EditableRow', () => {
     });
 
     it('renders save and cancel buttons', () => {
-      render(<EditableRow {...defaultProps} />);
+      render(<EditableRow {...baseProps} />);
 
       expect(screen.getByRole('button', { name: /save changes/i })).toBeInTheDocument();
       expect(screen.getByRole('button', { name: /cancel editing/i })).toBeInTheDocument();
@@ -271,7 +271,7 @@ describe('[component] EditableRow', () => {
       const user = userEvent.setup();
       const onSave = vi.fn();
 
-      render(<EditableRow {...defaultProps} onSave={onSave} />);
+      render(<EditableRow {...baseProps} onSave={onSave} />);
 
       const saveButton = screen.getByRole('button', { name: /save changes/i });
       await user.click(saveButton);
@@ -437,7 +437,7 @@ describe('[component] EditableRow', () => {
 
       render(
         <EditableRow
-          {...defaultProps}
+          {...baseProps}
           editingData={editingData}
         />
       );
@@ -447,7 +447,7 @@ describe('[component] EditableRow', () => {
     });
 
     it('falls back to original row values when editingData is empty', () => {
-      render(<EditableRow {...defaultProps} editingData={{}} />);
+      render(<EditableRow {...baseProps} editingData={{}} />);
 
       expect(screen.getByDisplayValue('John Doe')).toBeInTheDocument();
       expect(screen.getByDisplayValue('john@example.com')).toBeInTheDocument();
@@ -468,21 +468,21 @@ describe('[component] EditableRow', () => {
 
   describe('Accessibility', () => {
     it('provides proper ARIA attributes for row', () => {
-      render(<EditableRow {...defaultProps} />);
+      render(<EditableRow {...baseProps} />);
 
       const row = screen.getByRole('row');
       expect(row).toHaveAttribute('role', 'row');
     });
 
     it('provides aria-label for save button', () => {
-      render(<EditableRow {...defaultProps} />);
+      render(<EditableRow {...baseProps} />);
 
       const saveButton = screen.getByRole('button', { name: /save changes/i });
       expect(saveButton).toHaveAttribute('aria-label', 'Save changes');
     });
 
     it('provides aria-label for cancel button', () => {
-      render(<EditableRow {...defaultProps} />);
+      render(<EditableRow {...baseProps} />);
 
       const cancelButton = screen.getByRole('button', { name: /cancel editing/i });
       expect(cancelButton).toHaveAttribute('aria-label', 'Cancel editing');
@@ -491,7 +491,7 @@ describe('[component] EditableRow', () => {
 
   describe('Edge Cases', () => {
     it('handles empty editingData gracefully', () => {
-      render(<EditableRow {...defaultProps} editingData={{}} />);
+      render(<EditableRow {...baseProps} editingData={{}} />);
 
       expect(screen.getByRole('row')).toBeInTheDocument();
     });
@@ -512,7 +512,7 @@ describe('[component] EditableRow', () => {
       const row = createMockRow();
 
       expect(() => {
-        render(<EditableRow {...defaultProps} row={row} />);
+        render(<EditableRow {...baseProps} row={row} />);
       }).not.toThrow();
     });
   });

package/src/components/DataTable/components/__tests__/ExpandButton.test.tsx

@@ -25,7 +25,7 @@ vi.mock('lucide-react', () => ({
 }));
 
 describe('[component] ExpandButton', () => {
-  const defaultProps = {
+  const baseProps = {
     rowId: 'row-1',
     isExpanded: false,
     hasChildren: true,
@@ -44,7 +44,7 @@ describe('[component] ExpandButton', () => {
     it('returns null when hasChildren is false', () => {
       const { container } = render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           hasChildren={false}
         />
       );
@@ -53,27 +53,27 @@ describe('[component] ExpandButton', () => {
     });
 
     it('renders button when hasChildren is true', () => {
-      render(<ExpandButton {...defaultProps} />);
+      render(<ExpandButton {...baseProps} />);
 
       expect(screen.getByRole('button')).toBeInTheDocument();
     });
 
     it('renders ChevronRight icon when not expanded', () => {
-      render(<ExpandButton {...defaultProps} isExpanded={false} />);
+      render(<ExpandButton {...baseProps} isExpanded={false} />);
 
       expect(screen.getByTestId('chevron-right-icon')).toBeInTheDocument();
       expect(screen.queryByTestId('chevron-down-icon')).not.toBeInTheDocument();
     });
 
     it('renders ChevronDown icon when expanded', () => {
-      render(<ExpandButton {...defaultProps} isExpanded={true} />);
+      render(<ExpandButton {...baseProps} isExpanded={true} />);
 
       expect(screen.getByTestId('chevron-down-icon')).toBeInTheDocument();
       expect(screen.queryByTestId('chevron-right-icon')).not.toBeInTheDocument();
     });
 
     it('has correct testId attribute', () => {
-      render(<ExpandButton {...defaultProps} rowId="test-row-123" />);
+      render(<ExpandButton {...baseProps} rowId="test-row-123" />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('data-testid', 'expand-button-test-row-123');
@@ -87,7 +87,7 @@ describe('[component] ExpandButton', () => {
 
       render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -104,7 +104,7 @@ describe('[component] ExpandButton', () => {
 
       render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -122,7 +122,7 @@ describe('[component] ExpandButton', () => {
 
       render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -142,7 +142,7 @@ describe('[component] ExpandButton', () => {
       render(
         <div onClick={handleParentClick}>
           <ExpandButton
-            {...defaultProps}
+            {...baseProps}
             onClick={handleClick}
           />
         </div>
@@ -163,7 +163,7 @@ describe('[component] ExpandButton', () => {
       render(
         <div onKeyDown={handleParentKeyDown}>
           <ExpandButton
-            {...defaultProps}
+            {...baseProps}
             onClick={handleClick}
           />
         </div>
@@ -180,33 +180,33 @@ describe('[component] ExpandButton', () => {
 
   describe('Accessibility', () => {
     it('has aria-label for collapsed state', () => {
-      render(<ExpandButton {...defaultProps} isExpanded={false} />);
+      render(<ExpandButton {...baseProps} isExpanded={false} />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-label', 'Expand row');
     });
 
     it('has aria-label for expanded state', () => {
-      render(<ExpandButton {...defaultProps} isExpanded={true} />);
+      render(<ExpandButton {...baseProps} isExpanded={true} />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-label', 'Collapse row');
     });
 
     it('has aria-expanded attribute', () => {
-      render(<ExpandButton {...defaultProps} isExpanded={false} />);
+      render(<ExpandButton {...baseProps} isExpanded={false} />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-expanded', 'false');
     });
 
     it('updates aria-expanded when state changes', () => {
-      const { rerender } = render(<ExpandButton {...defaultProps} isExpanded={false} />);
+      const { rerender } = render(<ExpandButton {...baseProps} isExpanded={false} />);
 
       let button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-expanded', 'false');
 
-      rerender(<ExpandButton {...defaultProps} isExpanded={true} />);
+      rerender(<ExpandButton {...baseProps} isExpanded={true} />);
 
       button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-expanded', 'true');
@@ -217,7 +217,7 @@ describe('[component] ExpandButton', () => {
     it('applies custom className', () => {
       render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           className="custom-class"
         />
       );
@@ -227,7 +227,7 @@ describe('[component] ExpandButton', () => {
     });
 
     it('has proper button styling classes', () => {
-      render(<ExpandButton {...defaultProps} />);
+      render(<ExpandButton {...baseProps} />);
 
       const button = screen.getByRole('button');
       expect(button.className).toContain('inline-flex');
@@ -243,7 +243,7 @@ describe('[component] ExpandButton', () => {
 
       render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -260,7 +260,7 @@ describe('[component] ExpandButton', () => {
       expect(() => {
         render(
           <ExpandButton
-            {...defaultProps}
+            {...baseProps}
             onClick={undefined as any}
           />
         );
@@ -270,7 +270,7 @@ describe('[component] ExpandButton', () => {
     it('handles empty rowId', () => {
       render(
         <ExpandButton
-          {...defaultProps}
+          {...baseProps}
           rowId=""
         />
       );
@@ -282,7 +282,7 @@ describe('[component] ExpandButton', () => {
 });
 
 describe('[component] ExpandButtonMinimal', () => {
-  const defaultProps = {
+  const baseProps = {
     rowId: 'row-1',
     isExpanded: false,
     hasChildren: true,
@@ -301,7 +301,7 @@ describe('[component] ExpandButtonMinimal', () => {
     it('returns null when hasChildren is false', () => {
       const { container } = render(
         <ExpandButtonMinimal
-          {...defaultProps}
+          {...baseProps}
           hasChildren={false}
         />
       );
@@ -310,27 +310,27 @@ describe('[component] ExpandButtonMinimal', () => {
     });
 
     it('renders button when hasChildren is true', () => {
-      render(<ExpandButtonMinimal {...defaultProps} />);
+      render(<ExpandButtonMinimal {...baseProps} />);
 
       expect(screen.getByRole('button')).toBeInTheDocument();
     });
 
     it('renders minus sign when expanded', () => {
-      render(<ExpandButtonMinimal {...defaultProps} isExpanded={true} />);
+      render(<ExpandButtonMinimal {...baseProps} isExpanded={true} />);
 
       expect(screen.getByText('−')).toBeInTheDocument();
       expect(screen.queryByText('+')).not.toBeInTheDocument();
     });
 
     it('renders plus sign when not expanded', () => {
-      render(<ExpandButtonMinimal {...defaultProps} isExpanded={false} />);
+      render(<ExpandButtonMinimal {...baseProps} isExpanded={false} />);
 
       expect(screen.getByText('+')).toBeInTheDocument();
       expect(screen.queryByText('−')).not.toBeInTheDocument();
     });
 
     it('has correct testId attribute', () => {
-      render(<ExpandButtonMinimal {...defaultProps} rowId="test-row-456" />);
+      render(<ExpandButtonMinimal {...baseProps} rowId="test-row-456" />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('data-testid', 'expand-button-minimal-test-row-456');
@@ -344,7 +344,7 @@ describe('[component] ExpandButtonMinimal', () => {
 
       render(
         <ExpandButtonMinimal
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -361,7 +361,7 @@ describe('[component] ExpandButtonMinimal', () => {
 
       render(
         <ExpandButtonMinimal
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -379,7 +379,7 @@ describe('[component] ExpandButtonMinimal', () => {
 
       render(
         <ExpandButtonMinimal
-          {...defaultProps}
+          {...baseProps}
           onClick={handleClick}
         />
       );
@@ -399,7 +399,7 @@ describe('[component] ExpandButtonMinimal', () => {
       render(
         <div onClick={handleParentClick}>
           <ExpandButtonMinimal
-            {...defaultProps}
+            {...baseProps}
             onClick={handleClick}
           />
         </div>
@@ -415,21 +415,21 @@ describe('[component] ExpandButtonMinimal', () => {
 
   describe('Accessibility', () => {
     it('has aria-label for collapsed state', () => {
-      render(<ExpandButtonMinimal {...defaultProps} isExpanded={false} />);
+      render(<ExpandButtonMinimal {...baseProps} isExpanded={false} />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-label', 'Expand row');
     });
 
     it('has aria-label for expanded state', () => {
-      render(<ExpandButtonMinimal {...defaultProps} isExpanded={true} />);
+      render(<ExpandButtonMinimal {...baseProps} isExpanded={true} />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-label', 'Collapse row');
     });
 
     it('has aria-expanded attribute', () => {
-      render(<ExpandButtonMinimal {...defaultProps} isExpanded={false} />);
+      render(<ExpandButtonMinimal {...baseProps} isExpanded={false} />);
 
       const button = screen.getByRole('button');
       expect(button).toHaveAttribute('aria-expanded', 'false');
@@ -440,7 +440,7 @@ describe('[component] ExpandButtonMinimal', () => {
     it('applies custom className', () => {
       render(
         <ExpandButtonMinimal
-          {...defaultProps}
+          {...baseProps}
           className="custom-class"
         />
       );
@@ -450,7 +450,7 @@ describe('[component] ExpandButtonMinimal', () => {
     });
 
     it('has proper button styling classes', () => {
-      render(<ExpandButtonMinimal {...defaultProps} />);
+      render(<ExpandButtonMinimal {...baseProps} />);
 
       const button = screen.getByRole('button');
       expect(button.className).toContain('inline-flex');