fix typed choices, make working with different Django 5x choices options

[GeyseR]

An attempt to fix #1476

While working on the fix, I realized that Graphene doesn't support choice fields initialized with a types class itself without calling choices. Fixed this as well.

Django 5.1 introduced a convenient normalize_choices method that I use if a newer django version installed

[sjdemartini]

Thanks for the PR. You mention that this should fix #1476. Could you please add a test or two accordingly for the mutation behavior, to ensure it's fixed and won't regress in the future?

[sjdemartini]

Looks like this effectively replaces BlankValueField. Can you add a comment or something stating as much, and "marking" BlankValueField as deprecated? I assume it's effectively only ever been used internally, but we shouldn't remove it without a major version bump, unfortunately.

[GeyseR]

actually, my idea was to inherit from the BlankValueField class, but I forgot to. The interesting thing is that I've seen the test that was added when this class was added (test_choice_enum_blank_value), but it didn't catch the bug in my code as the test itself was wrong (the field in the test instance was null instead of blank).
I've tested a bug in the test and classes inheritance, so they match my initial idea, but if you want me to move all the logic to the EnumValueField and deprecate BlankValueField, I can do that

[GeyseR]

@sjdemartini how do you want to resolve this one?

[sjdemartini]

I think what you have (with inheritance from BlankValueField) is fine 👍

[GeyseR]

You mention that this should fix #1476. Could you please add a test or two accordingly for the mutation behavior, to ensure it's fixed and won't regress in the future?

the source of the problem was that the library incorrectly resolved typed choice values assigned to model choice fields (which is a totally valid case and can happen not only in mutations). I've added a test_typed_choice_value test that covers that

[sjdemartini]

You mention that this should fix #1476. Could you please add a test or two accordingly for the mutation behavior, to ensure it's fixed and won't regress in the future?

the source of the problem was that the library incorrectly resolved typed choice values assigned to model choice fields (which is a totally valid case and can happen not only in mutations). I've added a test_typed_choice_value test that covers that

Would you be able to add a test for the mutation specifically, for comprehensiveness and good measure?

[GeyseR]

Would you be able to add a test for the mutation specifically, for comprehensiveness and good measure?

extended the added test with the mutation call. Thanks for the suggestion!

[sjdemartini]

Sorry for the delay, thanks again for the work on this and for adding the extra mutation test!

[sjdemartini]

I think what you have (with inheritance from BlankValueField) is fine 👍