With the upcoming changes to make /home/autotest a mount pt for /mnt/stateful_partition/autotest, using os.rename in logging_CrashSender causes "OSError: [Errno 18] Invalid cross-device link". I've switched these calls to use shutil.move instead.
Change-Id: I554b27e4c649867481e9aee76bd752ca90dcf582
BUG=chromium-os:8336
TEST=Ran test with new framework. Verified pass.
Review URL: http://codereview.chromium.org/5041001
diff --git a/client/bin/site_crash_test.py b/client/bin/site_crash_test.py
index 9f2d52e..cc3e97f 100644
--- a/client/bin/site_crash_test.py
+++ b/client/bin/site_crash_test.py
@@ -2,7 +2,7 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-import logging, os, re
+import logging, os, re, shutil
from autotest_lib.client.bin import site_log_reader, site_utils, test
from autotest_lib.client.common_lib import error, utils
@@ -85,13 +85,15 @@
def _push_consent(self):
if os.path.exists(self._CONSENT_FILE):
- os.rename(self._CONSENT_FILE, self._get_pushed_consent_file_path())
+ shutil.move(self._CONSENT_FILE,
+ self._get_pushed_consent_file_path())
def _pop_consent(self):
self._set_consent(False)
if os.path.exists(self._get_pushed_consent_file_path()):
- os.rename(self._get_pushed_consent_file_path(), self._CONSENT_FILE)
+ shutil.move(self._get_pushed_consent_file_path(),
+ self._CONSENT_FILE)
def _get_crash_dir(self, username):