Modify autotest to support preserving minidumps generated during tests.

client/cros/crash_test.py

Index: client/cros/crash_test.py
diff --git a/client/cros/crash_test.py b/client/cros/crash_test.py
index 774f5bbf9050e0d221b343345ccb8ac98a75bbf9..069fdde62a3ecdf20026480a38291ba9b3816b46 100644
--- a/client/cros/crash_test.py
+++ b/client/cros/crash_test.py
@@ -17,6 +17,7 @@ class CrashTest(test.test):
     _CRASH_SENDER_PATH = '/sbin/crash_sender'
     _CRASH_SENDER_RATE_DIR = '/var/lib/crash_sender'
     _CRASH_SENDER_RUN_PATH = '/var/run/crash_sender.pid'
+    _CRASH_TEST_IN_PROGRESS = '/tmp/crash-test-in-progress'

[kmixter1, 2011/02/23 02:00:39]

This is similar in use to _MOCK_CRASH_SENDING.  I can believe you can't reuse
it, but just want to make sure you've verified it.

[ericli, 2011/02/23 04:49:34]

any reason why not in constants.py?
On 2011/02/23 02:00:39, kmixter1 wrote:

[thieule, 2011/03/03 01:27:41]

_MOCK_CRASH_SENDING is set after the crash but before sending the info.  This is
unfortunately too late since I need to know if we're running crash tests while
handling the crash in UserCollector::HandleCrash().

[thieule, 2011/03/03 01:27:41]

I'm going to leave this constant in this file to be consistent with the rest of
the constants here.  We can have a separate CL that cleans up all these private
constants.

     _MOCK_CRASH_SENDING = '/tmp/mock-crash-sending'
     _PAUSE_FILE = '/var/lib/crash_sender_paused'
     _SYSTEM_CRASH_DIR = '/var/spool/crash'
@@ -81,6 +82,14 @@ class CrashTest(test.test):
             utils.system('rm -f "%s"' % (self._CONSENT_FILE))
 
 
+    def _set_crash_test_in_progress(self, in_progress):
+        if in_progress:
+            utils.open_write_close(self._CRASH_TEST_IN_PROGRESS, 'in-progress')
+            logging.info('Created ' + self._CRASH_TEST_IN_PROGRESS)
+        else:
+            utils.system('rm -f "%s"' % (self._CRASH_TEST_IN_PROGRESS))
+
+
     def _get_pushed_consent_file_path(self):
         return os.path.join(self.bindir, 'pushed_consent')
 
@@ -313,6 +322,7 @@ class CrashTest(test.test):
         self._leave_crash_sending = True
         self._automatic_consent_saving = True
         self.enable_crash_filtering('none')
+        self._set_crash_test_in_progress(True)
 
 
     def cleanup(self):
@@ -323,6 +333,7 @@ class CrashTest(test.test):
         if self._automatic_consent_saving:
             self._pop_consent()
         self.disable_crash_filtering()
+        self._set_crash_test_in_progress(False)
         test.test.cleanup(self)