Update tests

Author: Xon

test-e2e/tests/select-multiple.spec.ts

@@ -246,13 +246,15 @@ describe(`Choices - select multiple`, () => {
           for (let i = 1; i < count + 1; i++) {
             await suite.expectVisibleDropdown();
             await suite.getChoiceWithText(`Choice ${i}`).click();
-            await expect(suite.getChoiceWithText(`Choice ${i}`)).toHaveClass(/is-selected/);
             await suite.advanceClock();
             await suite.expectedItemCount(i);
-            await expect(suite.selectableChoices).toHaveCount(count);
+            if (i < count) {
+              await expect(suite.getChoiceWithText(`Choice ${i}`)).toHaveClass(/is-selected/);
+              await expect(suite.selectableChoices).toHaveCount(count);
+            } else {
+              await suite.expectVisibleNoticeHtml('No choices to choose from');
+            }
           }
-
-          await suite.expectVisibleNoticeHtml('No choices to choose from');
         });
       });
 

test/scripts/choices.test.ts

@@ -45,6 +45,7 @@ describe('choices', () => {
             ...DEFAULT_CONFIG,
             searchEnabled: false,
             closeDropdownOnSelect: true,
+            renderSelectedChoices: false,
           });
         });
       });
@@ -65,6 +66,7 @@ describe('choices', () => {
             ...DEFAULT_CONFIG,
             searchEnabled: false,
             closeDropdownOnSelect: true,
+            renderSelectedChoices: false,
             ...config,
           });
         });
@@ -109,7 +111,7 @@ describe('choices', () => {
               renderSelectedChoices: 'test' as any,
             });
 
-            expect(instance.config.renderSelectedChoices).to.equal('auto');
+            expect(instance.config.renderSelectedChoices).to.equal(false);
           });
         });
       });

test/scripts/templates.test.ts

@@ -11,13 +11,13 @@ import { NoticeTypes, Templates as TemplatesInterface } from '../../src/scripts/
  */
 function expectEqualElements(element1, element2): void {
   expect(element1.tagName).to.equal(element2.tagName);
-  expect(element1.attributes.length).to.equal(element2.attributes.length);
   expect(Object.keys(element1.dataset)).to.have.members(Object.keys(element2.dataset));
   expect(element1.classList).to.include(element2.classList);
   // compare attributes values
   for (const attribute of Object.values(element1.attributes)) {
     expect(element1.getAttribute(attribute)).to.equal(element2.getAttribute(attribute));
   }
+  expect(element1.attributes.length).to.equal(element2.attributes.length);
 }
 
 function createOptionsWithPartialClasses(classNames: Partial<ClassNames>, options: Partial<Options> = {}): Options {
@@ -563,7 +563,7 @@ describe('templates', () => {
             ${data.label}
           </div>
         `);
-        const actualOutput = templates.choice(choiceOptions, data, itemSelectText);
+        const actualOutput = templates.choice(choiceOptions, data, itemSelectText, "Group text");
 
         expectEqualElements(actualOutput, expectedOutput);
       });