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

feat: Allow specifying the /forcequit argument for WAD #282

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

papauorg
Copy link

The WinAppDriver supports a command line argument that will close the program under test ignoring any open windows or message boxes. This is very helpful for avoiding stale open applications in case the test did not properly clean up and close the program or an error occured during testing.

I extended the available capabilities by ms:forcequit which can be specified by the test. It will then start the WinAppDriver with an additional argument /forcequit if set.

Copy link

linux-foundation-easycla bot commented Nov 15, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@mykola-mokhnach
Copy link

thank you @papauorg
Could you please sign CLA?

README.md Outdated Show resolved Hide resolved
@papauorg
Copy link
Author

papauorg commented Nov 15, 2024

Hi @mykola-mokhnach,

I adressed your comments for your ultra-fast review :) Thanks for that!

Could you please sign CLA?

i did sign it. But to be honest it is kind of irritating to sign a 3 page contract for a contribution like this.

@papauorg papauorg force-pushed the feature/wad-forcequit branch from 1c5d855 to 4270b44 Compare November 15, 2024 10:13
@mykola-mokhnach
Copy link

Could you please sign CLA?

i did sign it. But to be honest it is kind of irritating to sign a 3 page contract for a contribution like this.

Sorry for that, this CLA is common for all OSS projects under Linux Foundation.

@papauorg papauorg force-pushed the feature/wad-forcequit branch from 4270b44 to f779ac1 Compare November 15, 2024 10:20
@mykola-mokhnach mykola-mokhnach merged commit a9dda08 into appium:master Nov 15, 2024
7 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 15, 2024
## [3.1.0](v3.0.2...v3.1.0) (2024-11-15)

### Features

* Allow specifying the /forcequit argument for WAD ([#282](#282)) ([a9dda08](a9dda08))
Copy link

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@papauorg papauorg deleted the feature/wad-forcequit branch November 15, 2024 11:55
@KazuCocoa KazuCocoa added the size:S contribution size: S label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released size:S contribution size: S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants