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

Modify Execute Process Result Assertion #52

Closed
threeal opened this issue May 28, 2024 · 0 comments · Fixed by #53
Closed

Modify Execute Process Result Assertion #52

threeal opened this issue May 28, 2024 · 0 comments · Fixed by #53
Assignees
Labels
Milestone

Comments

@threeal
Copy link
Owner

threeal commented May 28, 2024

This issue suggests modifying the execute process result assertion as follows:

  • Remove the RESULT argument from the assert_execute_process function, which is used for asserting the status code of the executed process.
  • By default, make the assert_execute_process function assert whether the executed process exits successfully with status code 0.
  • If the ERROR argument is specified, make the assert_execute_process function assert whether the executed process exits with a failure and a status code that is not 0.
@threeal threeal added the feat label May 28, 2024
@threeal threeal added this to the Version 0.2.0 milestone May 28, 2024
@threeal threeal self-assigned this May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant