Commit ff052757 authored by Alfredo Sumaran's avatar Alfredo Sumaran

Add feature tests for Dashboard Groups page

parent 4934c709
...@@ -6,21 +6,28 @@ describe 'Dashboard Groups page', js: true, feature: true do ...@@ -6,21 +6,28 @@ describe 'Dashboard Groups page', js: true, feature: true do
let!(:nested_group) { create(:group, :nested) } let!(:nested_group) { create(:group, :nested) }
let!(:another_group) { create(:group) } let!(:another_group) { create(:group) }
before do it 'shows groups user is member of' do
group.add_owner(user) group.add_owner(user)
nested_group.add_owner(user) nested_group.add_owner(user)
login_as(user) login_as(user)
visit dashboard_groups_path visit dashboard_groups_path
end
it 'shows groups user is member of' do
expect(page).to have_content(group.full_name) expect(page).to have_content(group.full_name)
expect(page).to have_content(nested_group.full_name) expect(page).to have_content(nested_group.full_name)
expect(page).not_to have_content(another_group.full_name) expect(page).not_to have_content(another_group.full_name)
end end
describe 'when filtering groups' do
before do
group.add_owner(user)
nested_group.add_owner(user)
login_as(user)
visit dashboard_groups_path
end
it 'filters groups' do it 'filters groups' do
fill_in 'filter_groups', with: group.name fill_in 'filter_groups', with: group.name
wait_for_requests wait_for_requests
...@@ -42,4 +49,81 @@ describe 'Dashboard Groups page', js: true, feature: true do ...@@ -42,4 +49,81 @@ describe 'Dashboard Groups page', js: true, feature: true do
expect(page).not_to have_content(another_group.full_name) expect(page).not_to have_content(another_group.full_name)
expect(page.all('.js-groups-list-holder .content-list li').length).to eq 2 expect(page.all('.js-groups-list-holder .content-list li').length).to eq 2
end end
end
describe 'group with subgroups' do
let!(:subgroup) { create(:group, :public, parent: group) }
before do
group.add_owner(user)
subgroup.add_owner(user)
login_as(user)
visit dashboard_groups_path
end
it 'shows subgroups inside of its parent group' do
expect(page).to have_selector('.groups-list-tree-container .group-list-tree', count: 2)
expect(page).to have_selector(".groups-list-tree-container #group-#{group.id} #group-#{subgroup.id}", count: 1)
end
it 'can toggle parent group' do
# Expanded by default
expect(page).to have_selector("#group-#{group.id} .fa-caret-down", count: 1)
expect(page).not_to have_selector("#group-#{group.id} .fa-caret-right")
# Collapse
find("#group-#{group.id}").trigger('click')
expect(page).not_to have_selector("#group-#{group.id} .fa-caret-down")
expect(page).to have_selector("#group-#{group.id} .fa-caret-right", count: 1)
expect(page).not_to have_selector("#group-#{group.id} #group-#{subgroup.id}")
# Expand
find("#group-#{group.id}").trigger('click')
expect(page).to have_selector("#group-#{group.id} .fa-caret-down", count: 1)
expect(page).not_to have_selector("#group-#{group.id} .fa-caret-right")
expect(page).to have_selector("#group-#{group.id} #group-#{subgroup.id}")
end
end
describe 'when using pagination' do
let(:group2) { create(:group) }
before do
group.add_owner(user)
group2.add_owner(user)
allow(Kaminari.config).to receive(:default_per_page).and_return(1)
login_as(user)
visit dashboard_groups_path
end
it 'shows pagination' do
expect(page).to have_selector('.gl-pagination')
expect(page).to have_selector('.gl-pagination .page', count: 2)
end
it 'loads results for next page' do
# Check first page
expect(page).to have_content(group2.full_name)
expect(page).to have_selector("#group-#{group2.id}")
expect(page).not_to have_content(group.full_name)
expect(page).not_to have_selector("#group-#{group.id}")
# Go to next page
find(".gl-pagination .page:not(.active) a").trigger('click')
wait_for_requests
# Check second page
expect(page).to have_content(group.full_name)
expect(page).to have_selector("#group-#{group.id}")
expect(page).not_to have_content(group2.full_name)
expect(page).not_to have_selector("#group-#{group2.id}")
end
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment