Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Failing tests with Angular 1.6 #1877

Closed
pete-otaqui opened this issue Dec 19, 2016 · 3 comments
Closed

Failing tests with Angular 1.6 #1877

pete-otaqui opened this issue Dec 19, 2016 · 3 comments

Comments

@pete-otaqui
Copy link
Contributor

4 tests fail with Angular 1.6 (the latest version that is installed by default since it's release).

Specifically related to this breaking change between 1.5 and 1.6:

angular/angular.js@e13eeab

Exceptions inside promises are now handled as rejections.

pete-otaqui added a commit to pete-otaqui/ui-select that referenced this issue Dec 19, 2016
Angular 1.6 alters the way uncaught promises are handled - errors cause promise rejection rather than outright exceptions to be thrown.  Update existing exception tests to use the exceptionHandler service instead of `.throws()`

Closes angular-ui#1877
aaronroberson pushed a commit that referenced this issue Jan 4, 2017
Angular 1.6 alters the way uncaught promises are handled - errors cause promise rejection rather than outright exceptions to be thrown.  Update existing exception tests to use the exceptionHandler service instead of `.throws()`

Closes #1877
@Jefiozie
Copy link
Contributor

Hi @pete-otaqui , this PR was only to support angular version 1.5 and 1.6 right?

I see a lot of problems with new PR's and it looks like this change has to do with it.
Thanks for the time.

@pete-otaqui
Copy link
Contributor Author

Hi @Jefiozie

It was only to support angular 1.6, yes, since it gobbles up errors, and requires you to use angular's own exceptionHandler rather than ... you know ... Javascript's own means for doing that.

What specific problems are you referring to?

@Jefiozie
Copy link
Contributor

Hi @pete-otaqui ,

Thanks for the quick reply.

Since the merge of your PR I see that all "new" PR's have build errors. I removed your adjustments and as it looks the problems are gone. (see: #1917 )
Still I don't see the root cause of this maybe you see it? Is it a angular problem or are the test not correct implemented for 1.6.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants