test(ngMock): fix the tests to not use global msie

vojtajina · vojtajina · commit affcbad501ad · 2014-01-06T19:08:05.000-08:00
My bad when merging 7e91645. These tests are run with compiled Angular and then the msie is not defined.
diff --git a/test/ngMock/angular-mocksSpec.js b/test/ngMock/angular-mocksSpec.js
@@ -863,22 +863,30 @@ describe('ngMock', function() {
         });
       });
 
+
       // We don't run the following tests on IE8.
       // IE8 throws "Object does not support this property or method." error,
       // when thrown from a function defined on window (which `inject` is).
-      if (msie <= 8) return;
 
-      it('should not change thrown Errors', function() {
-        expect(function(){
-          throw new Error('test message');
+      it('should not change thrown Errors', inject(function($sniffer) {
+        if ($sniffer.msie <= 8) return;
+
+        expect(function() {
+          inject(function() {
+            throw new Error('test message');
+          });
         }).toThrow('test message');
-      });
+      }));
 
-      it('should not change thrown strings', function(){
-        expect(function(){
-          throw 'test message';
+      it('should not change thrown strings', inject(function($sniffer) {
+        if ($sniffer.msie <= 8) return;
+
+        expect(function() {
+          inject(function() {
+            throw 'test message';
+          });
         }).toThrow('test message');
-      });
+      }));
     });
   });
 
