838079 - get memtest.py on mozbase or kill it

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Jeff Hammel]

http://hg.mozilla.org/mozilla-central/file/2360c3c46aca/testing/tests/memtest.py

A few things:

* I'm not sure if this file is used or not.  Casual MXRing suggests
  not.  If this is the case, we should just kill the testing/tests
  directory

* Speaking of, testing/tests is an awful name for where this is.
  Let's put it somewhere better if we keep it.  I suggest
  testing/memtest

* Also, if there was a README I'd probably have a better idea if it
  was still relevent.  If we keep it, lets add a README

* And some docstrings and other in-code documentation

* There is some overlap between this file and mozbase: Firefox_runner,
  etc. We should make use of the relevent mozbase software for this.

* And let's change the shebang to /usr/bin/env python

* Instead of having a main section delimited by comments,
  http://hg.mozilla.org/mozilla-central/file/2360c3c46aca/testing/tests/memtest.py#l431
  , lets have a main function that is called if __name__ == '__main__'

*
  http://hg.mozilla.org/mozilla-central/file/2360c3c46aca/testing/tests/memtest.py#l445
  : to mozprocess if its worth it

Comment 1

[Jeff Hammel]

Really a question to anyone but can this be killed?

Comment 2

[Jeff Hammel]

Attachment: remove it

Comment 3

[Jeff Hammel]

pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=aee04d0eeff0

Comment 4

[Jeff Hammel]

ted informed me that this file is obselete

Comment 5

[Jeff Hammel]

thanks, ted.  pushed to inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/0780ca1d62d4

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/0780ca1d62d4