BUG REPRO: ie8 bug when module() inside it() throws and exception (thrown during in...

[chirayuk]

ie8 bug when module() inside it() throws and exception (thrown during inject() call)

This is a bug where my exception object is getting lost. Here's the
result from a Karma trace:

Running "autotest:jqlite" (autotest) task
INFO [karma]: Karma v0.11.0 server started at http://localhost:9876/
WARN [web-server]: 404: /favicon.ico
INFO [IE 8.0.0 (Windows XP)]: Connected on socket hfE_1IJ4LeFwf6RJcVOe
INFO [watcher]: Changed file "/Users/chirayu/work/angular.js/test/ng/sceSpecs.js".
INFO [karma]: Delaying execution, these browsers are not ready: IE 8.0.0 (Windows XP)
IE 8.0.0 (Windows XP) SCE CKCK: IE8 TypeError FAILED
        Expected function to throw CKCK: foo , but it threw Object doesn't support this property or method
IE 8.0.0 (Windows XP): Executed 1 of 2225 (1 FAILED) (skipped 2224) ERROR (7.738 secs / 0.031 secs)

I don't see the real exception but get a TypeError. There is only one
stack frame between when the exception object is the one I want and the
one where it's been replaced by TypeError. No code executes between
those two frames afaik from looking at the IE8 debugger.

In the inject() call:

  if (!injector) {
    injector = currentSpec.$injector = angular.injector(modules);
  }

the exception is thrown in "injector = currentSpec.$injector = angular.injector(modules);"

making injector undefined. The caller frame is the anonymous function
in the expect() line:

expect(function() { inject(angular.nop); }).toThrow('CKCK: foo');

When control reaches that frame through, the exception object is no
longer "CKCK: foo" but a TypeError. :(

[mary-poppins]

OMG Chirayu!  Have you lost your marbles? What kind of a PR is this?? I flatly disapprove!

But … on second thoughts, as long as you promise not to merge it in, and since you're such a great guy, feel free to continue to fool around.  👯

[chirayuk]

Simplest version – see screenshot at http://i.imgur.com/dwj6tWy.png.