It seems the tests do not run. @psss any pointers?

Try renaming to tests/tests_python.yml.

Yes, Standard Test Interface requires the tests* prefix.

[citest]

It seems all the tests passed, including asyncio which actually failed initially here:

ERROR: test_start_tls_server_1 (test.test_asyncio.test_sslproto.SelectorStartTLSTests)
Traceback (most recent call last):
File "/usr/lib64/python3.7/test/test_asyncio/test_sslproto.py", line 507, in test_start_tls_server_1
self.loop.run_until_complete(run_main())
File "/usr/lib64/python3.7/asyncio/base_events.py", line 584, in run_until_complete
return future.result()
File "/usr/lib64/python3.7/test/test_asyncio/test_sslproto.py", line 502, in run_main
loop=self.loop, timeout=self.TIMEOUT)
File "/usr/lib64/python3.7/asyncio/tasks.py", line 423, in wait_for
raise futures.TimeoutError()
concurrent.futures._base.TimeoutError

But then on the verbose mode rerun it passed.

Note to self: remove gcc and see if things work.

[citest]

[citest]

[citest]

This seems good to go for now (will squash the commits later on), but test_asyncio fails due to https://bugs.python.org/issue35998

call it differently? e.g. python_selftest or python. So in the artifacts section on the CI etc. people know what selftest it is?

[citest]

[citest]

+ python3.7 -m test -wW -j0 test_ssl test_hashlib test_asyncio test_ftplib test_httplib test_imaplib test_logging test_nntplib test_poplib test_urllib2_localnet test_urllib test_xmlrpc
Run tests in parallel using 23 child processes
0:00:04 load avg: 0.93 [ 1/12] test_nntplib passed
0:00:05 load avg: 0.93 [ 2/12] test_urllib passed
0:00:05 load avg: 0.93 [ 3/12] test_httplib passed
0:00:06 load avg: 0.93 [ 4/12] test_urllib2_localnet passed
0:00:10 load avg: 0.85 [ 5/12] test_hashlib passed
0:00:11 load avg: 0.85 [ 6/12] test_poplib passed
0:00:12 load avg: 0.85 [ 7/12] test_imaplib passed
0:00:13 load avg: 0.85 [ 8/12] test_xmlrpc passed
0:00:14 load avg: 0.94 [ 9/12] test_ftplib passed
0:00:15 load avg: 0.94 [10/12] test_ssl passed
0:00:31 load avg: 0.81 [11/12] test_logging passed -- running: test_asyncio (31 sec 35 ms)
0:00:50 load avg: 0.58 [12/12] test_asyncio passed (48 sec 464 ms)

== Tests result: SUCCESS ==

All 12 tests OK.

Total duration: 50 sec 726 ms
Tests result: SUCCESS

@cstratak Let's un-WIP this?

squashed and ready for review.

[citest]

is tkinter needed for ssl tests?

[citest]

Is gcc needed for the tests listed?

In theory, if you use VERSION=3 instead of 3.7, this should work with the "normal" python3 package and when we update it to 3.8, we don't need to go back here and change it. Untested.

They are indeed needed due to running pythoninfo

Alright. Will try it out after that CI run finishes.

oh damn broken pipeline again

[citest]

pipeline's broken very much on rawhide. it seems it relied on yum.

https://pagure.io/fedora-ci/general/issue/43#comment-563992

======================================================================
ERROR: test_start_tls_server_1 (test.test_asyncio.test_sslproto.SelectorStartTLSTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib64/python3.7/test/test_asyncio/test_sslproto.py", line 507, in test_start_tls_server_1
    self.loop.run_until_complete(run_main())
  File "/usr/lib64/python3.7/asyncio/base_events.py", line 584, in run_until_complete
    return future.result()
  File "/usr/lib64/python3.7/test/test_asyncio/test_sslproto.py", line 502, in run_main
    loop=self.loop, timeout=self.TIMEOUT)
  File "/usr/lib64/python3.7/asyncio/tasks.py", line 423, in wait_for
    raise futures.TimeoutError()
concurrent.futures._base.TimeoutError

----------------------------------------------------------------------

That test is flaky for the time being and upstream is aware of that.

https://bugs.python.org/issue35998

What would you think, better to skip test_asyncio for now or just merge as is? IMHO let's not skip it, upstream is aware of it and when it's fixed, we'll not need to adjust the tests in this repo.

If we merge when flaky, the test results are useless. I'd skip for now and remember to reenable later.

Done.

[citest]

LGTM

@tmraz can we merge this?

LGTM as well.