-
Notifications
You must be signed in to change notification settings - Fork 17
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
(feat): add appEntryPoint for executing electron directly with a main script #345
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.
This is awesome 🎉 I will make a PR to the config wizard to have this be the default behavior.
It will be awesome once it works 🤣 It's running electron but not managing to load the window in WDIO for the tests to work. Loading manually with e.g. |
I wonder if it is Chromedriver which passes some command line arguments that make it not work 🤔 |
I don't have the time to investigate why this isn't working at the moment, currently focussing on the mocking rework. |
CI build fails - window issue
ea56a5a
to
76f9223
Compare
I logged out the args being passed using a shell script:
Not all of these are available on the list of command line switches but I can't see anything obviously causing this apart from perhaps the UPDATE: here are the args as logged out from
|
You can change it by providing I've tried excluding all switches with |
Closing in favour of #545 |
Prototyping the feature proposed in #331.