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

docs(webdriver): protractor style docs for getDriver, getId, and click #3101

Merged
merged 1 commit into from
Apr 9, 2016

Conversation

cnishina
Copy link
Contributor

@cnishina cnishina commented Apr 7, 2016

No description provided.

* @example
* // returns the dog web element
* var dog = element(by.css('.dog')).getWebElement();
* expect(dog.getId()).not.toBe(undefined);
Copy link
Contributor

Choose a reason for hiding this comment

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

not toBe undefined? What is it then? An empty string?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

getId returns a string that is a number. So, I'm getting the id to the web element and it exists.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, I understand now. You should add a line up top explaining what this method actually does, because I assumed it got the id attribute of an element

@sjelin sjelin self-assigned this Apr 7, 2016
@juliemr juliemr assigned cnishina and unassigned sjelin Apr 8, 2016
@cnishina cnishina force-pushed the pr_fixdocs_webdriver branch 2 times, most recently from b81167a to a1fa6a2 Compare April 8, 2016 17:41
@cnishina cnishina assigned sjelin and unassigned cnishina Apr 8, 2016
@cnishina cnishina force-pushed the pr_fixdocs_webdriver branch from a1fa6a2 to 184be54 Compare April 9, 2016 07:07
@cnishina cnishina merged commit 6f3a02f into angular:master Apr 9, 2016
@cnishina cnishina deleted the pr_fixdocs_webdriver branch April 9, 2016 07:39
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.

3 participants