Add test to check $setViewValue on watch (using 3rd parameter to be be compatible with v1.3.0-beta+)

Author: dimirc

Diff for: src/select.js

@@ -681,7 +681,9 @@
 
         if ($select.multiple){
           scope.$watchCollection('$select.selected', function(newValue) {
-            ngModel.$setViewValue(newValue, null, true); //Third parameter (revalidate) is true, to force $parsers to recreate model
+            //On v1.2.19 the 2nd and 3rd parameteres are ignored
+            //On v1.3.0-beta+ 3rd parameter (revalidate) is true, to force $parsers to recreate model
+            ngModel.$setViewValue(newValue, null, true);
           });
           focusser.prop('disabled', true); //Focusser isn't needed if multiple
         }else{

Diff for: test/select.spec.js

@@ -1077,6 +1077,35 @@ describe('ui-select tests', function() {
 
     });
 
+    it('should watch changes for $select.selected and update formatted value 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> \
+          '
+      );
+
+      var el2 = compileTemplate('<span class="resultDiv" ng-bind="selection.selectedMultiple"></span>');
+
+      expect(el.find('.ui-select-match-item [uis-transclude-append]:not(.ng-hide)').text())
+        .toBe("Wladimir <[email protected]>Samantha <[email protected]>");
+
+      clickItem(el, 'Nicole');
+
+      expect(el.find('.ui-select-match-item [uis-transclude-append]:not(.ng-hide)').text())
+        .toBe("Wladimir <[email protected]>Samantha <[email protected]>Nicole <[email protected]>");
+
+      expect(scope.selection.selectedMultiple.length).toBe(3);
+
+    });
+
   });