From bd530e225750e9b30b6300fc3d7447a5f1071667 Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Wed, 2 May 2012 16:34:54 -0700 Subject: chore($compile): remove obsolete <> transclusion This stuff was never documented and is an accidental leftover from the time when the compiler was rewritten. If any code depends on this, it should be rewritten to use ngTransclude directive intead. --- test/ng/compileSpec.js | 54 ++++++++++++++++++++++++++++---------------------- 1 file changed, 30 insertions(+), 24 deletions(-) (limited to 'test') diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js index 54837bf7..74eaec8a 100644 --- a/test/ng/compileSpec.js +++ b/test/ng/compileSpec.js @@ -364,7 +364,7 @@ describe('$compile', function() { $compileProvider.directive('replace', valueFn({ restrict: 'CAM', replace: true, - template: '
Hello: <>
', + template: '
Replace!
', compile: function(element, attr) { attr.$set('compiled', 'COMPILED'); expect(element).toBe(attr.$$element); @@ -372,7 +372,7 @@ describe('$compile', function() { })); $compileProvider.directive('append', valueFn({ restrict: 'CAM', - template: '
Hello: <>
', + template: '
Append!
', compile: function(element, attr) { attr.$set('compiled', 'COMPILED'); expect(element).toBe(attr.$$element); @@ -382,34 +382,34 @@ describe('$compile', function() { it('should replace element with template', inject(function($compile, $rootScope) { - element = $compile('
content
')($rootScope); - expect(element.text()).toEqual('Hello: content'); + element = $compile('
ignore
')($rootScope); + expect(element.text()).toEqual('Replace!'); expect(element.find('div').attr('compiled')).toEqual('COMPILED'); })); it('should append element with template', inject(function($compile, $rootScope) { - element = $compile('
content
')($rootScope); - expect(element.text()).toEqual('Hello: content'); + element = $compile('
ignore
')($rootScope); + expect(element.text()).toEqual('Append!'); expect(element.find('div').attr('compiled')).toEqual('COMPILED'); })); - it('should compile replace template', inject(function($compile, $rootScope, log) { - element = $compile('
{{ "angular" }}
') + it('should compile template when replacing', inject(function($compile, $rootScope, log) { + element = $compile('
ignore
') ($rootScope); $rootScope.$digest(); - expect(element.text()).toEqual('Hello: angular'); + expect(element.text()).toEqual('Replace!'); // HIGH goes after MEDIUM since it executes as part of replaced template expect(log).toEqual('MEDIUM; HIGH; LOG'); })); - it('should compile append template', inject(function($compile, $rootScope, log) { - element = $compile('
{{ "angular" }}
') + it('should compile template when appending', inject(function($compile, $rootScope, log) { + element = $compile('
ignore
') ($rootScope); $rootScope.$digest(); - expect(element.text()).toEqual('Hello: angular'); + expect(element.text()).toEqual('Append!'); expect(log).toEqual('HIGH; LOG; MEDIUM'); })); @@ -436,23 +436,23 @@ describe('$compile', function() { } })); - it('should play nice with repeater when inline', inject(function($compile, $rootScope) { + it('should play nice with repeater when replacing', inject(function($compile, $rootScope) { element = $compile( '
' + - '
{{i}};
' + + '
' + '
')($rootScope); $rootScope.$digest(); - expect(element.text()).toEqual('Hello: 1; Hello: 2; '); + expect(element.text()).toEqual('Replace!Replace!'); })); - it('should play nice with repeater when append', inject(function($compile, $rootScope) { + it('should play nice with repeater when appending', inject(function($compile, $rootScope) { element = $compile( '
' + - '
{{i}};
' + + '
' + '
')($rootScope); $rootScope.$digest(); - expect(element.text()).toEqual('Hello: 1; Hello: 2; '); + expect(element.text()).toEqual('Append!Append!'); })); @@ -494,8 +494,12 @@ describe('$compile', function() { beforeEach(module( function($compileProvider) { - $compileProvider.directive('hello', valueFn({ restrict: 'CAM', templateUrl: 'hello.html' })); - $compileProvider.directive('cau', valueFn({ restrict: 'CAM', templateUrl:'cau.html' })); + $compileProvider.directive('hello', valueFn({ + restrict: 'CAM', templateUrl: 'hello.html', transclude: true + })); + $compileProvider.directive('cau', valueFn({ + restrict: 'CAM', templateUrl:'cau.html' + })); $compileProvider.directive('cError', valueFn({ restrict: 'CAM', @@ -930,9 +934,10 @@ describe('$compile', function() { })); - it('should work when widget is in root element', inject( + it('should work when directive is on the root element', inject( function($compile, $httpBackend, $rootScope) { - $httpBackend.expect('GET', 'hello.html').respond('3==<>'); + $httpBackend.expect('GET', 'hello.html'). + respond('3=='); element = jqLite('{{1+2}}'); $compile(element)($rootScope); @@ -942,9 +947,10 @@ describe('$compile', function() { )); - it('should work when widget is a repeater', inject( + it('should work when directive is a repeater', inject( function($compile, $httpBackend, $rootScope) { - $httpBackend.expect('GET', 'hello.html').respond('i=<>;'); + $httpBackend.expect('GET', 'hello.html'). + respond('i=;'); element = jqLite('
{{i}}
'); $compile(element)($rootScope); -- cgit v1.2.3