Actually run the tests in python 3.12 and 3.13 and remove snapshottest dependency

[PoByBolek]

I noticed that 614449e (PR #1550) introduced an invalid tox environment: py12 instead of py312. This mistake was later repeated in 6834385 (PR #1561): py13 instead of py313.

On my machine, tox list outputs:

default environments:
py38       -> [no description]
py39       -> [no description]
py310      -> [no description]
py311      -> [no description]
py312      -> [no description]
py313      -> [no description]
mypy       -> [no description]
pre-commit -> [no description]

additional environments:
py12       -> [no description]
py13       -> [no description]

And tox run doesn't run any tests in the py312 and py313 environments. This is also true in the Github workflows.

For example, in this "successful" run in the py312 environment from 2024-07-01 tox only runs pip install .[test] and nothing else.

This looks very different from this run in the py311 environment, which also runs pytest:

---

This PR actually runs the tests in the py312 and py313 and drops the snapshottest dependency (which was only used in the example tests) so that the tests also work in Python 3.12 and above. This is necessary because snapshottest (which appears dead) is using Python's imp module, which has been deprected since 3.4 and was removed in 3.12.

[PoByBolek]

From what I can tell there are at least two open pull requests in snapshottest that could fix our test failures in py312 and py313:

• Use importlib (Python 3.12 compatible) syrusakbary/snapshottest#173
• Use importlib instead of imp syrusakbary/snapshottest#168

[PoByBolek]

I also dropped the snapshottest dependency and simply moved the expectactions from the snap_*.py files to the corresponding test cases in the example tests. So the tests should now work across all environments again 😄

[dulmandakh]

I did notice that tests on Python 3.12 are not working as expected, but hadn't chance to dig into it. Thank you 🙏

I agree that Graphene needs to drop snapshottest as it's failing to work on Python 3.12 and above. I'll try to take time and review your changes.

[dulmandakh]

LGTM

[PoByBolek]

@erikwrede ping?

[erikwrede]

Sorry for the delay, currently really busy in my day-to-day.

Thanks for picking this up and pinging me!

[erikwrede]

Seems like we have some test failures in 3.13. Will investigate later.

[PoByBolek]

looks like the json module raises a new error message for trailing commas since python/cpython@cfa25fe (Python 3.13.0a3)

[erikwrede]

Thank you!