Closed
Description
Bug report
Bug description:
See https://github.com/apache/thrift/blob/5cf71b2beec3c67a4c8452ddabbbc6ae43fff16f/lib/py/test/test_sslsocket.py for a full example, we skipped all tests in that file via:
@unittest.skip("failing SSL test to be fixed in subsequent pull request") class TSSLSocketTest(unittest.TestCase): ...On python 3.11 this is fine with:
/usr/bin/python3 test/test_sslsocket.py sssssssssss ---------------------------------------------------------------------- Ran 11 tests in 0.000s OK (skipped=11) But on python 3.12, this starts to become an error:
/opt/hostedtoolcache/Python/3.12.1/x64/bin/python test/test_sslsocket.py WARNING:thrift.transport.sslcompat:using legacy validation callback sssssssssss ---------------------------------------------------------------------- Ran 0 tests in 0.000s NO TESTS RAN (skipped=11) make[1]: *** [Makefile:645: py3-test] Error 5 I have to add a not-skipped dummy test to work around this (https://github.com/apache/thrift/pull/2914/files)
Is this an intentional change? I found it weird that we consider a skipped test as a failure.
CPython versions tested on:
3.12
Operating systems tested on:
Linux