Merge pull request #103 from samuel-gomez/feature/fix-cover-table

fix(table): cover and clean

Author: samuel-gomez

scripts/clean.js

@@ -30,13 +30,11 @@ const pathsDelete = [
   './src/pages/Home/Galleries',
   './src/assets',
   './src/shared/testsUtils/clearString.ts',
-  './src/shared/toolkit.ts',
   './src/shared/components/LiveCode',
   './src/shared/components/Cards',
   './src/shared/components/LazyImage',
   './src/shared/components/DownloadLink',
   './src/shared/components/Editor',
-  './src/shared/components/testsUtils/clearString.ts',
   './src/pages/Demos',
   './features/Demos',
   'renovate.json',

src/shared/components/Table/Headers/Header/__tests__/Header.test.tsx

@@ -16,4 +16,18 @@ describe('Header', () => {
     within(baseElement).getByRole(scope === 'col' || scope === undefined ? 'columnheader' : 'rowheader');
     screen.getByText('label');
   });
+
+  it('Should render Header with class "af-table__th--blank" when isBlank is true', () => {
+    const { baseElement } = renderWithContainer(<Header isBlank />, container);
+
+    const header = within(baseElement).getByRole('columnheader');
+    expect(header).toHaveClass('af-table__th af-table__th--blank');
+  });
+
+  it('Should render Header without class "af-table__th--blank" when isBlank is false', () => {
+    const { baseElement } = renderWithContainer(<Header isBlank={false} />, container);
+
+    const header = within(baseElement).getByRole('columnheader');
+    expect(header).toHaveClass('af-table__th');
+  });
 });

src/shared/testsUtils/sharedScenarios/table/utils.ts

@@ -43,7 +43,10 @@ export const expectTable = async (tableItemsType = DEFAULT_TABLE_ITEMS_TYPE) =>
 };
 
 export const expectCellsContent = (cells: HTMLElement[], ...args: string[]) => {
-  args
+  const argsWithoutLast = [...args];
+  argsWithoutLast.pop();
+
+  argsWithoutLast
     .filter(item => !isNull(item))
     .forEach((headerLabel, index) => {
       expect(cells[index]).toHaveTextContent(RegExp(headerLabel));