Add regression tests for Enum.name and .value inference

[nelfin]

Steps

• Add yourself to CONTRIBUTORS if you are a new contributor.
• Add a ChangeLog entry describing what your PR does.
• If it's a new feature or an important bug fix, add a What's New entry in
  doc/whatsnew/<current release.rst>.
• Write a good description on what the PR does.

Description

Captures the regression tests specified in #1932 and #2062 and fixed by pylint-dev/astroid#1020.

Type of Changes

	Type
✓	📜 Docs

Related Issue

Closes #1932. Closes #2062. Depends on pylint-dev/astroid#1020.

[cdce8p]

@nelfin Feel free to add Closes #... to the PR description. Otherwise the issues will stay open, probably forever.
Could you add a small note to the ChangeLog? I imagine not many will read the one for astroid.

Lastly, I saw by accident that #2306 appears to be fixed as well, at least for Python < 3.10. Something changed of for 3.10 though.

[nelfin]

@nelfin Feel free to add Closes #... to the PR description. Otherwise the issues will stay open, probably forever.

This PR doesn't actually fix those issues though?

Could you add a small note to the ChangeLog? I imagine not many will read the one for astroid.

Sure.

Lastly, I saw by accident that #2306 appears to be fixed as well, at least for Python < 3.10. Something changed of for 3.10 though.

The implementation of enum.Enum changed in 3.10 away from just a plain DynamicClassAttribute. I guess I can take a look at what it takes to get that test to pass.

[cdce8p]

@nelfin Feel free to add Closes #... to the PR description. Otherwise the issues will stay open, probably forever.

This PR doesn't actually fix those issues though?

I thought it did. If you meant that pylint-dev/astroid#1020 actually fixed it, that is no problem. I frequently use Closes ... only for the last PR in pylint. As long as they are related.

Could you add a small note to the ChangeLog? I imagine not many will read the one for astroid.

Sure.

Lastly, I saw by accident that #2306 appears to be fixed as well, at least for Python < 3.10. Something changed of for 3.10 though.

The implementation of enum.Enum changed in 3.10 away from just a plain DynamicClassAttribute. I guess I can take a look at what it takes to get that test to pass.

Thanks 👍🏻

[nelfin]

I thought it did. If you meant that PyCQA/astroid#1020 actually fixed it, that is no problem.

That is what I meant. I've updated the description.

[Pierre-Sassoulas]

👍

[cdce8p]

@nelfin Thanks for fixing the issue with 3.10! Would you mind adding the regression test for it here? Just so we don't lose it

[nelfin]

@cdce8p, do you mean this regression test: #4570? I opened that one when I pushed up the fix, but I can squash that PR into this one if that's your preference.

[cdce8p]

@cdce8p, do you mean this regression test: #4570? I opened that one when I pushed up the fix, but I can squash that PR into this one if that's your preference.

That works as well! Didn't saw it before.

[coveralls]

Coverage remained the same at 92.037% when pulling c60fddd on nelfin:fix/1932+2062-enum-name-member into e3010e6 on PyCQA:master.