-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(debugger): fix issue where output does not display circular dep and ... #1889
Conversation
then(function() { | ||
var result = execFn_(); | ||
if (webdriver.promise.isPromise(result)) { | ||
return result.then(function(val) {return val;}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was the reason for this in the past? It doesn't appear to do anything
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
element(by.xyz).getText()
is an elementFinder/promise.
When someone enters element(by.xyz).getText()
, we need to return the text value of the elementFinder (i.e. by calling then) instead of the text representation of the elementFinder.
8349cdb
to
6a28eb5
Compare
execute: function(code) { | ||
var execFn_ = function() { | ||
// Run code through vm so that we can maintain a local scope which is | ||
// isolated from the rest of the execution. | ||
return vm_.runInThisContext(code); | ||
var res = vm_.runInThisContext(code); | ||
if(!webdriver.promise.isPromise(res)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: add a space after it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
6a28eb5
to
11fc94e
Compare
LGTM sorry for the delay |
merged in 5d23280 |
...functions