aboutsummaryrefslogtreecommitdiffstats
path: root/test/CompilerSpec.js
diff options
context:
space:
mode:
authorIgor Minar2011-10-20 15:37:37 -0700
committerIgor Minar2011-10-20 15:51:14 -0700
commitf38010d3a2f457a53798212ef72418637dabe189 (patch)
tree1825bd10dfbcc61f71c2f03d3cc3c90df9c987b1 /test/CompilerSpec.js
parent7fc18b263dc74f52bb677e446f23e35d64948841 (diff)
downloadangular.js-f38010d3a2f457a53798212ef72418637dabe189.tar.bz2
fix(compiler): revert 8611ebe6 - calling \$digest after linking
Change introduced by me in 8611ebe6 results in considerable inefficiencies when the compiler and linker is used from within a widget, in which case, we call $digest unnecessary since it will be called by the $apply which called the directive/widget in the first place. There are only two places when the extra $digest call can be useful - when manually bootstrapping the app or in tests. However even in tests this behavior can result in unwanted results (especially when ng:controller is involved). So it is better to leave it for the developer to call $digest when it is really needed.
Diffstat (limited to 'test/CompilerSpec.js')
-rw-r--r--test/CompilerSpec.js4
1 files changed, 0 insertions, 4 deletions
diff --git a/test/CompilerSpec.js b/test/CompilerSpec.js
index 2cb7d19d..860cea4a 100644
--- a/test/CompilerSpec.js
+++ b/test/CompilerSpec.js
@@ -110,7 +110,6 @@ describe('compiler', function() {
expect(sortedHtml(scope.$element)).
toEqual('<div>' +
'before<#comment></#comment>' +
- '<span>x</span>' +
'after' +
'</div>');
scope.value = 1;
@@ -119,7 +118,6 @@ describe('compiler', function() {
toEqual('<div>' +
'before<#comment></#comment>' +
'<span>x</span>' +
- '<span>x</span>' +
'after' +
'</div>');
scope.value = 2;
@@ -129,7 +127,6 @@ describe('compiler', function() {
'before<#comment></#comment>' +
'<span>x</span>' +
'<span>x</span>' +
- '<span>x</span>' +
'after' +
'</div>');
scope.value = 3;
@@ -140,7 +137,6 @@ describe('compiler', function() {
'<span>x</span>' +
'<span>x</span>' +
'<span>x</span>' +
- '<span>x</span>' +
'after' +
'</div>');
});