fix(app): enforce new scope for each brick

Author: passy

src/angular-masonry.js

@@ -23,7 +23,7 @@
         if (!found) {
           this.scheduleMasonry.apply(null, arguments);
         }
-      }
+      };
 
       // Make sure it's only executed once within a reasonable time-frame in
       // case multiple elements are removed or added at once.
@@ -43,7 +43,7 @@
           });
           schedule = [];
         }, 30);
-      }
+      };
 
       function defaultLoaded($element) {
         $element.addClass('loaded');
@@ -116,6 +116,7 @@
       return {
         restrict: 'AC',
         require: '^masonry',
+        scope: true,
         link: function postLink(scope, element, attrs, ctrl) {
           var id = scope.$id;
 

test/spec/directive.coffee

@@ -114,3 +114,24 @@ describe 'angular-masonry', ->
       expect(@appendBrick).toHaveBeenCalledThrice()
       expect(@removeBrick).toHaveBeenCalledOnce()
     )
+
+  describe 'masonry-brick internals', =>
+    beforeEach ->
+      $.fn.imagesLoaded = (cb) -> cb()
+
+    afterEach ->
+      delete $.fn.imagesLoaded
+
+    it 'should append three elements to the controller', inject(($compile) =>
+      element = angular.element '''
+        <masonry>
+          <div class="masonry-brick"></div>
+          <div class="masonry-brick"></div>
+          <div class="masonry-brick"></div>
+        </masonry>
+      '''
+      element = $compile(element)(@scope)
+      @scope.$digest()
+      # 2 is resize and one layout, 3 for the elements
+      expect($.fn.masonry.callCount).toBe(2 + 3)
+    )