From ae8a329df5056ad7ae25a40ecc40c20cca2f0d78 Mon Sep 17 00:00:00 2001
From: Conor Darcy <conor.darcy@genesys.com>
Date: Thu, 17 Oct 2024 11:32:11 +0100
Subject: [PATCH] feat(table): removing unnecessary method

---
 .../components/stable/gux-table/example.html  | 31 +------------------
 .../gux-all-row-select/gux-all-row-select.tsx |  6 ----
 .../tests/gux-all-row-select.spec.ts          | 18 -----------
 3 files changed, 1 insertion(+), 54 deletions(-)

diff --git a/packages/genesys-spark-components/src/components/stable/gux-table/example.html b/packages/genesys-spark-components/src/components/stable/gux-table/example.html
index 72ef01715..470579dbd 100644
--- a/packages/genesys-spark-components/src/components/stable/gux-table/example.html
+++ b/packages/genesys-spark-components/src/components/stable/gux-table/example.html
@@ -438,26 +438,11 @@ <h2>Data table with sortable columns</h2>
 </gux-table>
 
 <h2>Object table with rows selection</h2>
-<ul>
-  <li>
-    Use the <strong>setChecked</strong> method on
-    <strong>gux-all-row-select</strong> to programatically manage the selected
-    state.
-  </li>
-  <li>
-    Use the <strong>setIndeterminate</strong> method to set the
-    indeterminate/unknown state on <strong>gux-all-row-select</strong>. Remember
-    to set it to false when the checkbox is checked or unchecked as it overrides
-    these states.
-  </li>
-</ul>
-<gux-button id="selectAll">Select all</gux-button>
-<gux-button id="deselecAll">Deselect all</gux-button>
 <gux-table object-table selectable-rows>
   <table slot="data">
     <thead>
       <tr data-row-id="head">
-        <th><gux-all-row-select id="allRowSelect"></gux-all-row-select></th>
+        <th><gux-all-row-select></gux-all-row-select></th>
         <th data-column-name="first-name">First name</th>
         <th data-column-name="last-name">Last name</th>
         <th data-column-name="age" data-cell-numeric>Age</th>
@@ -986,20 +971,6 @@ <h2>Data table with fixed columns</h2>
 
 
     });
-
-  const selectAll = document.getElementById('selectAll');
-  const deselectAll = document.getElementById('deselecAll');
-  const allRowSelect = document.getElementById('allRowSelect');
-
-  selectAll.addEventListener('click', () => {
-    allRowSelect.setIndeterminate(false);
-    allRowSelect.setChecked(true);
-  });
-
-  deselectAll.addEventListener('click', () => {
-    allRowSelect.setIndeterminate(false);
-    allRowSelect.setChecked(false);
-  });
   })()"
 >
   .not-used {
diff --git a/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/gux-all-row-select.tsx b/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/gux-all-row-select.tsx
index 20c1fa135..7e1d316de 100644
--- a/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/gux-all-row-select.tsx
+++ b/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/gux-all-row-select.tsx
@@ -49,12 +49,6 @@ export class GuxAllRowSelect {
     this.inputElement.indeterminate = indeterminate;
   }
 
-  @Method()
-  async setChecked(checked: boolean): Promise<void> {
-    this.selected = checked;
-    this.internalallrowselectchange.emit(this.selected);
-  }
-
   async componentWillLoad(): Promise<void> {
     this.i18n = await buildI18nForComponent(
       this.root,
diff --git a/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/tests/gux-all-row-select.spec.ts b/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/tests/gux-all-row-select.spec.ts
index c1618ca9b..6c27a05ec 100644
--- a/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/tests/gux-all-row-select.spec.ts
+++ b/packages/genesys-spark-components/src/components/stable/gux-table/gux-all-row-select/tests/gux-all-row-select.spec.ts
@@ -27,22 +27,4 @@ describe('gux-all-row-select', () => {
     await component.setIndeterminate(false);
     expect(inputElement.indeterminate).toBe(false);
   });
-
-  it('should handle the setChecked method', async () => {
-    const html = '<gux-all-row-select></gux-all-row-select>';
-    const page = await newSpecPage({ components, html, language });
-
-    const component = page.rootInstance;
-    const inputElement = page.root.shadowRoot.querySelector('input');
-
-    expect(inputElement.checked).toBe(false);
-
-    await component.setChecked(true);
-    await page.waitForChanges();
-    expect(inputElement.checked).toBe(true);
-
-    await component.setChecked(false);
-    await page.waitForChanges();
-    expect(inputElement.checked).toBe(false);
-  });
 });