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

More precise testing of plugins #935

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

Conversation

ericwb
Copy link
Member

@ericwb ericwb commented Jul 14, 2022

Instead of relying on a count of the severity and confidence
levels found within an example file, make use of Python's native
unit testing to verify the results of a plugin.

The existing method of confirming counts can be inaccurate. It's
very easy to have a false positive simply because one issue extra
was found and one issue was missed, thus giving the same count. It
tells nothing of the validation of a particular line of problematic
code.

Relates to #352

Signed-off-by: Eric Brown [email protected]

@ericwb ericwb force-pushed the plugin_unit_tests branch 2 times, most recently from 05be4af to 31da3fe Compare July 17, 2022 18:10
@ericwb ericwb force-pushed the plugin_unit_tests branch 6 times, most recently from 33cfb3e to 7d15f64 Compare March 13, 2023 05:24
@ericwb ericwb force-pushed the plugin_unit_tests branch from 181de9c to 1c33807 Compare April 7, 2023 16:46
Instead of relying on a count of the severity and confidence
levels found within an example file, make use of Python's native
unit testing to verify the results of a plugin.

The existing method of confirming counts can be inaccurate. It's
very easy to have a false positive simply because one issue extra
was found and one issue was missed, thus giving the same count. It
tells nothing of the validation of a particular line of problematic
code.

Relates to PyCQA#352

Signed-off-by: Eric Brown <[email protected]>
@ericwb ericwb force-pushed the plugin_unit_tests branch from 1c33807 to 8c193a4 Compare April 7, 2023 17:19
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.

1 participant