Chat Suggestions: Clean suggestions when there are no items (DevExpress#33467)

Co-authored-by: marker dao ® <youdontknow@marker-dao.eth>

Author: marker-dao

packages/devextreme/js/__internal/ui/chat/suggestions.ts

@@ -40,18 +40,18 @@ class Suggestions {
   }
 
   private _initButtonGroup(options: SuggestionsOptions = {}): void {
-    if (this._hasOptions(options) && this._$element) {
+    if (this._hasItems(options) && this._$element) {
       this._$element.addClass(CHAT_SUGGESTIONS_CLASS);
       this._buttonGroup = new ButtonGroup(this._$element.get(0), this._getConfiguration(options));
     }
   }
 
-  private _hasOptions(options: SuggestionsOptions | undefined): boolean {
-    return Boolean(Object.keys(options ?? {}).length);
+  private _hasItems(options: SuggestionsOptions | undefined): boolean {
+    return Boolean(options?.items?.length);
   }
 
   updateOptions(options: SuggestionsOptions | undefined): void {
-    if (!this._hasOptions(options)) {
+    if (!this._hasItems(options)) {
       this.clean();
       return;
     }

packages/devextreme/package.json

@@ -243,6 +243,7 @@
     "build:community-localization": "gulp generate-community-locales",
     "build:systemjs": "gulp transpile-systemjs",
     "dev": "cross-env DEVEXTREME_TEST_CI=true gulp dev",
+    "dev:playground": "vite",
     "dev:watch": "cross-env DEVEXTREME_TEST_CI=true gulp dev-watch",
     "transpile-tests": "gulp transpile-tests",
     "update-ts-reexports": "dx-tools generate-reexports --sources ./js --exclude \"((dialog|export|list_light|notify|overlay|palette|set_template_engine|splitter_control|themes|themes_callback|track_bar|utils|validation_engine|validation_message)[.d.ts])\" --compiler-options \"{ \\\"typeRoots\\\": [] }\"",
@@ -251,7 +252,6 @@
     "validate-ts": "gulp validate-ts",
     "validate-declarations": "dx-tools validate-declarations --sources ./js --exclude \"js/(renovation|__internal|.eslintrc.js)\" --compiler-options \"{ \\\"typeRoots\\\": [] }\"",
     "testcafe-in-docker": "docker build -f ./testing/testcafe/docker/Dockerfile -t testcafe-testing . && docker run -it testcafe-testing",
-    "dev:playground": "vite",
     "test-jest": "cross-env NODE_OPTIONS='--expose-gc' jest --no-coverage --runInBand --selectProjects jsdom-tests",
     "test-jest:watch": "jest --watch",
     "test-jest:node": "jest --no-coverage --runInBand --selectProjects node-tests",

packages/devextreme/testing/tests/DevExpress.ui.widgets/chatParts/suggestions.tests.js

@@ -53,12 +53,11 @@ QUnit.module('Suggestions', moduleConfig, () => {
             assert.strictEqual(this.getItems().length, 1, 'item rendered correctly');
         });
 
-        QUnit.test('should update items to empty list', function(assert) {
+        QUnit.test('should clean ButtonGroup when items is empty array', function(assert) {
             this.instance.updateOptions({ items: [] });
 
-            const $items = this.getItems();
-
-            assert.strictEqual($items.length, 0, 'all items removed');
+            assert.strictEqual(this.getItems().length, 0, 'all items removed');
+            assert.strictEqual(this.getSuggestions().hasClass(BUTTON_GROUP_CLASS), false, 'ButtonGroup is cleaned');
         });
 
         QUnit.test('should clean ButtonGroup when called with undefined', function(assert) {