On Mon, Jun 17, 2019 at 4:05 PM Vojtech Juranek <vjuranek@redhat.com> wrote:
LGTM

Thanks, pushed

 

> Streamline tests output by using normal mode and not listing skipped
> tests in the short test summary.
>
>
> Here is example output from travis build:
>
> $ pytest
> ============================= test session starts
> ==============================
 platform linux -- Python 3.7.1,
> pytest-4.3.1, py-1.7.0, pluggy-0.8.0 rootdir:
> /home/travis/build/nirs/sanlock, inifile: tox.ini
> collected 210 items                                                         
>   
 tests/daemon_test.py ...........                                       
>  [  5%] tests/direct_test.py ..                                           
>      [  6%] tests/python_test.py
> ........ssssssssssss........ssssssssssssss......... [ 30%]
> ...............ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss. [
> 64%]
> ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.......... [
> 99%] ..                                                                   
>   [100%] ========================== slowest 10 test durations
> =========================== 6.02s call   
> tests/python_test.py::test_add_rem_lockspace_async 6.02s call   
> tests/python_test.py::test_add_rem_lockspace[1099511627776-1099510579200]
> 6.02s call     tests/python_test.py::test_add_rem_lockspace[1048576-0]
> 4.06s call   
> tests/python_test.py::test_acquire_release_resource[1048576-0] 4.05s call 

> tests/python_test.py::test_acquire_release_resource[1099511627776-109951057
> 9200] 3.21s call     tests/daemon_test.py::test_create
> 3.12s call     tests/daemon_test.py::test_delete
> 3.09s call     tests/daemon_test.py::test_lookup
> 3.05s call     tests/daemon_test.py::test_lookup_missing
> 0.42s call     tests/daemon_test.py::test_start_after_kill
> =================== 104 passed, 106 skipped in 46.73 seconds
> ===================
 The command "pytest" exited with 0.
>
> 0.60s$ flake8 --statistics tests python
> The command "flake8 --statistics tests python" exited with 0.
>
>
> Based on commit edf1e08326daa39a2e453d936fa9a99dd0ec136c (python:
> Convert example.py to python 3)
>
>
> Nir Soffer (2):
>   python: Don't report skipped tests
>   python: Use pytest normal mode
>
>  README.dev | 9 +++++++++
>  tox.ini    | 5 ++---
>  2 files changed, 11 insertions(+), 3 deletions(-)
>
> --
> 2.17.2
> _______________________________________________
> sanlock-devel mailing list -- sanlock-devel@lists.fedorahosted.org
> To unsubscribe send an email to sanlock-devel-leave@lists.fedorahosted.org
> Fedora Code of Conduct:
> https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List
> Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List
> Archives:
> https://lists.fedorahosted.org/archives/list/sanlock-devel@lists.fedorahost
> ed.org

_______________________________________________
sanlock-devel mailing list -- sanlock-devel@lists.fedorahosted.org
To unsubscribe send an email to sanlock-devel-leave@lists.fedorahosted.org
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedorahosted.org/archives/list/sanlock-devel@lists.fedorahosted.org