From fa921073f8716d87052549af457f63a027cc77c7 Mon Sep 17 00:00:00 2001
From: Melanie Sumner <melaniersumner@gmail.com>
Date: Tue, 5 Aug 2025 19:05:31 -0500
Subject: [PATCH] tests pass

---
 .../src/components/hds/dropdown/toggle/icon.ts    | 15 ++++++++-------
 .../components/hds/dropdown/toggle/icon-test.js   |  8 ++++----
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/packages/components/src/components/hds/dropdown/toggle/icon.ts b/packages/components/src/components/hds/dropdown/toggle/icon.ts
index 764c38004ea..26fb04b5950 100644
--- a/packages/components/src/components/hds/dropdown/toggle/icon.ts
+++ b/packages/components/src/components/hds/dropdown/toggle/icon.ts
@@ -21,7 +21,7 @@ import type Owner from '@ember/owner';
 export const DEFAULT_SIZE = HdsDropdownToggleIconSizeValues.Medium;
 export const SIZES: string[] = Object.values(HdsDropdownToggleIconSizeValues);
 
-export const ALLOWED_LIST: string[] = Object.values(
+export const ALLOWED_ICON_LIST: string[] = Object.values(
   HdsDropdownToggleIconAllowedListValues
 );
 
@@ -119,14 +119,15 @@ export default class HdsDropdownToggleIcon extends Component<HdsDropdownToggleIc
    * @default true
    */
   get hasChevron(): boolean {
-    const { icon, hasChevron } = this.args;
-
-    if (icon && !ALLOWED_LIST.includes(icon) && hasChevron === false) {
+    if (
+      this.args.icon &&
+      !ALLOWED_ICON_LIST.includes(this.args.icon) &&
+      this.args.hasChevron === false
+    ) {
       assert(
-        `@hasChevron for "Hds::Dropdown::Toggle::Icon" must be true unless the icon is one of the following: ${ALLOWED_LIST.join(
+        `@hasChevron for "Hds::Dropdown::Toggle::Icon" must be true unless the icon is one of the following: ${ALLOWED_ICON_LIST.join(
           ', '
-        )}; received: ${icon}`,
-        ALLOWED_LIST.includes(icon)
+        )}; received: ${this.args.icon}`
       );
     }
 
diff --git a/showcase/tests/integration/components/hds/dropdown/toggle/icon-test.js b/showcase/tests/integration/components/hds/dropdown/toggle/icon-test.js
index 1fb40dd0532..da588ef9110 100644
--- a/showcase/tests/integration/components/hds/dropdown/toggle/icon-test.js
+++ b/showcase/tests/integration/components/hds/dropdown/toggle/icon-test.js
@@ -69,7 +69,7 @@ module('Integration | Component | hds/dropdown/toggle/icon', function (hooks) {
   });
   test('toggle-icon renders no chevron when hasChevron is set to false', async function (assert) {
     await render(
-      hbs`<Hds::Dropdown::Toggle::Icon @icon="user" @text="user menu" id="test-toggle-icon" @hasChevron={{false}} />`,
+      hbs`<Hds::Dropdown::Toggle::Icon @icon="more-horizontal" @text="user menu" id="test-toggle-icon" @hasChevron={{false}} />`,
     );
     assert.dom('.hds-icon.hds-icon-chevron-down').doesNotExist();
   });
@@ -155,15 +155,15 @@ module('Integration | Component | hds/dropdown/toggle/icon', function (hooks) {
       throw new Error(errorMessage);
     });
   });
-  test('it should throw an assertion if an incorrect value for @icon is provided and @hasChevron is false', async function (assert) {
+  test('it should throw an assertion if an incorrect value for @icon is provided while @hasChevron is false', async function (assert) {
     const errorMessage =
-      '@hasChevron for "Hds::Dropdown::Toggle::Icon" must be true unless the icon is one of the following: more-horizontal, more-vertical; received: user';
+      '@hasChevron for "Hds::Dropdown::Toggle::Icon" must be true unless the icon is one of the following: more-horizontal, more-vertical; received: apple';
     assert.expect(2);
     setupOnerror(function (error) {
       assert.strictEqual(error.message, `Assertion Failed: ${errorMessage}`);
     });
     await render(
-      hbs`<Hds::Dropdown::Toggle::Icon @icon="user" @text="user menu" @hasChevron={{false}}/>`,
+      hbs`<Hds::Dropdown::Toggle::Icon @icon="apple" @text="user menu" @hasChevron={{false}}/>`,
     );
     assert.throws(function () {
       throw new Error(errorMessage);