Skip to content

Commit 726e0c2

Browse files
committed
test(ngRepeat): disable an element directive test on IE8
1 parent c46c592 commit 726e0c2

File tree

1 file changed

+19
-15
lines changed

1 file changed

+19
-15
lines changed

test/ng/directive/ngRepeatSpec.js

+19-15
Original file line numberDiff line numberDiff line change
@@ -523,21 +523,25 @@ describe('ngRepeat', function() {
523523
}));
524524

525525

526-
it('should work when placed on a root element of element directive with ASYNC replaced template',
527-
inject(function($templateCache, $compile, $rootScope) {
528-
$compileProvider.directive('replaceMeWithRepeater', function() {
529-
return {
530-
restrict: 'E',
531-
replace: true,
532-
templateUrl: 'replace-me-with-repeater.html'
533-
};
534-
});
535-
$templateCache.put('replace-me-with-repeater.html', '<div ng-repeat="i in [1,2,3]">{{i}}</div>');
536-
element = $compile('<div><replace-me-with-repeater></replace-me-with-repeater></div>')($rootScope);
537-
expect(element.text()).toBe('');
538-
$rootScope.$apply();
539-
expect(element.text()).toBe('123');
540-
}));
526+
if (!msie || msie > 8) {
527+
// only IE>8 supports element directives
528+
529+
it('should work when placed on a root element of element directive with ASYNC replaced template',
530+
inject(function($templateCache, $compile, $rootScope) {
531+
$compileProvider.directive('replaceMeWithRepeater', function() {
532+
return {
533+
restrict: 'E',
534+
replace: true,
535+
templateUrl: 'replace-me-with-repeater.html'
536+
};
537+
});
538+
$templateCache.put('replace-me-with-repeater.html', '<div ng-repeat="i in [1,2,3]">{{i}}</div>');
539+
element = $compile('<div><replace-me-with-repeater></replace-me-with-repeater></div>')($rootScope);
540+
expect(element.text()).toBe('');
541+
$rootScope.$apply();
542+
expect(element.text()).toBe('123');
543+
}));
544+
}
541545
});
542546

543547

0 commit comments

Comments
 (0)