[pylint] Fix false positives, add missing methods, and support positional-only parameters (PLE0302)

[dcarrier]

Summary

Resolves 3/4 requests in #16217:

• ✅ Remove not special methods: __cmp__, __div__, __nonzero__, and __unicode__.
• ✅ Add special methods: __next__, __buffer__, __class_getitem__, __mro_entries__, __release_buffer__, and __subclasshook__.
• ✅ Support positional-only arguments.
• ❌ Add support for module functions __dir__ and __getattr__. As mentioned in the issue the check is scoped for methods rather than module functions. I am hesitant to expand the scope of this check without a discussion.

Test Plan

• Manually confirmed each example file from the issue functioned as expected.
• Ran cargo nextest to ensure unexpected_special_method_signature test still passed.

Fixes #16217.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[MichaReiser]

@ntBre do you want to take this one?

[ntBre]

Sure!

[ntBre]

Thanks for working on this! The code changes look right to me, but I think it would be great to paste in all of the examples from the issue and add their snapshot results just to make sure.

I definitely agree that the module functions are good to leave for a separate rule, so this will be perfect with a few more tests.

[dcarrier]

Thank you for the review! I have added the test cases along with the snapshots as requested.

Side note, we are asserting __cmp__, __div__, __nonzero__, and __unicode__ are skipped by including failure cases which do not generate a snapshot.

[ntBre]

Nice, this looks great. Thanks again!