Match the Created column's ID to what is expected by React Table (#3174)

Author: mattseddon

webview/src/experiments/components/App.test.tsx

@@ -736,10 +736,8 @@ describe('App', () => {
 
     it('should have the same options in the empty placeholders', () => {
       renderTableWithPlaceholder()
-      const header = screen.getByTestId('header-timestamp')
-      const placeholders = screen.getAllByTestId(
-        /header-timestamp.+placeholder/
-      )
+      const header = screen.getByTestId('header-Created')
+      const placeholders = screen.getAllByTestId(/header-Created.+placeholder/)
       const entireColumn = [header, ...placeholders]
 
       expect(entireColumn).toHaveLength(5)
@@ -787,7 +785,7 @@ describe('App', () => {
       it('should send the column id and not the placeholder id as the message payload', () => {
         renderTableWithPlaceholder()
         const placeholders = screen.getAllByTestId(
-          /header-timestamp.+placeholder/
+          /header-Created.+placeholder/
         )
         const placeholder = placeholders[0]
         fireEvent.contextMenu(placeholder, { bubbles: true })
@@ -801,7 +799,7 @@ describe('App', () => {
 
         expect(mockPostMessage).toHaveBeenCalledTimes(1)
         expect(mockPostMessage).toHaveBeenCalledWith({
-          payload: 'timestamp',
+          payload: 'Created',
           type: MessageFromWebviewType.HIDE_EXPERIMENTS_TABLE_COLUMN
         })
       })

webview/src/experiments/components/table/Table.test.tsx

@@ -192,23 +192,23 @@ describe('Table', () => {
     it('should not have the workspaceChange class on a cell if there are no changes', async () => {
       renderExperimentsTable()
 
-      const row = await screen.findByTestId('timestamp___workspace')
+      const row = await screen.findByTestId('Created___workspace')
 
       expect(row?.className.includes(styles.workspaceChange)).toBe(false)
     })
 
     it('should not have the workspaceChange class on a cell if there are changes to other columns but not this one', async () => {
       renderExperimentsTable({ changes: ['a_change'] })
 
-      const row = await screen.findByTestId('timestamp___workspace')
+      const row = await screen.findByTestId('Created___workspace')
 
       expect(row?.className.includes(styles.workspaceChange)).toBe(false)
     })
 
     it('should have the workspaceChange class on a cell if there are changes matching the column id', async () => {
-      renderExperimentsTable({ changes: ['timestamp'] })
+      renderExperimentsTable({ changes: ['Created'] })
 
-      const row = await screen.findByTestId('timestamp___workspace')
+      const row = await screen.findByTestId('Created___workspace')
 
       expect(row?.className.includes(styles.workspaceChange)).toBe(true)
     })
@@ -250,13 +250,7 @@ describe('Table', () => {
     })
 
     it('should order the columns with the columnOrder from the data', async () => {
-      const columnOrder = [
-        'id',
-        'timestamp',
-        'params:C',
-        'params:B',
-        'params:A'
-      ]
+      const columnOrder = ['id', 'Created', 'params:C', 'params:B', 'params:A']
       const tableDataWithCustomColOrder = {
         ...sortingTableDataFixture,
         columnOrder

webview/src/experiments/util/buildColumns.tsx

@@ -51,7 +51,7 @@ const getMainColumnProperties = (
         CellContext<Column, CellValue>
       >,
       header: () => <TimestampHeader />,
-      id: ColumnType.TIMESTAMP
+      id: 'Created'
     }
   }