Skip to content
This repository has been archived by the owner on Nov 23, 2023. It is now read-only.

Testing upstream #6

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Testing upstream #6

wants to merge 10 commits into from

Conversation

dmorgan-fa
Copy link

@dmorgan-fa dmorgan-fa commented Dec 22, 2022

What

Testing upstream main with additional logging around TimeoutError caused by wait_for_idle

Changes on this branch are:

  • Commits from 0.13.0 tag -> main
  • additional logging around TimeoutError to understand the new failures

@dmorgan-fa
Copy link
Author

dmorgan-fa commented Dec 22, 2022

Calls to wait_for_network_idle cause failures for 302 redirects that have completed. They're now being marked as still pending, because of:

Example:

Ferrum::TimeoutError:
         Timed out waiting for response. It's possible that this happened because something took a very long time (for example a page load was slow). If so, setting the :timeout option to a higher value might help.
         Connections still pending:
          {:staus=>302, :status_text=>"Found", :url=>"http://practice1.lvh.me:9888/practice_dashboard/companies/895397944/notes/1"}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants