Tests: Python 3.12 enable warnings as errors (#1027)

Author: robsdedude

testkit/_common.py

@@ -41,11 +41,6 @@ def get_python_version():
 
 def run_python(args, env=None, warning_as_error=True):
     cmd = [TEST_BACKEND_VERSION, "-u"]
-    if get_python_version() >= (3, 12):
-        # Ignore warnings for Python 3.12 for now
-        # https://github.com/dateutil/dateutil/issues/1284 needs to be released
-        # and propagate through our dependency graph
-        warning_as_error = False
     if warning_as_error:
         cmd += ["-W", "error"]
     cmd += list(args)

testkitbackend/__main__.py

@@ -47,11 +47,7 @@ async def main():
 
 
 if __name__ == "__main__":
-    if sys.version_info < (3, 12):
-        # Ignore warnings for Python 3.12 for now
-        # https://github.com/dateutil/dateutil/issues/1284 needs to be released
-        # and propagate through our dependency graph
-        warnings.simplefilter("error")
+    warnings.simplefilter("error")
     if len(sys.argv) == 2 and sys.argv[1].lower().strip() == "async":
         async_main()
     else:

tox.ini

@@ -6,12 +6,8 @@ passenv = TEST_NEO4J_*
 deps = -r requirements-dev.txt
 setenv = COVERAGE_FILE={envdir}/.coverage
 usedevelop = true
-# Ignore warnings for Python 3.12 for now
-# https://github.com/dateutil/dateutil/issues/1284 needs to be released
-# and propagate through our dependency graph
 warnargs =
-    py{37,38,39,310,311}: -W error
-    py312:
+    py{37,38,39,310,311,312}: -W error
 commands =
     coverage erase
     unit: coverage run -m pytest {[testenv]warnargs} -v {posargs} tests/unit