revert: feat(ngMock): destroy $rootScope after each test

Matias Niemela · Matias Niemela · commit 2ef92c329bb7 · 2016-03-14T17:27:49.000-04:00
diff --git a/src/ngMock/angular-mocks.js b/src/ngMock/angular-mocks.js
@@ -2330,7 +2330,6 @@ if (window.jasmine || window.mocha) {
       }
       angular.element.cleanData(cleanUpNodes);
 
-      injector.get('$rootScope').$destroy();
     }
 
     // clean up jquery's fragment cache
diff --git a/test/ngMock/angular-mocksSpec.js b/test/ngMock/angular-mocksSpec.js
@@ -1603,25 +1603,6 @@ describe('ngMock', function() {
   });
 
 
-  describe('$rootScope', function() {
-    var destroyed = false;
-    var oldRootScope;
-
-    it('should destroy $rootScope after each test', inject(function($rootScope) {
-      $rootScope.$on('$destroy', function() {
-        destroyed = true;
-      });
-      oldRootScope = $rootScope;
-    }));
-
-    it('should have destroyed the $rootScope from the previous test', inject(function($rootScope) {
-      expect(destroyed).toBe(true);
-      expect($rootScope).not.toBe(oldRootScope);
-      expect(oldRootScope.$$destroyed).toBe(true);
-    }));
-  });
-
-
   describe('$rootScopeDecorator', function() {
 
     describe('$countChildScopes', function() {

Original file line number	Diff line number	Diff line change
`@@ -2330,7 +2330,6 @@ if (window.jasmine \|\| window.mocha) {`
`2330`	`2330`	`}`
`2331`	`2331`	`angular.element.cleanData(cleanUpNodes);`
`2332`	`2332`
`2333`		`- injector.get('$rootScope').$destroy();`
`2334`	`2333`	`}`
`2335`	`2334`
`2336`	`2335`	`// clean up jquery's fragment cache`