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'; | 
