fixup! fix(cdk-experimental/ui-patterns): listbox selection fixes

Author: wagnermaciel

src/cdk-experimental/ui-patterns/behaviors/list-selection/list-selection.spec.ts

@@ -212,7 +212,7 @@ describe('List Selection', () => {
       selection.selectFromPrevSelectedItem(); // [3, 1, 2]
 
       // TODO(wagnermaciel): Order the values when inserting them.
-      expect(selection.inputs.value()).toEqual([1, 2, 3]);
+      expect(selection.inputs.value()).toEqual([3, 1, 2]);
     });
   });
 });

src/cdk-experimental/ui-patterns/behaviors/list-selection/list-selection.ts

@@ -60,14 +60,7 @@ export class ListSelection<T extends ListSelectionItem<V>, V> {
 
     // TODO: Need to discuss when to drop this.
     this._anchor();
-
-    const orderedValues = [];
-    for (const {value} of this.inputs.items()) {
-      if (this.inputs.value().includes(value()) || item.value() === value()) {
-        orderedValues.push(value());
-      }
-    }
-    this.inputs.value.set(orderedValues);
+    this.inputs.value.update(values => values.concat(item.value()));
   }
 
   /** Deselects the item at the current active index. */

src/cdk-experimental/ui-patterns/listbox/listbox.spec.ts

@@ -248,7 +248,7 @@ describe('Listbox Pattern', () => {
         listbox.onKeydown(down());
         listbox.onKeydown(up({shift: true}));
         listbox.onKeydown(up({shift: true}));
-        expect(listbox.inputs.value()).toEqual(['Apple', 'Apricot']);
+        expect(listbox.inputs.value()).toEqual(['Apricot', 'Apple']);
       });
 
       it('should select contiguous items from the most recently selected item to the focused item on Shift + Space (or Enter)', () => {
@@ -331,7 +331,7 @@ describe('Listbox Pattern', () => {
         listbox.onKeydown(down());
         listbox.onKeydown(up({shift: true}));
         listbox.onKeydown(up({shift: true}));
-        expect(listbox.inputs.value()).toEqual(['Apple', 'Apricot', 'Banana']);
+        expect(listbox.inputs.value()).toEqual(['Banana', 'Apricot', 'Apple']);
       });
 
       it('should select contiguous items from the most recently selected item to the focused item on Shift + Space (or Enter)', () => {
@@ -349,7 +349,7 @@ describe('Listbox Pattern', () => {
         listbox.onKeydown(down({control: true}));
         listbox.onKeydown(down());
         listbox.onKeydown(home({control: true, shift: true}));
-        expect(listbox.inputs.value()).toEqual(['Apple', 'Apricot', 'Banana', 'Blackberry']);
+        expect(listbox.inputs.value()).toEqual(['Blackberry', 'Apple', 'Apricot', 'Banana']);
       });
 
       it('should select the focused option and all options down to the last option on Ctrl + Shift + End', () => {