jiridanek commented on pull request #1102: URL: https://github.com/apache/qpid-dispatch/pull/1102#issuecomment-813030070
Going over this again, it looks to me that all the necessary pieces are there. Just few small details missing, around it. ## TODO * get rid of this silly warning ``` 34: =============================== warnings summary =============================== 34: ../../tests/system_test.py:1310 34: /foo/qpid-dispatch/tests/system_test.py:1310: PytestCollectionWarning: cannot collect test class 'TestTimeout' because it has a __init__ constructor (from: ) 34: class TestTimeout(object): 34: 34: -- Docs: https://docs.pytest.org/en/stable/warnings.html 34: - generated xml file: /foo/qpid-dispatch/build/tests/junitxmls/system_tests_two_routers.xml - 34: =================== 32 passed, 1 warning in 64.90s (0:01:04) =================== ``` * figure out how best to install pytest on RHEL 7 * decide on precise pytest command we want, it should probably include `--junit-prefix=` * use this in ASF Jenkins? * does it still work without `tox`? (see related cmake magic) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
