Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Don't rewrap WebDriver promise into q promise in DriverProvider.prototype.quitDriver #3902

Closed
sjelin opened this issue Dec 30, 2016 · 0 comments

Comments

@sjelin
Copy link
Contributor

sjelin commented Dec 30, 2016

Necessary for #3899 so that we return a WebDriver promise and don't mix promise types.

We did this because quitDriver's promise gets passed to teardownEnv and shutdown_, each of which use q promises. But we'll just have to move the rewrapping one level up I guess.

sjelin added a commit to sjelin/protractor that referenced this issue Jan 20, 2017
Wrapping it in a `q` promise is blocking angular#3899

Closes angular#3902

Custom frameworks might not make this change but it'll be fine.  It'll only be a
problem in edge cases and they probably weren't returning the right promise
before anyway.
sjelin added a commit that referenced this issue Jan 20, 2017
#3992)

Wrapping it in a `q` promise is blocking #3899

Closes #3902

Custom frameworks might not make this change but it'll be fine.  It'll only be a
problem in edge cases and they probably weren't returning the right promise
before anyway.
sjelin added a commit that referenced this issue Jan 23, 2017
#3992)

Wrapping it in a `q` promise is blocking #3899

Closes #3902

Custom frameworks might not make this change but it'll be fine.  It'll only be a
problem in edge cases and they probably weren't returning the right promise
before anyway.
sjelin added a commit that referenced this issue Jan 26, 2017
#3992)

Wrapping it in a `q` promise is blocking #3899

Closes #3902

Custom frameworks might not make this change but it'll be fine.  It'll only be a
problem in edge cases and they probably weren't returning the right promise
before anyway.
@sjelin sjelin closed this as completed in 46c8898 Jan 27, 2017
igniteram pushed a commit to igniteram/protractor that referenced this issue Feb 21, 2017
angular#3992)

Wrapping it in a `q` promise is blocking angular#3899

Closes angular#3902

Custom frameworks might not make this change but it'll be fine.  It'll only be a
problem in edge cases and they probably weren't returning the right promise
before anyway.
bodyduardU pushed a commit to bodyduardU/protractor that referenced this issue Dec 5, 2022
…y (#3992)

Wrapping it in a `q` promise is blocking angular/protractor#3899

Closes angular/protractor#3902

Custom frameworks might not make this change but it'll be fine.  It'll only be a
problem in edge cases and they probably weren't returning the right promise
before anyway.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant