Add Python 3.12 to test matrix and add classifiers

[foarsitter]

djangorestframework-simplejwt is not yet compatible with Python 3.12. Took me a while to figure out that the real errors gets silenced.

drf-spectacular/tests/contrib/test_simplejwt.py

Lines 8 to 18 in 6e48a70

	try:
	from rest_framework_simplejwt.authentication import (
	JWTAuthentication, JWTTokenUserAuthentication,
	)
	from rest_framework_simplejwt.views import (
	TokenObtainPairView, TokenObtainSlidingView, TokenRefreshView, TokenVerifyView,
	)
	except ImportError:
	JWTAuthentication = None

Is this is something acceptable in a different PR I'm happily to submit.

except ImportError as e:
    if "rest_framework_simplejwt" in str(e):
        JWTAuthentication = None
    else:
        raise e

[foarsitter]

Seems they have already fixed it: jazzband/djangorestframework-simplejwt#754

[foarsitter]

After installing setuptools the next error I'm running into is with dj_rest_auth buth the error doesn't make any sense to me.

    def get_token_serializer_class():
>       from dj_rest_auth.app_settings import JWTSerializer, TokenSerializer
E       ImportError: cannot import name 'JWTSerializer' from 'dj_rest_auth.app_settings' 

[tfranzel]

Good question. I suppose the ecosystem is not there yet with 3.12

I have this issue on linux: https://stackoverflow.com/questions/77364550/attributeerror-module-pkgutil-has-no-attribute-impimporter-did-you-mean

And on mac I have pip issues with installing psyopg2-binary

So I couldn't even reproduce the issue here in the pipeline.

[tfranzel]

looks like this is essentially only blocked by jazzband/djangorestframework-simplejwt#762

We have a bug in the except clause. It is missing

JWTAuthentication = None
+ JWTTokenUserAuthentication = None

then everything "works", except simplejwt and 2 other contrib libraries that have interdependent imports going on.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ad7e4a0) 98.58% compared to head (c515633) 98.58%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1096   +/-   ##
=======================================
  Coverage   98.58%   98.58%           
=======================================
  Files          68       68           
  Lines        8462     8462           
=======================================
  Hits         8342     8342           
  Misses        120      120           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tfranzel]

looks good to me. thx @foarsitter.

though I'm a bit confused why simplejwt now passes for 3.12, given that they have not released and we didn't change much. I'll merge it.

[foarsitter]

Thats indeed strange. There has to be a change in a underlying lib or something.

Thanks for merging!