[terra-table] Remove Console Warning Message when No Pinned Columns (#…

…1923)

packages/terra-data-grid/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Unreleased
 
+* Changed
+  * Removed console warning message when no pinned columns exist.
+
 ## 1.5.0 - (December 1, 2023)
 
 * Changed

packages/terra-data-grid/tests/jest/DataGrid.test.jsx

@@ -1,9 +1,8 @@
 import React from 'react';
 /* eslint-disable-next-line import/no-extraneous-dependencies */
-import { mountWithIntl, shallowWithIntl } from 'terra-enzyme-intl';
+import { mountWithIntl } from 'terra-enzyme-intl';
 import { v4 as uuidv4 } from 'uuid';
 import DataGrid from '../../src/DataGrid';
-import ERRORS from '../../src/utils/constants';
 
 // Source data for tests
 const dataFile = {
@@ -244,46 +243,5 @@ describe('with pinned columns', () => {
     const pinnedColumnHeaderCells = wrapper.find('.pinned');
 
     expect(pinnedColumnHeaderCells).toHaveLength(1 * (dataFile.rows.length + 1));
-    expect(console.warn).toHaveBeenCalledWith(expect.stringContaining(ERRORS.PINNED_COLUMNS_UNDEFINED)); // eslint-disable-line no-console
-  });
-});
-
-describe('Error handling - prop types', () => {
-  it('throws an error if rowHeaderIndex is not an integer', () => {
-    shallowWithIntl(
-      <DataGrid
-        id="test-terra-data-grid"
-        rows={dataFile.rows}
-        rowHeaderIndex="2"
-      />,
-    );
-
-    expect(console.error).toHaveBeenCalledWith(expect.stringContaining(ERRORS.ROW_HEADER_INDEX_NOT_AN_INTEGER)); // eslint-disable-line no-console
-  });
-
-  it('throws an error if rowHeaderIndex is not a positive integer', () => {
-    shallowWithIntl(
-      <DataGrid
-        id="test-terra-data-grid"
-        rows={dataFile.rows}
-        rowHeaderIndex={-1}
-      />,
-    );
-
-    expect(console.error).toHaveBeenCalledWith(expect.stringContaining(ERRORS.ROW_HEADER_INDEX_LESS_THAN_ZERO)); // eslint-disable-line no-console
-  });
-
-  it('throws an error if rowHeaderIndex is greater than the length of pinned columns', () => {
-    shallowWithIntl(
-      <DataGrid
-        id="test-terra-data-grid"
-        pinnedColumns={dataFile.cols.slice(0, 2)}
-        overflowColumns={dataFile.cols.slice(2)}
-        rowHeaderIndex={2}
-        rows={dataFile.rows}
-      />,
-    );
-
-    expect(console.error).toHaveBeenCalledWith(expect.stringContaining(ERRORS.ROW_HEADER_INDEX_EXCEEDS_PINNED)); // eslint-disable-line no-console
   });
 });

packages/terra-table/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Unreleased
 
+* Changed
+  * Removed console warning message when no pinned columns exist.
+
 ## 5.2.1 - (December 1, 2023)
 
 * Fixed

packages/terra-table/src/Table.jsx

@@ -13,7 +13,6 @@ import Section from './subcomponents/Section';
 import ColumnHeader from './subcomponents/ColumnHeader';
 import ColumnContext from './utils/ColumnContext';
 import columnShape from './proptypes/columnShape';
-import ERRORS from './utils/constants';
 import GridContext, { GridConstants } from './utils/GridContext';
 import rowShape from './proptypes/rowShape';
 import validateRowHeaderIndex from './proptypes/validators';
@@ -208,11 +207,6 @@ function Table(props) {
     intl,
   } = props;
 
-  if (pinnedColumns.length === 0) {
-    // eslint-disable-next-line no-console
-    console.warn(ERRORS.PINNED_COLUMNS_UNDEFINED);
-  }
-
   // Manage column resize
   const [tableHeight, setTableHeight] = useState(0);
   const [activeIndex, setActiveIndex] = useState(null);

packages/terra-table/src/utils/constants.js

@@ -2,7 +2,6 @@ const ERRORS = {
   ROW_HEADER_INDEX_EXCEEDS_PINNED: 'Prop rowHeaderIndex exceeds the size of pinnedColumns.',
   ROW_HEADER_INDEX_LESS_THAN_ZERO: 'Prop rowHeaderIndex must be a positive integer.',
   ROW_HEADER_INDEX_NOT_AN_INTEGER: 'Prop rowHeaderIndex must be an integer.',
-  PINNED_COLUMNS_UNDEFINED: 'To be properly accessible, the row header column should be a pinned column. please set pinned columns',
 };
 
 export default ERRORS;

packages/terra-table/tests/jest/Table.test.jsx

@@ -474,7 +474,6 @@ describe('with pinned columns', () => {
     const pinnedColumnHeaderCells = wrapper.find('.pinned');
 
     expect(pinnedColumnHeaderCells).toHaveLength(1 * (tableData.rows.length + 1));
-    expect(console.warn).toHaveBeenCalledWith(expect.stringContaining(ERRORS.PINNED_COLUMNS_UNDEFINED)); // eslint-disable-line no-console
   });
 });