From 428fa7f8113a015b017db723de110cd0e2ac4ce9 Mon Sep 17 00:00:00 2001 From: mlorb Date: Tue, 7 Nov 2017 11:29:37 +0100 Subject: [PATCH] follow @ZmagoD's remarks --- features/step_definitions/shared_steps.rb | 6 ------ features/step_definitions/team_steps.rb | 1 + 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/features/step_definitions/shared_steps.rb b/features/step_definitions/shared_steps.rb index c3df271d6..7a4d982df 100644 --- a/features/step_definitions/shared_steps.rb +++ b/features/step_definitions/shared_steps.rb @@ -2,12 +2,6 @@ When(/^I click "(.+)" button$/) do |button| click_on(button) end -When(/^I click "(.+)" button within "(.+)"$/) do |button, container| - within(find(container)) do - click_on button - end -end - Given(/^Show me the page$/) do save_and_open_page end diff --git a/features/step_definitions/team_steps.rb b/features/step_definitions/team_steps.rb index ccd3a9ba9..c5401b2e7 100644 --- a/features/step_definitions/team_steps.rb +++ b/features/step_definitions/team_steps.rb @@ -19,6 +19,7 @@ end Then(/^I should see "(.+)" in Role column of "(.+)" within Team members table$/) do |role, email| wait_for_ajax + sleep 0.3 mail_td = find('td', text: /\A#{email}\z/) parent = mail_td.first(:xpath, './/..') expect(parent).to have_css('td', text: /\A#{role}\z/)