fixup

- remove unused deps
- clarify variables
- simplify error expects

Author: Narretz

test/ng/compileSpec.js

@@ -1219,13 +1219,13 @@ describe('$compile', function() {
             {
               'no root element': 'dada',
               'multiple root elements': '<div></div><div></div>'
-            }, function($prop) {
+            }, function(directiveTemplate) {
 
-              inject(function($compile, $templateCache, $rootScope) {
-                templateVar = $prop;
+              inject(function($compile) {
+                templateVar = directiveTemplate;
                 expect(function() {
                   $compile('<p template></p>');
-                }).toThrowMinErr('$compile', 'tplrt', 'Template for directive \'template\' must have exactly one root element.');
+                }).toThrowMinErr('$compile', 'tplrt');
               });
           });
 
@@ -1234,14 +1234,14 @@ describe('$compile', function() {
               'whitespace': '  <div>Hello World!</div> \n',
               'comments': '<!-- oh hi --><div>Hello World!</div> \n',
               'comments + whitespace': '  <!-- oh hi -->  <div>Hello World!</div>  <!-- oh hi -->\n'
-            }, function($prop) {
+            }, function(directiveTemplate) {
 
-              inject(function($compile, $templateCache, $rootScope) {
-                templateVar = $prop;
+              inject(function($compile, $rootScope) {
+                templateVar = directiveTemplate;
                 var element;
                 expect(function() {
                   element = $compile('<p template></p>')($rootScope);
-                }).not.toThrowMinErr('$compile', 'tplrt', 'Template for directive \'template\' must have exactly one root element.');
+                }).not.toThrow();
                 expect(element.length).toBe(1);
                 expect(element.text()).toBe('Hello World!');
               });
@@ -2067,15 +2067,14 @@ describe('$compile', function() {
             {
               'no root element': 'dada',
               'multiple root elements': '<div></div><div></div>'
-            }, function($prop) {
+            }, function(directiveTemplate) {
 
               inject(function($compile, $templateCache, $rootScope) {
-                // no root element
-                $templateCache.put('template.html', $prop);
+                $templateCache.put('template.html', directiveTemplate);
                 $compile('<p template></p>')($rootScope);
                 expect(function() {
                   $rootScope.$digest();
-                }).toThrowMinErr('$compile', 'tplrt', 'Template for directive \'template\' must have exactly one root element. template.html');
+                }).toThrowMinErr('$compile', 'tplrt');
               });
           });
 
@@ -2084,15 +2083,14 @@ describe('$compile', function() {
               'whitespace': '  <div>Hello World!</div> \n',
               'comments': '<!-- oh hi --><div>Hello World!</div> \n',
               'comments + whitespace': '  <!-- oh hi -->  <div>Hello World!</div>  <!-- oh hi -->\n'
-            }, function($prop) {
+            }, function(directiveTemplate) {
 
               inject(function($compile, $templateCache, $rootScope) {
-                // no root element
-                $templateCache.put('template.html', $prop);
+                $templateCache.put('template.html', directiveTemplate);
                 element = $compile('<p template></p>')($rootScope);
                 expect(function() {
                   $rootScope.$digest();
-                }).not.toThrowMinErr('$compile', 'tplrt', 'Template for directive \'template\' must have exactly one root element. template.html');
+                }).not.toThrow();
                 expect(element.length).toBe(1);
                 expect(element.text()).toBe('Hello World!');
               });