diff options
| author | metaweta | 2013-01-29 11:34:14 -0800 |
|---|---|---|
| committer | Igor Minar | 2013-01-29 13:26:06 -0800 |
| commit | 3b317c5dcb93e1907fe6fae8f1e9b83c35a7a81f (patch) | |
| tree | 438db768644b8a78188090dd9ca61714828f4fb0 | |
| parent | e4cfb9d938c1d9cd08942d3192c281e16d2c200d (diff) | |
| download | angular.js-3b317c5dcb93e1907fe6fae8f1e9b83c35a7a81f.tar.bz2 | |
test(ngBindHtml): prevent variable name leak
Add "var" so element is local instead of global
Strict mode doesn't allow undeclared global vars, and these really should be local anyway.
| -rw-r--r-- | test/ngSanitize/directive/ngBindHtmlSpec.js | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/test/ngSanitize/directive/ngBindHtmlSpec.js b/test/ngSanitize/directive/ngBindHtmlSpec.js index be23e5a5..6759c6d2 100644 --- a/test/ngSanitize/directive/ngBindHtmlSpec.js +++ b/test/ngSanitize/directive/ngBindHtmlSpec.js @@ -1,8 +1,11 @@ +'use strict'; + + describe('ngBindHtml', function() { beforeEach(module('ngSanitize')); it('should set html', inject(function($rootScope, $compile) { - element = $compile('<div ng-bind-html="html"></div>')($rootScope); + var element = $compile('<div ng-bind-html="html"></div>')($rootScope); $rootScope.html = '<div unknown>hello</div>'; $rootScope.$digest(); expect(angular.lowercase(element.html())).toEqual('<div>hello</div>'); @@ -10,7 +13,7 @@ describe('ngBindHtml', function() { it('should reset html when value is null or undefined', inject(function($compile, $rootScope) { - element = $compile('<div ng-bind-html="html"></div>')($rootScope); + var element = $compile('<div ng-bind-html="html"></div>')($rootScope); angular.forEach([null, undefined, ''], function(val) { $rootScope.html = 'some val'; |
