Commit cc00b289 authored by Mike Greiling's avatar Mike Greiling

fix flaky tests by removing unneeded clicks and focus actions

parent 72157766
...@@ -28,7 +28,6 @@ feature 'GFM autocomplete', :js do ...@@ -28,7 +28,6 @@ feature 'GFM autocomplete', :js do
it 'opens autocomplete menu when field starts with text' do it 'opens autocomplete menu when field starts with text' do
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
find('#note-body').native.send_keys('')
find('#note-body').native.send_keys('@') find('#note-body').native.send_keys('@')
end end
...@@ -46,7 +45,6 @@ feature 'GFM autocomplete', :js do ...@@ -46,7 +45,6 @@ feature 'GFM autocomplete', :js do
it 'doesnt select the first item for non-assignee dropdowns' do it 'doesnt select the first item for non-assignee dropdowns' do
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
find('#note-body').native.send_keys('')
find('#note-body').native.send_keys(':') find('#note-body').native.send_keys(':')
end end
...@@ -86,7 +84,6 @@ feature 'GFM autocomplete', :js do ...@@ -86,7 +84,6 @@ feature 'GFM autocomplete', :js do
it 'selects the first item for assignee dropdowns' do it 'selects the first item for assignee dropdowns' do
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
find('#note-body').native.send_keys('')
find('#note-body').native.send_keys('@') find('#note-body').native.send_keys('@')
end end
...@@ -112,7 +109,6 @@ feature 'GFM autocomplete', :js do ...@@ -112,7 +109,6 @@ feature 'GFM autocomplete', :js do
it 'selects the first item for non-assignee dropdowns if a query is entered' do it 'selects the first item for non-assignee dropdowns if a query is entered' do
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
find('#note-body').native.send_keys('')
find('#note-body').native.send_keys(':1') find('#note-body').native.send_keys(':1')
end end
...@@ -127,9 +123,8 @@ feature 'GFM autocomplete', :js do ...@@ -127,9 +123,8 @@ feature 'GFM autocomplete', :js do
it 'wraps the result in double quotes' do it 'wraps the result in double quotes' do
note = find('#note-body') note = find('#note-body')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys('') find('#note-body').native.send_keys('')
simulate_input('#note-body', "~#{label.title[0]}") simulate_input('#note-body', "~#{label.title[0]}")
note.click
end end
label_item = find('.atwho-view li', text: label.title) label_item = find('.atwho-view li', text: label.title)
...@@ -152,16 +147,13 @@ feature 'GFM autocomplete', :js do ...@@ -152,16 +147,13 @@ feature 'GFM autocomplete', :js do
it "does not show dropdown when preceded with a special character" do it "does not show dropdown when preceded with a special character" do
note = find('#note-body') note = find('#note-body')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys('')
note.native.send_keys("@") note.native.send_keys("@")
note.click
end end
expect(page).to have_selector('.atwho-container') expect(page).to have_selector('.atwho-container')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys("@") note.native.send_keys("@")
note.click
end end
expect(page).to have_selector('.atwho-container', visible: false) expect(page).to have_selector('.atwho-container', visible: false)
...@@ -170,9 +162,7 @@ feature 'GFM autocomplete', :js do ...@@ -170,9 +162,7 @@ feature 'GFM autocomplete', :js do
it "does not throw an error if no labels exist" do it "does not throw an error if no labels exist" do
note = find('#note-body') note = find('#note-body')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys('')
note.native.send_keys('~') note.native.send_keys('~')
note.click
end end
expect(page).to have_selector('.atwho-container', visible: false) expect(page).to have_selector('.atwho-container', visible: false)
...@@ -181,7 +171,6 @@ feature 'GFM autocomplete', :js do ...@@ -181,7 +171,6 @@ feature 'GFM autocomplete', :js do
it 'doesn\'t wrap for assignee values' do it 'doesn\'t wrap for assignee values' do
note = find('#note-body') note = find('#note-body')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys('')
note.native.send_keys("@#{user.username[0]}") note.native.send_keys("@#{user.username[0]}")
end end
...@@ -193,9 +182,7 @@ feature 'GFM autocomplete', :js do ...@@ -193,9 +182,7 @@ feature 'GFM autocomplete', :js do
it 'doesn\'t wrap for emoji values' do it 'doesn\'t wrap for emoji values' do
note = find('#note-body') note = find('#note-body')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys('')
note.native.send_keys(":cartwheel_") note.native.send_keys(":cartwheel_")
note.click
end end
emoji_item = find('.atwho-view li', text: 'cartwheel_tone1') emoji_item = find('.atwho-view li', text: 'cartwheel_tone1')
...@@ -222,9 +209,7 @@ feature 'GFM autocomplete', :js do ...@@ -222,9 +209,7 @@ feature 'GFM autocomplete', :js do
it 'triggers autocomplete after selecting a quick action' do it 'triggers autocomplete after selecting a quick action' do
note = find('#note-body') note = find('#note-body')
page.within '.timeline-content-form' do page.within '.timeline-content-form' do
note.native.send_keys('')
note.native.send_keys('/as') note.native.send_keys('/as')
note.click
end end
find('.atwho-view li', text: '/assign') find('.atwho-view li', text: '/assign')
......
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