Currently each runcppunittests.py expects each test to run to be listed separately on the command line, which is unwieldy when we want to run all of the tests in the test package in bug 811409. Adding a --test-path option would clean this up.

I think the simplest thing would be to not add a new option, but accept either test binaries or directories on the command line, and expand directories out to their contents so that we run all test binaries in a directory.

Changed summary as per Ted's comment.

Comment on attachment 774094 [details] [diff] [review] Patch to add support for specifying directories on the command line Review of attachment 774094 [details] [diff] [review]: ----------------------------------------------------------------- ::: testing/runcppunittests.py @@ +141,5 @@ > sys.exit(1) > + progs = [] > + for p in args: > + if os.path.isdir(p): > + progs.extend([os.path.abspath(os.path.join(p, x)) for x in os.listdir(p) if not x.endswith('.py')]) A little funky to only exclude .py files, but I'm guessing you actually needed this, so it's fine.

Thanks! I added a comment to explain exclusion of .py files (necessary due to the way we packaged the cppunittests in the test archive.) https://hg.mozilla.org/integration/mozilla-inbound/rev/1eb8567e0743