Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit c25ee8d

Browse files
committed
fixup! perf(ngStyleDirective): use $watchCollection
1 parent b5b798b commit c25ee8d

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

test/ng/directive/ngStyleSpec.js

+15
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,21 @@ describe('ngStyle', function() {
2323
}));
2424

2525

26+
it('should not deep watch objects', inject(function($rootScope, $compile) {
27+
element = $compile('<div ng-style="{height: heightObj}"></div>')($rootScope);
28+
$rootScope.$digest();
29+
expect(parseInt(element.css('height') + 0, 10)).toEqual(0); // height could be '' or '0px'
30+
$rootScope.heightObj = {toString: function() { return '40px'; }};
31+
$rootScope.$digest();
32+
expect(element.css('height')).toBe('40px');
33+
34+
element.css('height', '10px');
35+
$rootScope.heightObj.otherProp = 123;
36+
$rootScope.$digest();
37+
expect(element.css('height')).toBe('10px');
38+
}));
39+
40+
2641
it('should support binding for object literals', inject(function($rootScope, $compile) {
2742
element = $compile('<div ng-style="{height: heightStr}"></div>')($rootScope);
2843
$rootScope.$digest();

0 commit comments

Comments
 (0)