diff options
author | Ezekiel Kigbo <ekigbo@gitlab.com> | 2019-03-04 08:57:40 +0300 |
---|---|---|
committer | Ezekiel Kigbo <ekigbo@gitlab.com> | 2019-03-04 16:58:23 +0300 |
commit | e4f9c3f63ad7e8147acecd2324de90b63ffcd940 (patch) | |
tree | 270c67e86a3080d34da75ef1db59ba33166f83fd /spec/features/projects/user_sees_sidebar_spec.rb | |
parent | 3e42a0c175d43609ea7b12f39ca3caae22eea817 (diff) |
Fix collapsing behaviour at 1200px
Diffstat (limited to 'spec/features/projects/user_sees_sidebar_spec.rb')
-rw-r--r-- | spec/features/projects/user_sees_sidebar_spec.rb | 38 |
1 files changed, 20 insertions, 18 deletions
diff --git a/spec/features/projects/user_sees_sidebar_spec.rb b/spec/features/projects/user_sees_sidebar_spec.rb index 43fe477fb29..736a574e937 100644 --- a/spec/features/projects/user_sees_sidebar_spec.rb +++ b/spec/features/projects/user_sees_sidebar_spec.rb @@ -11,8 +11,6 @@ describe 'Projects > User sees sidebar' do sign_in(user) end - # refactor behaviours into shared behaviours - shared_examples 'has a collapsible mobile nav sidebar' do it 'has a collapsed desktop nav-sidebar on load' do expect(page).not_to have_content('Collapse sidebar') @@ -26,6 +24,21 @@ describe 'Projects > User sees sidebar' do end end + shared_examples 'has a desktop nav sidebar' do + it 'has a expanded desktop nav-sidebar on load' do + expect(page).to have_content('Collapse sidebar') + expect(page).not_to have_selector('.sidebar-collapsed-desktop') + expect(page).not_to have_selector('.sidebar-expanded-mobile') + end + + it 'can collapse the nav-sidebar' do + page.find('.nav-sidebar .js-toggle-sidebar').click + expect(page).to have_selector('.sidebar-collapsed-desktop') + expect(page).not_to have_content('Collapse sidebar') + expect(page).not_to have_selector('.sidebar-expanded-mobile') + end + end + context 'with xs size' do before do resize_screen_xs @@ -56,9 +69,9 @@ describe 'Projects > User sees sidebar' do it_behaves_like 'has a collapsible mobile nav sidebar' end - context 'at 1200px exactly' do + context 'with size 1199px' do before do - resize_window(1200, 800) + resize_window(1199, 800) visit project_path(project) expect(page).to have_selector('.nav-sidebar') end @@ -66,25 +79,14 @@ describe 'Projects > User sees sidebar' do it_behaves_like 'has a collapsible mobile nav sidebar' end - context 'at 1201px exactly' do + context 'with a larger screen' do before do - resize_window(1201, 800) + resize_window(1200, 800) visit project_path(project) expect(page).to have_selector('.nav-sidebar') end - it 'has a expanded desktop nav-sidebar on load' do - expect(page).to have_content('Collapse sidebar') - expect(page).not_to have_selector('.sidebar-collapsed-desktop') - expect(page).not_to have_selector('.sidebar-expanded-mobile') - end - - it 'can collapse the nav-sidebar' do - page.find('.nav-sidebar .js-toggle-sidebar').click - expect(page).to have_selector('.sidebar-collapsed-desktop') - expect(page).not_to have_content('Collapse sidebar') - expect(page).not_to have_selector('.sidebar-expanded-mobile') - end + it_behaves_like 'has a desktop nav sidebar' end end |