From 79991bc09b064ba3d598980c31e4355949f34e82 Mon Sep 17 00:00:00 2001
From: Allan Oliveira <138060158+allan-chagas-brisa@users.noreply.github.com>
Date: Wed, 13 Sep 2023 09:45:02 -0300
Subject: [PATCH] fix: loading on ion-table (#822) :hammer:

Co-authored-by: Iury Nogueira <iunog@hotmail.com>
---
 .../ion/src/lib/table/table.component.html    | 23 ++++++++++++++--
 .../ion/src/lib/table/table.component.spec.ts | 27 +++++++++++++++++++
 projects/ion/src/lib/table/table.module.ts    |  2 ++
 3 files changed, 50 insertions(+), 2 deletions(-)

diff --git a/projects/ion/src/lib/table/table.component.html b/projects/ion/src/lib/table/table.component.html
index eb5a2082f..cbec8bf55 100644
--- a/projects/ion/src/lib/table/table.component.html
+++ b/projects/ion/src/lib/table/table.component.html
@@ -183,17 +183,36 @@
     </tbody>
   </table>
   <div *ngIf="smartData.length === 0" class="noData">
+    <ion-spinner
+      *ngIf="config.loading; else noData"
+      [size]="42"
+      color="primary"
+    ></ion-spinner>
+  </div>
+
+  <ng-template #noData>
     <ion-icon size="40" type="exclamation-rounded"></ion-icon>
     <span>Não há dados</span>
-  </div>
+  </ng-template>
+
   <div
     *ngIf="config.pagination"
     data-testid="pagination-container"
     class="footer-table"
   >
-    <span>{{ config.pagination.total }}</span>
+    <span *ngIf="!config.loading">{{ config.pagination.total }}</span>
+
+    <span
+      *ngIf="config.loading"
+      class="loading-message"
+      data-testid="loading-pagination"
+    >
+      Carregando página
+    </span>
+
     <div style="overflow-x: auto">
       <ion-pagination
+        [loading]="config.loading"
         [total]="config.pagination.total"
         [itemsPerPage]="config.pagination.itemsPerPage"
         [page]="config.pagination.page"
diff --git a/projects/ion/src/lib/table/table.component.spec.ts b/projects/ion/src/lib/table/table.component.spec.ts
index bf9c13bf0..5d63d0d9a 100644
--- a/projects/ion/src/lib/table/table.component.spec.ts
+++ b/projects/ion/src/lib/table/table.component.spec.ts
@@ -19,6 +19,7 @@ import { SafeAny } from '../utils/safe-any';
 import { StatusType } from './../core/types/status';
 import { IonTableComponent } from './table.component';
 import { IonTableModule } from './table.module';
+import { IonSpinnerModule } from './../spinner/spinner.module';
 import { ActionTable, Column, ColumnType, ConfigTable } from './utilsTable';
 
 const disabledArrowColor = '#CED2DB';
@@ -88,6 +89,7 @@ const sut = async (
       IonTagModule,
       IonPopConfirmModule,
       IonTooltipModule,
+      IonSpinnerModule,
     ],
   });
 };
@@ -243,6 +245,7 @@ describe('Table > Changes', () => {
         IonTagModule,
         IonPopConfirmModule,
         IonTooltipModule,
+        IonSpinnerModule,
       ],
     });
     const newData = [{ name: 'Meteora', deleted: false, id: 2 }];
@@ -264,6 +267,7 @@ describe('Table > Changes', () => {
         IonTagModule,
         IonPopConfirmModule,
         IonTooltipModule,
+        IonSpinnerModule,
       ],
     });
     propsToChange.config.data = [];
@@ -692,6 +696,28 @@ describe('Table > Pagination', () => {
     },
   };
 
+  it('should render loading when table is loading', async () => {
+    const tableWithLoading = JSON.parse(
+      JSON.stringify(tableWithPagination)
+    ) as IonTableProps<Disco>;
+    const totalPagination = screen.queryByTestId('total-pagination');
+    tableWithLoading.config.loading = true;
+    await sut(tableWithLoading);
+    expect(screen.getByTestId('loading-pagination')).toBeInTheDocument();
+    expect(totalPagination).not.toBeInTheDocument();
+  });
+
+  it('should render loading when dont have data and table is loading', async () => {
+    const tableWithLoading = JSON.parse(
+      JSON.stringify(tableWithPagination)
+    ) as IonTableProps<Disco>;
+    tableWithLoading.config.loading = true;
+    tableWithLoading.config.data = [];
+
+    await sut(tableWithLoading);
+    expect(screen.getByTestId('ion-spinner')).toBeInTheDocument();
+  });
+
   it('should render the pagination', async () => {
     await sut(tableWithPagination);
     expect(screen.getByTestId('pagination-container')).toBeInTheDocument();
@@ -876,6 +902,7 @@ const sutCustomRowTemplate = async (
       IonTagModule,
       IonPopConfirmModule,
       IonTableModule,
+      IonSpinnerModule,
     ],
   });
 };
diff --git a/projects/ion/src/lib/table/table.module.ts b/projects/ion/src/lib/table/table.module.ts
index 7597b2bee..1e773ce99 100644
--- a/projects/ion/src/lib/table/table.module.ts
+++ b/projects/ion/src/lib/table/table.module.ts
@@ -8,6 +8,7 @@ import { IonIconModule } from '../icon/icon.module';
 import { IonPaginationModule } from '../pagination/pagination.module';
 import { IonPopConfirmModule } from '../popconfirm/popconfirm.module';
 import { IonTooltipModule } from '../tooltip/tooltip.module';
+import { IonSpinnerModule } from '../spinner/spinner.module';
 
 @NgModule({
   declarations: [IonTableComponent],
@@ -20,6 +21,7 @@ import { IonTooltipModule } from '../tooltip/tooltip.module';
     IonPaginationModule,
     IonPopConfirmModule,
     IonTooltipModule,
+    IonSpinnerModule,
   ],
   exports: [IonTableComponent],
 })