diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2017-03-04 15:30:12 +0200 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2017-03-04 15:30:12 +0200 |
commit | c13c1e1b166edd5520057814dc4698450b578c9a (patch) | |
tree | 2aef40f264e6a93fd7b5e9ad6f2e67dff4b32527 | |
parent | 8a910ba297b229171a64794b8958401431354b4a (diff) | |
download | gitlab-ce-c13c1e1b166edd5520057814dc4698450b578c9a.tar.gz |
Improve explore projects spinach test
Project name "Internal" is too generic and can lead to false
positive/negative when there is a visibility filter on the page. So we
ensure we check for project inside list holder css class.
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-rw-r--r-- | features/steps/shared/project.rb | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/features/steps/shared/project.rb b/features/steps/shared/project.rb index dae248b8b7e..345a28f27dc 100644 --- a/features/steps/shared/project.rb +++ b/features/steps/shared/project.rb @@ -166,11 +166,15 @@ module SharedProject end step 'I should see project "Internal"' do - expect(page).to have_content "Internal" + page.within '.js-projects-list-holder' do + expect(page).to have_content "Internal" + end end step 'I should not see project "Internal"' do - expect(page).not_to have_content "Internal" + page.within '.js-projects-list-holder' do + expect(page).not_to have_content "Internal" + end end step 'public project "Community"' do |