test(ngMocks): prevent Safari 10+ optimization that breaks test

gkalpak · gkalpak · commit 1d2f5e41b794 · 2016-12-07T21:37:36.000+02:00
diff --git a/test/ngMock/angular-mocksSpec.js b/test/ngMock/angular-mocksSpec.js
@@ -936,9 +936,10 @@ describe('ngMock', function() {
         })();
 
         function testCaller() {
-          return inject(function() {
-            throw new Error();
-          });
+          // Assigning to a local variable first, prevents Safari 10+ from inlining
+          // the function call and having it appear as anonymous in the stack trace.
+          var fn = inject(function() { throw new Error(); });
+          return fn;
         }
         var throwErrorFromInjectCallback = testCaller();
 
