Make TrialDirMixIn so it can be used stand alone.

tests/trial_dir.py

 #!/usr/bin/env python
 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 
 import atexit
 import logging
 import os
 import sys
@@ skipping 47 matching lines @@
   @staticmethod
   def _clean():
     """Cleans the root trial directory."""
     if not TrialDir.SHOULD_LEAK:
       logging.debug('Removing %s' % TrialDir.TRIAL_ROOT)
       gclient_utils.RemoveDirectory(TrialDir.TRIAL_ROOT)
     else:
       logging.error('Leaking %s' % TrialDir.TRIAL_ROOT)
 
 
-class TestCase(unittest.TestCase):
-  """Base unittest class that cleans off a trial directory in tearDown()."""
+class TrialDirMixIn(object):
+  """Stand alone mixin."""

[Dirk Pranke, 2011/06/14 06:22:09]

This comment doesn't tell me anything about what this is, and worse, I think
these three words are self-contradictory. I'd probably just delete it.

   def setUp(self):
     # Create a specific directory just for the test.
     self.trial = TrialDir(self.id())
     self.trial.set_up()
 
   def tearDown(self):
     self.trial.tear_down()
 
   @property
   def root_dir(self):
     return self.trial.root_dir
 
 
+class TestCase(unittest.TestCase, TrialDirMixIn):
+  """Base unittest class that cleans off a trial directory in tearDown()."""
+  def setUp(self):
+    unittest.TestCase.setUp(self)
+    TrialDirMixIn.setUp(self)
+
+  def tearDown(self):
+    TrialDirMixIn.tearDown(self)
+    unittest.TestCase.tearDown(self)
+
+
 if '-l' in sys.argv:
   # See SHOULD_LEAK definition in TrialDir for its purpose.
   TrialDir.SHOULD_LEAK = True
   print 'Leaking!'
   sys.argv.remove('-l')