Merge pull request #161 from thgreasi/master

test(sortable.spec): add simple $apply test to increase code coverage.

Author: thgreasi

test/sortable.spec.js

@@ -4,6 +4,13 @@ describe('uiSortable', function() {
 
   // Ensure the sortable angular module is loaded
   beforeEach(module('ui.sortable'));
+  beforeEach(module('ui.sortable.testHelper'));
+
+  var listContent;
+
+  beforeEach(inject(function (sortableTestHelper) {
+    listContent = sortableTestHelper.listContent;
+  }));
 
   describe('Simple use', function() {
 
@@ -29,6 +36,24 @@ describe('uiSortable', function() {
       });
     });
 
+    it('should refresh sortable properly after an apply', function() {
+      inject(function($compile, $rootScope, $timeout) {
+        var element;
+        var childScope = $rootScope.$new();
+        element = $compile('<ul ui-sortable ng-model="items"><li ng-repeat="item in items">{{ item }}</li></ul>')(childScope);
+        $rootScope.$apply(function() {
+          childScope.items = ['One', 'Two', 'Three'];
+        });
+
+        expect(function() {
+          $timeout.flush();
+        }).not.toThrow();
+
+        expect(childScope.items).toEqual(['One', 'Two', 'Three']);
+        expect(childScope.items).toEqual(listContent(element));
+      });
+    });
+
     it('should not refresh sortable if destroyed', function() {
       inject(function($compile, $rootScope, $timeout) {
         var element;