Commit d2b026f0 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Test both for having existing issues or not

parent 7d3c98e7
...@@ -525,7 +525,7 @@ describe 'Issues', feature: true do ...@@ -525,7 +525,7 @@ describe 'Issues', feature: true do
end end
describe 'new issue by email' do describe 'new issue by email' do
context 'click the button to show modal for new issue email', js: true do shared_examples 'show the email in the modal' do
before do before do
stub_incoming_email_setting(enabled: true, address: "p+%{key}@gl.ab") stub_incoming_email_setting(enabled: true, address: "p+%{key}@gl.ab")
...@@ -533,7 +533,7 @@ describe 'Issues', feature: true do ...@@ -533,7 +533,7 @@ describe 'Issues', feature: true do
click_button('Email a new issue') click_button('Email a new issue')
end end
it 'shows the email in the modal' do it 'click the button to show modal for the new email' do
page.within '#issue-email-modal' do page.within '#issue-email-modal' do
email = project.new_issue_address(@user) email = project.new_issue_address(@user)
...@@ -541,6 +541,16 @@ describe 'Issues', feature: true do ...@@ -541,6 +541,16 @@ describe 'Issues', feature: true do
end end
end end
end end
context 'with existing issues' do
let!(:issue) { create(:issue, project: project, author: @user) }
it_behaves_like 'show the email in the modal'
end
context 'without existing issues' do
it_behaves_like 'show the email in the modal'
end
end end
describe 'due date' do describe 'due date' do
......
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