[libvirt] [dbus PATCH] gitignore: add tests/.pytest_cache

Andrea Bolognani abologna at redhat.com
Mon Jul 9 13:52:08 UTC 2018


[re-adding libvirt-list]

On Mon, 2018-07-09 at 09:23 -0400, Anya Harter wrote:
> On 07/09/2018 04:34 AM, Andrea Bolognani wrote:
> > On Thu, 2018-07-05 at 13:48 -0400, Anya Harter wrote:
> > > +/tests/.pytest_cache
> > 
> > When does the directory (or file?) get created? I just ran 'make
> > test' but it's not there for me.
> 
> The tests/.pytest_cache/ directory is created when make check is run and
> the test suite fails.
> 
> Additionally, a .pytest_cache/ directory is created in the home
> directory if ./run tests/<testfile> fails.

s/home directory/top-level directory of the project/ ;)

> If you want I can give you a patch that breaks the test suite and
> creates the directories in the two ways above.

No need: I verified the above on my own, and in that light...

> > Either way, this looks like it should be a more generic exclude
> > pattern like the one we already have for __pycache__ directories
> > instead of being limited to tests/.

... this suggestion was spot-on, as adding the pattern you
suggested would take care of the failed 'make check' case but
not of the failed './run tests/...' case.

Looking forward to v2 :)

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list