Skip to content
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.

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

Electron 8 #1598

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

Electron 8 #1598

wants to merge 5 commits into from

Conversation

codebling
Copy link

This is a work in progress. Opening PR for more visibility.

Current state

Currently 10 tests are still failing. They are:

From navigation:

  1. should succeed properly if request handler is present
  2. should support javascript URLs that load pages
  3. should fail immediately/not time out for 304 statuses

From navigation/timeouts:

  1. should allow actions on a hanging page
  2. should allow loading a new page after timing out

From referer:

  1. should return referer from headers

From options:

  1. should set a single header
  2. should set all headers
  3. should set headers for that request
  4. should allow webPreferences settings

Looking for help

To help updating to Electron 8, submit PRs against the branch here. If there are issues, open them against that repo, as well.

plan

First step is to fix these tests. This is kind of a first "naive" pass at migrating. Since test coverage is quite good for this project, this might actually be all that is needed, but I'd still want to go over the list of breaking changes one by one to make sure each change has been accounted for.

@codebling
Copy link
Author

See also discussion in #1539

@mjgp2
Copy link

mjgp2 commented Apr 18, 2020

@codebling - I was trying to look at your failing tests, but some of them I can't really get to the bottom of (e.g. javascript urls not working)

Did you make any more progress?

@codebling
Copy link
Author

@mjgp2 Thanks for looking! I didn't work on it any more! I feel like it's 70% there. I don't need it on the project I'm working on right now, so I probably won't look at this again until I do. I'll leave this parked here and hopefully someone can pick up where I left off.

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

Successfully merging this pull request may close these issues.

2 participants