fix(ESSNTL-5092): Fix sorting by Total systems (/groups) (#1944)

Fixes https://issues.redhat.com/browse/ESSNTL-5092.
RedHatInsights · Jul 24, 2023 · 6a6c6a8 · 6a6c6a8
1 parent d42e7e0
commit 6a6c6a8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/src/components/GroupsTable/GroupsTable.cy.js b/src/components/GroupsTable/GroupsTable.cy.js
@@ -180,9 +180,9 @@ describe('url search parameters', () => {
   });
 
   it('applies sorting', () => {
-    mountTable('/?order_by=host_ids&order_how=desc');
+    mountTable('/?order_by=host_count&order_how=desc');
 
-    checkSorting('Total systems', 'descending', 'host_ids');
+    checkSorting('Total systems', 'descending', 'host_count');
   });
 });
 
@@ -194,13 +194,15 @@ describe('sorting', () => {
     cy.wait('@getGroups'); // first initial request
   });
 
-  _.zip(['name', 'host_ids'], SORTABLE_HEADERS).forEach(([category, label]) => {
-    SORTING_ORDERS.forEach((order) => {
-      it(`${order} by ${label}`, () => {
-        checkSorting(label, order, category);
+  _.zip(['name', 'host_count'], SORTABLE_HEADERS).forEach(
+    ([category, label]) => {
+      SORTING_ORDERS.forEach((order) => {
+        it(`${order} by ${label}`, () => {
+          checkSorting(label, order, category);
+        });
       });
-    });
-  });
+    }
+  );
 });
 
 describe('filtering', () => {

diff --git a/src/components/GroupsTable/GroupsTable.js b/src/components/GroupsTable/GroupsTable.js
@@ -70,7 +70,7 @@ const GROUPS_TABLE_COLUMNS = [
 const GROUPS_TABLE_COLUMNS_TO_URL = {
   0: '', // reserved for selection boxes
   1: 'name',
-  2: 'host_ids',
+  2: 'host_count',
   3: 'updated_at',
 };