Merge pull request #1508 from mshibuya/refresh-choices-on-selection-change

fix(uiSelectMultipleDirective): refresh choices upon selection change

Closes #1243

Author: aaronroberson

Diff for: src/uiSelectMultipleDirective.js

@@ -162,6 +162,7 @@ uis.directive('uiSelectMultiple', ['uiSelectMinErr','$timeout', function(uiSelec
           }
         }
         $select.selected = ngModel.$viewValue;
+        $selectMultiple.refreshComponent();
         scope.$evalAsync(); //To force $digest
       };
 

Diff for: test/select.spec.js

@@ -2034,6 +2034,30 @@ describe('ui-select tests', function() {
 
       });
 
+      it('should watch changes for $select.selected and refresh choices correctly', function () {
+
+          scope.selection.selectedMultiple = ['[email protected]', '[email protected]'];
+
+          var el = compileTemplate(
+              '<ui-select multiple ng-model="selection.selectedMultiple" theme="bootstrap" style="width: 800px;"> \
+                  <ui-select-match placeholder="Pick one...">{{$item.name}} &lt;{{$item.email}}&gt;</ui-select-match> \
+                  <ui-select-choices repeat="person.email as person in people | filter: $select.search"> \
+                    <div ng-bind-html="person.name | highlight: $select.search"></div> \
+                    <div ng-bind-html="person.email | highlight: $select.search"></div> \
+                  </ui-select-choices> \
+              </ui-select> \
+              '
+          );
+          scope.selection.selectedMultiple.splice(0, 1); // Remove Wladimir from selection
+
+          var searchInput = el.find('.ui-select-search');
+          triggerKeydown(searchInput, Key.Down); //Open dropdown
+
+          expect(el.find('.ui-select-choices-content').text())
+              .toContain("[email protected]");
+
+      });
+
       it('should ensure the multiple selection limit is respected', function () {
 
           scope.selection.selectedMultiple = ['[email protected]'];