Skip to content

Switch from PhantomJS to headless Chrome #101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jmuheim wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Switch from PhantomJS to headless Chrome #101

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
d184440
Refactor
jmuheim Jun 9, 2018
3e5e59c
Refactor, fix specs
jmuheim Jun 9, 2018
de2efe3
Refactor, fix specs
jmuheim Jun 9, 2018
6dd6787
More cleanup
jmuheim Jun 9, 2018
badd05d
Fixed deprecation warning
gurix Jun 9, 2018
d30ebdf
Adjusted some rubocop settings
gurix Jun 9, 2018
f086685
Fixed broken test
gurix Jun 9, 2018
edbce84
Remove Poltergeist hack
gurix Jun 9, 2018
b1463d6
capybara gem missing for travis
gurix Jun 9, 2018
b5a5c5e
Added chrome addon to travis
gurix Jun 10, 2018
557a98d
Adde more settings to run chrome driver
gurix Jun 10, 2018
d3f7a7e
Deprecation warning disapears when remove sqlite 3
gurix Jun 11, 2018
c665940
fixed typo
gurix Jun 11, 2018
ef1a817
Added sudo required
gurix Jun 11, 2018
5d4280f
travis do not find the chromedriver, probably it helps to use chromed…
gurix Jun 11, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions spec/features/pages/edit_spec.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,8 @@
fill_in 'page_navigation_title', with: 'A new navigation title'
fill_in 'page_content', with: "A new content with a ![existing image](@image-existing-image) and a ![new image](@image-new-image). Also an ![existing code](@code-existing-code) and a ![new code](@code-new-code). "
fill_in 'page_notes', with: 'A new note'

find('#page_images_attributes_0_file', visible: false).set base64_other_image[:data]
# As capybara does not finde thethe element having "display: none" in headless chrome we simulate it as js code
Copy link
Owner Author

@jmuheim jmuheim Jun 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice idea.

Some improvements:

  • Typos: finde thethe
  • Create a module with this method, e.g. find_hidden (see FocusElement module for inspiration) so we can reuse it.

page.execute_script("jQuery('#page_images_attributes_0_file').val('#{base64_other_image[:data]}')")
fill_in 'page_images_attributes_0_identifier', with: 'existing-image'

fill_in 'page_codes_attributes_0_identifier', with: 'existing-code'
Expand Down Expand Up @@ -114,7 +114,7 @@
.and change { @page.navigation_title }.to('A new navigation title')
.and change { @page.parent }.from(old_page_parent).to(new_parent_page)
.and change { @page.position }.from(1).to(2)
.and change { @page.content }.to("A new content with a ![existing image](@image-existing-image) and a ![new image](@image-new-image). Also an ![existing code](@code-existing-code) and a ![new code](@code-new-code).")
.and change { @page.content }.to('A new content with a [existing image](@image-existing-image) and a [new image](@image-new-image). Also an [existing code](@code-existing-code) and a [new code](@code-new-code).')
.and change { @page.notes }.to('A new note')
.and change { @page.images.count }.by(1)
.and change { @page.images.first.file.file.identifier }.to('file.png')
Expand Down