Fixed tests

iancharlesdouglas · Apr 18, 2024 · 11de008 · 11de008
1 parent 94cdc25
commit 11de008
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/components/multi-select/multi-select.test.tsx b/src/components/multi-select/multi-select.test.tsx
@@ -337,7 +337,7 @@ describe('MultiSelect', () => {
             <MultiSelect
               items={items}
               selectedItems={[selectedOption.value!]}
-              onChange$={async (items: Item[], item: Item) => {
+              onChange$={async (items: Item[], item?: Item) => {
                 selectedOption.value = item;
                 selected.item = item;
                 expect(item).toEqual('Apple');
@@ -614,7 +614,7 @@ describe('MultiSelect', () => {
         <Form>
           <MultiSelect
             items={items}
-            onChange$={$((items: Item[], item: Item) => {
+            onChange$={$((items: Item[], item?: Item) => {
               selected.item = item;
             })}
           />

diff --git a/src/components/multi-select/multi-select.tsx b/src/components/multi-select/multi-select.tsx
@@ -129,10 +129,10 @@ export const MultiSelect = component$((props: MultiSelectProps) => {
     }
   });
 
-  useTask$(({ track }) => {
+  useTask$(async ({ track }) => {
     track(() => declaredSelectedItems);
     state.selectedItems = declaredSelectedItems;
-    sorted.initialized = false;
+    sorted.items = await sortItems$(items, state.selectedItems, selectionFeedback === 'fixed', sortOptions);
   });
 
   useTask$(({ track }) => {