Angular fails to instantiate module ngLocale in tests after upgrade to 1.3.12

[nadavsinai]

In our CI the tests failed, I tried to reproduce locally and could not until I removed our bower libs and reinstalled. (we were asking for ~1.3).
I played with bower.json angular versions and found that 1.3.11 works and 1.3.12 fails.

the code that breaks our app is found in commit 6ec5946

line 2138 in angular-mocks.js

 Error: [$injector:modulerr] Failed to instantiate module ng due to:
  Error: [$injector:modulerr] Failed to instantiate module ngLocale due to:
  TypeError: 'undefined' is not an object (evaluating 'annotatedFunctions.push')
  at /Volumes/OSxData/websites/startapp/portal/public/lib/angular-mocks/angular-mocks.js:2145
  at invoke (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4163)
  at instantiate (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4193)
  at provider (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4028)
  at /Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4020
  at runInvokeQueue (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4091)
  at /Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4099
  at forEach (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:323)
  at loadModules (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4123)
  at /Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4098
  at forEach (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:323)
  at loadModules (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4123)
  at createInjector (/Volumes/OSxData/websites/startapp/portal/public/lib/angular/angular.js:4007)
  at /Volumes/OSxData/websites/startapp/portal/public/lib/sinon-ng/sinon-ng.js:4
  at /Volumes/OSxData/websites/startapp/portal/public/lib/sinon-ng/sinon-ng.js:30

this occurs in angular-mocks therefore relevant only in testing environment

Reproducable: always
Browsers : Chrome 40, Phantom JS
Operating system: OSX 10.10, Ubuntu 14, Fedora 20

[kuoloneous]

Also having the same issue.

[kodypeterson]

+1 Also seeing the same...

[donniewa]

+1

[ricardocasares]

+1

[brianwider]

+1 here

[lgalfaso]

@petebacondarwin would like to revert the commit that caused this regression until we have a better solution. WDYT?

[petebacondarwin]

Let's see if we can't fix this but revert before the next release if no joy

[shahata]

This seems to be the same issue that @jeffbcross mentioned for an internal test that failed for him. The fix is easy (initialize the variable to an empty array instead of undefined), but I would like to understand first why this happens and see if I can add a test to protect against this. Taking a deeper look.