-
Notifications
You must be signed in to change notification settings - Fork 5
Pass through all docs and make consistant (and correct minor issues) #52
Conversation
src/interactions/find-all.js
Outdated
* } | ||
* ``` | ||
* | ||
* ``` javascript | ||
* let $listItems = await new ListInteractor('ul').getItems() | ||
* let $listItems = await new ListInteractor('ul').items; |
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.
No need to await
here
src/interactions/find.js
Outdated
* } | ||
* ``` | ||
* | ||
* ``` javascript | ||
* let $heading = await new PageInteractor().getHeading() | ||
* let $heading = await new PageInteractor().heading; |
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.
No need to await
here either
src/interactions/find-all.js
Outdated
@@ -6,7 +6,7 @@ import { computed } from './helpers'; | |||
* matching the provided selector. | |||
* | |||
* ``` javascript | |||
* let $listItems = await new Interactor('ul').findAll('li'); | |||
* let $listItems = new Interactor('ul').findAll('li'); |
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.
The methods find
and findAll
still needs to await
. The property creators do not. You can see in the two function bodies for each one returns a convergence and the other returns a computed element.
The methods are used to wait for elements to exist, and the created properties can be used inside convergent assertions.
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.
Ahhh, gotcha. That makes sense
a7011c7
to
6b8ae82
Compare
What is this?
With #51 being merged through it was a good time to go through and make all of our inline docs consistent in style. With a fresh pair of eyes I went through all of the docs and double checked everything. It's possible I missed things, so please double check!
This will close #25