see the discussion at
https://lists.wikimedia.org/pipermail/qa/2014-October/002021.html
this would allow us to not have to make 2-line Then statements along the lines of
Then(/^the top post is an open discussion$/) do
on(FlowPage) do |page| page.flow_first_topic_moderation_msg_element.when_not_present expect(page.flow_first_topic_moderation_element).not_to be_visible end
end
Version: wmf-deployment
Severity: normal
Zeljko and I dug into this and we discovered that refactoring has removed most instance of this sort of thing. But here is one example still in place, from reply_steps.rb in Flow:
Then(/^I should see the topic reply form$/) do
on(FlowPage) do |page|
page.wait_until { page.new_reply_input_element.visible? }
expect(page.new_reply_input_element).to be_visible
endend
This should do the same thing:
Then(/^I should see the topic reply form$/) do expect(on(FlowPage).new_reply_input_element.when_visible).to be_visible end
I think this should actually be
Then(/^I should see the topic reply form$/) do
expect(on(FlowPage).new_reply_input_element.when_present).to be_visible end
but this makes sense
(In reply to Chris McMahon from comment #3)
Then(/^I should see the topic reply form$/) do
expect(on(FlowPage).new_reply_input_element.when_present).to be_visible end
If it's not present, won't this actually timeout on the when_present (as S brought up in his email), before the expect runs?