838075 - mozprocess tests should clean up after themselves

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Jeff Hammel]

In hooking up the mozprocess tests for `make check` (bug 790765), it
is observed that mozprocess tests leave around some litter:

"""
│hg st
? testing/mozbase/mozprocess/tests/iniparser/dictionary.o
? testing/mozbase/mozprocess/tests/iniparser/iniparser.o
? testing/mozbase/mozprocess/tests/iniparser/libiniparser.a
? testing/mozbase/mozprocess/tests/iniparser/libiniparser.so.0
? testing/mozbase/mozprocess/tests/proclaunch
"""

While this has slid by for development purposes, we don't want this on
m-c.  Instead the mozprocess tests should clean up after themselves.
If there is a case where we want to keep these around, we can add a
command line flag to do so or similar, but for the 99% use case we
want these gone.

See also https://bugzilla.mozilla.org/show_bug.cgi?id=778267

Comment 1

[Jeff Hammel]

Attachment: clean up and some more stuff

So this kinda falls into the category of "a lot more than needs to be done for bug 838075, not nearly enough to fix bug 778267" In addition to using setUpClass and tearDownClass which are 2.7isms, I've combined these into one file and class as I see no reason they should't be.  Consequently, some test names have changed.  I renamed the whole test_mozprocess.py instead of "mozprocess1.py" or what not.

As for the things that I didn't clean up, I'll write you a novella sometime.

Comment 2

[William Lachance (:wlach)]

Comment on attachment 722579 [details] [diff] [review]
clean up and some more stuff

Nice.

Comment 3

[Jeff Hammel]

pushed: https://github.com/mozilla/mozbase/commit/3a628434ad9a508d0a3ae8b29ca196bbef9d3025