feat(UncontrolledTable): allow selectable attribute to be passed in w…

…ith rows

src/components/Table/SortableTable.js

@@ -74,6 +74,7 @@ function defaultRenderRow(
 
 function getSelectableCell(row, rowSelected, onSelect) {
   const selectRowId = uniqueId('select-row-');
+  const rowSelectable = Object.hasOwn(row, 'selectable') ? row.selectable : true
   return (
     <>
       <Label for={selectRowId} hidden>
@@ -86,6 +87,7 @@ function getSelectableCell(row, rowSelected, onSelect) {
         checked={rowSelected(row)}
         onClick={(e) => e.stopPropagation()}
         onChange={(e) => onSelect(row, e.target.checked)}
+        disabled={!rowSelectable}
       />
     </>
   );

src/components/Table/SortableTable.spec.js

@@ -377,6 +377,19 @@ describe('<SortableTable />', () => {
       assert.equal(wrapper.find('input').length, 4, 'select checkbox missing');
     });
 
+    it('should disable select checkbox when specified', () => {
+      const rows = [
+        {name: 'Alpha', selectable: false},
+        {name: 'Bravo', selectable: true},
+        {name: 'Charlie', selectable: true},
+        {name: 'Delta', selectable: true}
+      ];
+      const wrapper = mount(<SortableTable columns={columns} rows={rows} rowSelected={() => false} />);
+
+      const trs = wrapper.find('tr');
+      assert.equal(trs.at(1).find('input').prop('disabled'), true);
+    });
+
     it('should call onSelect when clicked', () => {
       const onSelect = sinon.stub();
       const wrapper = mount(

src/components/Table/UncontrolledTable.js

@@ -69,7 +69,8 @@ export default class UncontrolledTable extends React.Component {
   };
 
   get allSelected() {
-    return this.props.rows.length && this.state.selected.length === this.props.rows.length;
+    const selectableRows = this.props.rows.filter((row) => Object.hasOwn(row, 'selectable') ? row.selectable : true)
+    return this.props.rows.length && this.state.selected.length === selectableRows.length;
   }
 
   selected(value) {
@@ -88,7 +89,8 @@ export default class UncontrolledTable extends React.Component {
   };
 
   toggleAll = () => {
-    const newSelection = this.allSelected ? [] : this.props.rows;
+    const selectableRows = this.props.rows.filter((row) => Object.hasOwn(row, 'selectable') ? row.selectable : true)
+    const newSelection = this.allSelected ? [] : selectableRows;
 
     this.setState({ selected: newSelection }, () => {
       this.props.onSelect(newSelection);

src/components/Table/UncontrolledTable.spec.js

@@ -302,6 +302,37 @@ describe('<UncontrolledTable />', () => {
     assert.equal(ths.length, columns.length + 1); // For selectable column
   });
 
+  it('should disable select checkbox when specified', () => {
+    const columns = [{ header: 'Name', cell: (row) => row.name }];
+    const rows = [
+      {name: 'Alpha', selectable: false},
+      {name: 'Bravo', selectable: true},
+      {name: 'Charlie', selectable: true},
+      {name: 'Delta', selectable: true}
+    ];
+    const wrapper = mount(<UncontrolledTable columns={columns} rows={rows} selectable  />);
+
+    const trs = wrapper.find('tr');
+    assert.equal(trs.at(1).find('input').prop('disabled'), true);
+  });
+
+  it('should only select on selectable rows when selecting selectAll', () => {
+    const columns = [{ header: 'Name', cell: (row) => row.name }];
+    const rows = [
+      {name: 'Alpha', selectable: false},
+      {name: 'Bravo', selectable: true},
+      {name: 'Charlie', selectable: true},
+      {name: 'Delta', selectable: true}
+    ];
+    const wrapper = mount(<UncontrolledTable columns={columns} rows={rows} selectable />);
+
+    wrapper
+      .find({ type: 'checkbox' })
+      .first()
+      .simulate('change', { target: { checked: true } });
+    assert.equal(wrapper.state().selected.length, 3);
+  });
+
   it('should call onSelect when selectable row picked', () => {
     const columns = [{ header: 'Name', cell: (row) => row }];
     const rows = ['Alpha', 'Bravo', 'Charlie', 'Delta', 'Echo', 'Foxtrot', 'Golf', 'Hotel'];

src/components/Table/UncontrolledTable.stories.js

@@ -77,6 +77,7 @@ const DATA = [
     last: 'Turner',
     email: '[email protected]',
     dob: new Date(1961, 8, 19),
+    selectable: false
   },
   {
     key: '000',
@@ -85,6 +86,7 @@ const DATA = [
     last: 'Headroom',
     email: '[email protected]',
     dob: new Date(1984, 6, 1),
+    selectable: false 
   },
 ];