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

fix(debugger): fix issue where output does not display circular dep and ... - #1889

Closed
hankduan wants to merge 1 commit into
angular:masterfrom
hankduan:fixelementexplorer
Closed

hankduan wants to merge 1 commit into
angular:masterfrom
hankduan:fixelementexplorer

Conversation

@hankduan

@hankduan hankduan commented Mar 3, 2015

Copy link
Copy Markdown
Contributor

...functions

Comment thread lib/protractor.js

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor Author

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.

@hankduan
hankduan force-pushed the fixelementexplorer branch from 8349cdb to 6a28eb5 Compare March 4, 2015 20:52
Comment thread lib/protractor.js Outdated

Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

@hankduan
hankduan force-pushed the fixelementexplorer branch from 6a28eb5 to 11fc94e Compare March 5, 2015 07:56
@juliemr

juliemr commented Mar 10, 2015

Copy link
Copy Markdown
Member

LGTM sorry for the delay

@hankduan

Copy link
Copy Markdown
Contributor Author

merged in 5d23280

@hankduan hankduan closed this Mar 11, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants