unexpected-keyword-arg in dataclasses with kw_only=True, inheritance and defaults

[sharhalakis]

Bug description

This is practically #6275 when using the new kw_only=True

The following code causes the bogus error. The key points are:

• the kw_only argument on class A
• the presence of a default in class A
• that class B includes a value without a default.

The problem doesn't happen if there's no default in-between or if inheritance isn't used.

# pylint: disable=missing-module-docstring,invalid-name,too-few-public-methods,missing-class-docstring

import dataclasses as dc

@dc.dataclass(kw_only=True)
class A:
    a: str
    b: int = 1

@dc.dataclass(kw_only=True)
class B(A):
    c: str

c = B(a='aaa', c='ccc')

Command used

pylint b.py

Pylint output

$ pylint b.py 
************* Module b
b.py:15:4: E1123: Unexpected keyword argument 'c' in constructor call (unexpected-keyword-arg)

------------------------------------------------------------------
Your code has been rated at 3.75/10 (previous run: 3.75/10, +0.00)

Expected behavior

No pylint errors. This is valid code.

Pylint version

$ pylint --version
pylint 2.12.2
astroid 2.9.3
Python 3.10.4 (main, Mar 24 2022, 13:07:27) [GCC 11.2.0]

OS / Environment

$ lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux bookworm/sid
Release:        testing
Codename:       bookworm

[anis-campos]

+1. kw_only is very useful and the completely disabling unexpected-keyword-arg seems counter productive.

So for now we are disabling the linter on every instantiation.

[smrnff]

+1 We really need the feature too.

So for now we are disabling the linter on every instantiation.

We do the same things. It freaks me out.

[DanielNoord]

Fixed with in pylint-dev/astroid#1764

@jacobtylerwalls Do we want test for this and for the original issue that that PR targeted in pylint? Or is astroid enough in your opinion?

[jacobtylerwalls]

Let's merge at least one test case in pylint, since the astroid one is a little convoluted (on purpose, to catch the string contains issue.)