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

Add innerHTML attribute support to HtmlUnitDriver #1206

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@Toilal
Contributor

Toilal commented Oct 30, 2015

To be consistent with WebRemoteDriver, innerHTML attribute should retrieve innerHTML of the underlying DOM Node. (Same idea as #1205)

@lukeis

This comment has been minimized.

Show comment
Hide comment
@lukeis

lukeis Oct 30, 2015

Member

merged as a3b32a3

Member

lukeis commented Oct 30, 2015

merged as a3b32a3

@lukeis lukeis closed this Oct 30, 2015

public void testShouldReturnInnerHtml() {
driver.get(pages.simpleTestPage);
String html = driver.findElement(By.id("wrappingtext")).getAttribute("innerHTML");

This comment has been minimized.

@ddavison

ddavison Oct 30, 2015

Member

this makes me wonder if we want to add a getProperty() in order to disambiguate properties from attributes..

@ddavison

ddavison Oct 30, 2015

Member

this makes me wonder if we want to add a getProperty() in order to disambiguate properties from attributes..

This comment has been minimized.

@lukeis

lukeis Oct 30, 2015

Member

ha... hahahahahah... you haven't been on this project very long, have you? :)

@lukeis

lukeis Oct 30, 2015

Member

ha... hahahahahah... you haven't been on this project very long, have you? :)

This comment has been minimized.

@ddavison

ddavison Oct 30, 2015

Member

what - has this come up before, or something?! :D

@ddavison

ddavison Oct 30, 2015

Member

what - has this come up before, or something?! :D

This comment has been minimized.

@Toilal

Toilal Oct 30, 2015

Contributor

I wonder i'll never need to test any testContent attribute ! it seems there are some magic keys in WebDriver... Any reason the API was not extended to support methods like getHtml or getNodeText ?

@Toilal

Toilal Oct 30, 2015

Contributor

I wonder i'll never need to test any testContent attribute ! it seems there are some magic keys in WebDriver... Any reason the API was not extended to support methods like getHtml or getNodeText ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment