Commit a6e312bf authored by Phil Hughes's avatar Phil Hughes

Updated wording in the specs

parent 48a360f2
...@@ -18,7 +18,7 @@ describe 'New/edit issue', feature: true, js: true do ...@@ -18,7 +18,7 @@ describe 'New/edit issue', feature: true, js: true do
visit new_namespace_project_issue_path(project.namespace, project) visit new_namespace_project_issue_path(project.namespace, project)
end end
it 'should allow user to create new issue' do it 'allows user to create new issue' do
fill_in 'issue_title', with: 'title' fill_in 'issue_title', with: 'title'
fill_in 'issue_description', with: 'title' fill_in 'issue_description', with: 'title'
...@@ -75,7 +75,7 @@ describe 'New/edit issue', feature: true, js: true do ...@@ -75,7 +75,7 @@ describe 'New/edit issue', feature: true, js: true do
visit edit_namespace_project_issue_path(project.namespace, project, issue) visit edit_namespace_project_issue_path(project.namespace, project, issue)
end end
it 'should allow user to update issue' do it 'allows user to update issue' do
expect(find('input[name="issue[assignee_id]"]', visible: false).value).to match(user.id.to_s) expect(find('input[name="issue[assignee_id]"]', visible: false).value).to match(user.id.to_s)
expect(find('input[name="issue[milestone_id]"]', visible: false).value).to match(milestone.id.to_s) expect(find('input[name="issue[milestone_id]"]', visible: false).value).to match(milestone.id.to_s)
......
...@@ -16,7 +16,7 @@ describe 'New/edit merge request', feature: true, js: true do ...@@ -16,7 +16,7 @@ describe 'New/edit merge request', feature: true, js: true do
before do before do
login_as(user) login_as(user)
end end
context 'new merge request' do context 'new merge request' do
before do before do
visit new_namespace_project_merge_request_path( visit new_namespace_project_merge_request_path(
...@@ -30,7 +30,7 @@ describe 'New/edit merge request', feature: true, js: true do ...@@ -30,7 +30,7 @@ describe 'New/edit merge request', feature: true, js: true do
}) })
end end
it 'should create new merge request' do it 'creates new merge request' do
click_button 'Assignee' click_button 'Assignee'
page.within '.dropdown-menu-user' do page.within '.dropdown-menu-user' do
click_link user.name click_link user.name
...@@ -91,7 +91,7 @@ describe 'New/edit merge request', feature: true, js: true do ...@@ -91,7 +91,7 @@ describe 'New/edit merge request', feature: true, js: true do
visit edit_namespace_project_merge_request_path(project.namespace, project, merge_request) visit edit_namespace_project_merge_request_path(project.namespace, project, merge_request)
end end
it 'should update merge request' do it 'updates merge request' do
click_button 'Assignee' click_button 'Assignee'
page.within '.dropdown-menu-user' do page.within '.dropdown-menu-user' do
click_link user.name click_link user.name
...@@ -160,7 +160,7 @@ describe 'New/edit merge request', feature: true, js: true do ...@@ -160,7 +160,7 @@ describe 'New/edit merge request', feature: true, js: true do
}) })
end end
it 'should create new merge request' do it 'creates new merge request' do
click_button 'Assignee' click_button 'Assignee'
page.within '.dropdown-menu-user' do page.within '.dropdown-menu-user' do
click_link user.name click_link user.name
......
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