Update check_test docstring to clarify state parameter

avocado-framework / avocado

Avocado is a set of tools and libraries to help with automated testing. One can call it a test framework with benefits. Native tests are written in Python and they follow the unittest pattern, but any executable can serve as a test.

https://avocado-framework.github.io/

Other

345 stars 343 forks source link

Update check_test docstring to clarify state parameter #6053

Closed mikigo closed 2 weeks ago

mikigo commented 3 weeks ago

In https://github.com/avocado-framework/avocado/blob/3b1f5cbbb07ea850498b55aa020d74cc0f7fa15f/avocado/core/result.py#L103

The parameter description of 'test' does not match the parameter 'state'

and

Refactor: Remove unused 'option = None' initialization

Signed-off-by: mikigo huangmingqiang@uniontech.com

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 54.48%. Comparing base (f8537a5) to head (46d0b63). Report is 14 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #6053 +/- ## ========================================== + Coverage 54.42% 54.48% +0.06% ========================================== Files 202 202 Lines 21885 21884 -1 ========================================== + Hits 11911 11924 +13 + Misses 9974 9960 -14 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mikigo commented 3 weeks ago

Thank you for your feedback! I have made the necessary changes as requested.

Modified Commit Messages: I have updated the commit messages to include the required information.
Split Commits: I have split the previous single commit into two separate commits.
Force Pushed: I have force pushed the changes to this PR.

Please review the changes and let me know if there is anything else I need to address.

Thanks again for your guidance!

Best regards, mikigo