-
Notifications
You must be signed in to change notification settings - Fork 29
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
fix Implement support for proxying pages inside ASAR archives
#44
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, it's fine, but there are some points to discuss:
-
I think it's better to package the test app just before the tests instead of storing the archive in the repository. How hard is it to create a Gulp task that will package the test app?
-
I remember that we decided to move the config.appEntryPoint into the injected script.
-
I'm sorry, but the config file for the tested app looks too complex for me. Maybe it's really better to create two separate config files and then start TestCafe with
electron:regular-app-config
andelectron-asar-app-config
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Superb, let's just wait for a stable TestCafe release.
DevExpress/testcafe-hammerhead#2033
Changes
appEntryPoint
resolving to src/injectable/index.js