[prev in list] [next in list] [prev in thread] [next in thread] 

List:       sanlock-devel
Subject:    Re: [PATCH 0/2] Streamline tests output
From:       Nir Soffer <nsoffer () redhat ! com>
Date:       2019-06-17 13:47:40
Message-ID: CAMRbyystZ7QLie+Pnkfu-eUkc1WuUc1zNrSOSW6_EYokDS0k7w () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


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
>

[Attachment #5 (text/html)]

<div dir="ltr"><div dir="ltr"><div class="gmail_default" \
style="font-size:small;color:#000000"><br></div></div><br><div \
class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 17, 2019 at 4:05 PM \
Vojtech Juranek &lt;<a href="mailto:vjuranek@redhat.com">vjuranek@redhat.com</a>&gt; \
wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px \
0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex">LGTM<br></blockquote><div><br></div><div><div \
class="gmail_default" style="font-size:small;color:rgb(0,0,0)">Thanks, \
pushed</div><br></div><div>  </div><blockquote class="gmail_quote" style="margin:0px \
0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br>
&gt; Streamline tests output by using normal mode and not listing skipped<br>
&gt; tests in the short test summary.<br>
&gt; <br>
&gt; <br>
&gt; Here is example output from travis build:<br>
&gt; <br>
&gt; $ pytest<br>
&gt; ============================= test session starts<br>
&gt; ==============================<br>
  platform linux -- Python 3.7.1,<br>
&gt; pytest-4.3.1, py-1.7.0, pluggy-0.8.0 rootdir:<br>
&gt; /home/travis/build/nirs/sanlock, inifile: tox.ini<br>
&gt; collected 210 items                                                              \
<br> &gt;     <br>
  tests/daemon_test.py ...........                                                    \
<br> &gt;   [   5%] tests/direct_test.py ..                                           \
<br> &gt;         [   6%] tests/python_test.py<br>
&gt; ........ssssssssssss........ssssssssssssss......... [ 30%]<br>
&gt; ...............ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss. [<br>
&gt; 64%]<br>
&gt; ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.ss.......... [<br>
&gt; 99%] ..                                                                          \
<br> &gt;     [100%] ========================== slowest 10 test durations<br>
&gt; =========================== 6.02s call      <br>
&gt; tests/python_test.py::test_add_rem_lockspace_async 6.02s call      <br>
&gt; tests/python_test.py::test_add_rem_lockspace[1099511627776-1099510579200]<br>
&gt; 6.02s call        tests/python_test.py::test_add_rem_lockspace[1048576-0]<br>
&gt; 4.06s call      <br>
&gt; tests/python_test.py::test_acquire_release_resource[1048576-0] 4.05s call   <br>
&gt;   <br>
&gt; tests/python_test.py::test_acquire_release_resource[1099511627776-109951057<br>
&gt; 9200] 3.21s call        tests/daemon_test.py::test_create<br>
&gt; 3.12s call        tests/daemon_test.py::test_delete<br>
&gt; 3.09s call        tests/daemon_test.py::test_lookup<br>
&gt; 3.05s call        tests/daemon_test.py::test_lookup_missing<br>
&gt; 0.42s call        tests/daemon_test.py::test_start_after_kill<br>
&gt; =================== 104 passed, 106 skipped in 46.73 seconds<br>
&gt; ===================<br>
  The command &quot;pytest&quot; exited with 0.<br>
&gt; <br>
&gt; 0.60s$ flake8 --statistics tests python<br>
&gt; The command &quot;flake8 --statistics tests python&quot; exited with 0.<br>
&gt; <br>
&gt; <br>
&gt; Based on commit edf1e08326daa39a2e453d936fa9a99dd0ec136c (python:<br>
&gt; Convert example.py to python 3)<br>
&gt; <br>
&gt; <br>
&gt; Nir Soffer (2):<br>
&gt;     python: Don&#39;t report skipped tests<br>
&gt;     python: Use pytest normal mode<br>
&gt; <br>
&gt;   README.dev | 9 +++++++++<br>
&gt;   tox.ini      | 5 ++---<br>
&gt;   2 files changed, 11 insertions(+), 3 deletions(-)<br>
&gt; <br>
&gt; -- <br>
&gt; 2.17.2<br>
&gt; _______________________________________________<br>
&gt; sanlock-devel mailing list -- <a \
href="mailto:sanlock-devel@lists.fedorahosted.org" \
target="_blank">sanlock-devel@lists.fedorahosted.org</a><br> &gt; To unsubscribe send \
an email to <a href="mailto:sanlock-devel-leave@lists.fedorahosted.org" \
target="_blank">sanlock-devel-leave@lists.fedorahosted.org</a><br> &gt; Fedora Code \
of Conduct:<br> &gt; <a \
href="https://docs.fedoraproject.org/en-US/project/code-of-conduct/" rel="noreferrer" \
target="_blank">https://docs.fedoraproject.org/en-US/project/code-of-conduct/</a> \
List<br> &gt; Guidelines: <a \
href="https://fedoraproject.org/wiki/Mailing_list_guidelines" rel="noreferrer" \
target="_blank">https://fedoraproject.org/wiki/Mailing_list_guidelines</a> List<br> \
&gt; Archives:<br> &gt; <a \
href="https://lists.fedorahosted.org/archives/list/sanlock-devel@lists.fedorahost" \
rel="noreferrer" target="_blank">https://lists.fedorahosted.org/archives/list/sanlock-devel@lists.fedorahost</a><br>
 &gt; <a href="http://ed.org" rel="noreferrer" target="_blank">ed.org</a><br>
<br>
_______________________________________________<br>
sanlock-devel mailing list -- <a href="mailto:sanlock-devel@lists.fedorahosted.org" \
target="_blank">sanlock-devel@lists.fedorahosted.org</a><br> To unsubscribe send an \
email to <a href="mailto:sanlock-devel-leave@lists.fedorahosted.org" \
target="_blank">sanlock-devel-leave@lists.fedorahosted.org</a><br> Fedora Code of \
Conduct: <a href="https://docs.fedoraproject.org/en-US/project/code-of-conduct/" \
rel="noreferrer" target="_blank">https://docs.fedoraproject.org/en-US/project/code-of-conduct/</a><br>
 List Guidelines: <a href="https://fedoraproject.org/wiki/Mailing_list_guidelines" \
rel="noreferrer" target="_blank">https://fedoraproject.org/wiki/Mailing_list_guidelines</a><br>
 List Archives: <a href="https://lists.fedorahosted.org/archives/list/sanlock-devel@lists.fedorahosted.org" \
rel="noreferrer" target="_blank">https://lists.fedorahosted.org/archives/list/sanlock-devel@lists.fedorahosted.org</a><br>
 </blockquote></div></div>


[Attachment #6 (text/plain)]

_______________________________________________
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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic