test(event): parameters for onSelect callback

Author: dimirc

test/select.spec.js

@@ -460,29 +460,54 @@ describe('ui-select tests', function() {
 
   it('should invoke select callback on select', function () {
 
-      scope.onSelectFn = function ($item, $model, $label) {
-        scope.$item = $item;
-        scope.$model = $model;
-      };
-      var el = compileTemplate(
-        '<ui-select on-select="onSelectFn($item, $model)" ng-model="selection.selected"> \
-          <ui-select-match placeholder="Pick one...">{{$select.selected.name}}</ui-select-match> \
-          <ui-select-choices repeat="person.name 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.onSelectFn = function ($item, $model, $label) {
+      scope.$item = $item;
+      scope.$model = $model;
+    };
+    var el = compileTemplate(
+      '<ui-select on-select="onSelectFn($item, $model)" ng-model="selection.selected"> \
+        <ui-select-match placeholder="Pick one...">{{$select.selected.name}}</ui-select-match> \
+        <ui-select-choices repeat="person.name 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>'
+    );
+
+    expect(scope.$item).toBeFalsy();
+    expect(scope.$model).toBeFalsy();
 
-      expect(scope.$item).toBeFalsy();
-      expect(scope.$model).toBeFalsy();
+    clickItem(el, 'Samantha');
+    expect(scope.selection.selected).toBe('Samantha');
 
-      clickItem(el, 'Samantha');
-      expect(scope.selection.selected).toBe('Samantha');
+    expect(scope.$item).toEqual(scope.people[5]);
+    expect(scope.$model).toEqual('Samantha');
 
-      expect(scope.$item).toEqual(scope.people[5]);
-      expect(scope.$model).toEqual('Samantha');
+  });
 
-    });
+  it('should set $item & $model correctly when invoking callback on select and no single prop. binding', function () {
+
+    scope.onSelectFn = function ($item, $model, $label) {
+      scope.$item = $item;
+      scope.$model = $model;
+    };
+
+    var el = compileTemplate(
+      '<ui-select on-select="onSelectFn($item, $model)" ng-model="selection.selected"> \
+        <ui-select-match placeholder="Pick one...">{{$select.selected.name}}</ui-select-match> \
+        <ui-select-choices repeat="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>'
+    );
+
+    expect(scope.$item).toBeFalsy();
+    expect(scope.$model).toBeFalsy();
+
+    clickItem(el, 'Samantha');
+    expect(scope.$item).toEqual(scope.$model);
+
+  });
 
 });