Properly check that issue list has persisted

A valid user needs to be signed in, and we need to reload the page to
verify that the issue list was saved on the backend.
This commit is contained in:
Stan Hu 2019-06-26 14:51:53 -07:00
parent 62a5a6b621
commit 25b3d905a0

View file

@ -102,10 +102,16 @@ describe 'Group issues page' do
end end
context 'manual ordering' do context 'manual ordering' do
let(:user_in_group) { create(:group_member, :maintainer, user: create(:user), group: group ).user }
let!(:issue1) { create(:issue, project: project, title: 'Issue #1', relative_position: 1) } let!(:issue1) { create(:issue, project: project, title: 'Issue #1', relative_position: 1) }
let!(:issue2) { create(:issue, project: project, title: 'Issue #2', relative_position: 2) } let!(:issue2) { create(:issue, project: project, title: 'Issue #2', relative_position: 2) }
let!(:issue3) { create(:issue, project: project, title: 'Issue #3', relative_position: 3) } let!(:issue3) { create(:issue, project: project, title: 'Issue #3', relative_position: 3) }
before do
sign_in(user_in_group)
end
it 'displays all issues' do it 'displays all issues' do
visit issues_group_path(group, sort: 'relative_position') visit issues_group_path(group, sort: 'relative_position')
@ -135,6 +141,16 @@ describe 'Group issues page' do
from_index: 0, from_index: 0,
to_index: 2) to_index: 2)
wait_for_requests
check_issue_order
visit issues_group_path(group, sort: 'relative_position')
check_issue_order
end
def check_issue_order
page.within('.manual-ordering') do page.within('.manual-ordering') do
expect(find('.issue:nth-child(1) .title')).to have_content('Issue #2') expect(find('.issue:nth-child(1) .title')).to have_content('Issue #2')
expect(find('.issue:nth-child(2) .title')).to have_content('Issue #3') expect(find('.issue:nth-child(2) .title')).to have_content('Issue #3')