search

pylint-dev / pylint

It's not just a linter that annoys you!
https://pylint.readthedocs.io/en/latest/
GNU General Public License v2.0
5.32k stars 1.14k forks source link

Add tests #9981

Closed nickdrozd closed 1 month ago

nickdrozd commented 1 month ago

Type of Changes

Type
✓ :test_tube: Tests

Description

Add tests for open issues #9668, #9815, #8893, #9941, #9642, and #9689. Some are false negatives and some are false positives.

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 95.80%. Comparing base (b9cc6b2) to head (f5ea1bd). Report is 2 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/pylint-dev/pylint/pull/9981/graphs/tree.svg?width=650&height=150&src=pr&token=ZETEzayrfk&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pylint-dev)](https://app.codecov.io/gh/pylint-dev/pylint/pull/9981?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pylint-dev) ```diff @@ Coverage Diff @@ ## main #9981 +/- ## ======================================= Coverage 95.80% 95.80% ======================================= Files 174 174 Lines 18946 18946 ======================================= Hits 18152 18152 Misses 794 794 ```
nickdrozd commented 1 month ago

I can fix up the commits, but it will require rerunning all the tests to update their output. Could you merge https://github.com/pylint-dev/pylint/pull/9982 first so I can run Pytest?