pylint icon indicating copy to clipboard operation
pylint copied to clipboard

[tests] Test no-member false positive on exception without str args

Open Pierre-Sassoulas opened this issue 1 year ago • 4 comments

Type of Changes

Type
:bug: Bug fix
:scroll: Docs

Description

Regression test for #7233, the fix is in https://github.com/PyCQA/astroid/pull/1722, it will require an astroid update.

Closes #7233

Pierre-Sassoulas avatar Jul 27 '22 08:07 Pierre-Sassoulas

See https://github.com/PyCQA/astroid/pull/1722#pullrequestreview-1056292990.

DanielNoord avatar Jul 30 '22 11:07 DanielNoord

Shall we close this and move the test to astroid?

DanielNoord avatar Aug 09 '22 21:08 DanielNoord

As #7233 is a pylint bug too, I think it makes sense to also have a test in pylint. Maybe we could also move this issue in astroid and ask for a lot more test in astroid. The problem with that is that astroid do not have efficients and easy to create functional tests like pylint.. What do you think?

Pierre-Sassoulas avatar Aug 10 '22 18:08 Pierre-Sassoulas

As #7233 is a pylint bug too, I think it makes sense to also have a test in pylint. Maybe we could also move this issue in astroid and ask for a lot more test in astroid. The problem with that is that astroid do not have efficients and easy to create functional tests like pylint.. What do you think?

This test in astroid should be fairly straightforward. Since this is really more an astroid issue I think it makes more sense to have the regression test there instead of in pylint.

DanielNoord avatar Aug 10 '22 19:08 DanielNoord

Fair enough. Should we move such issues to astroid too ?

Pierre-Sassoulas avatar Aug 13 '22 11:08 Pierre-Sassoulas

Fair enough. Should we move such issues to astroid too ?

Perhaps? Although I think for visibility towards users it's probably best to keep them here.

DanielNoord avatar Aug 14 '22 11:08 DanielNoord