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

Remove invalid pytest option in setup.cfg, improve readability of test result #9356

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TGoddessana
Copy link
Contributor

@TGoddessana TGoddessana commented Apr 2, 2024

Note: Before submitting this pull request, please review our contributing guidelines.

Description

If you run ./runtests.py in the current package, you'll get a warning like this. (unknown config option: testspath)
스크린샷 2024-04-02 오후 3 35 04

A patch was made in this commit to remove the warning, but it appears to have been reverted back here.

I've also added the -v option to verbosely display the results of running ./runtests.
(If this is an unnecessary change, I'll revert it).

(before -v option)
스크린샷 2024-04-02 오후 3 34 38

(after added -v option)
스크린샷 2024-04-02 오후 3 36 05

This change does not affect the results of the -q option.
(with -q option)
image

References

as you can see, testspath is not valid pytest option and should use testpaths

image

@TGoddessana TGoddessana changed the title fix: Remove invalid pytest option in setup.cfg, improve readability of test result Remove invalid pytest option in setup.cfg, improve readability of test result Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant