Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(358)

Side by Side Diff: client/site_tests/logging_UserCrash/logging_UserCrash.py

Issue 6090004: Move Autotest to /usr/local/autotest and fix hard-coded autodir paths in client tests. (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/autotest.git@master
Patch Set: Merge with HEAD. Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved. 1 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import grp, logging, os, pwd, re, stat, subprocess 5 import grp, logging, os, pwd, re, stat, subprocess
6 from signal import SIGSEGV 6 from signal import SIGSEGV
7 from autotest_lib.client.bin import site_crash_test, test, utils 7 from autotest_lib.client.bin import site_crash_test, test, utils
8 from autotest_lib.client.common_lib import error 8 from autotest_lib.client.common_lib import error
9 9
10 _COLLECTION_ERROR_SIGNATURE = 'crash_reporter-user-collection' 10 _COLLECTION_ERROR_SIGNATURE = 'crash_reporter-user-collection'
(...skipping 566 matching lines...) Expand 10 before | Expand all | Expand 10 after
577 if not ('log=' + result['log']) in utils.read_file(result['meta']): 577 if not ('log=' + result['log']) in utils.read_file(result['meta']):
578 raise error.TestFail('Meta file does not reference log') 578 raise error.TestFail('Meta file does not reference log')
579 579
580 580
581 def _test_crash_log_infinite_recursion(self): 581 def _test_crash_log_infinite_recursion(self):
582 recursion_triggering_crasher = os.path.join( 582 recursion_triggering_crasher = os.path.join(
583 os.path.dirname(self.bindir), 'crash_log_recursion_test') 583 os.path.dirname(self.bindir), 'crash_log_recursion_test')
584 # The configuration file hardcodes this path, so make sure it's still 584 # The configuration file hardcodes this path, so make sure it's still
585 # the same. 585 # the same.
586 if (recursion_triggering_crasher != 586 if (recursion_triggering_crasher !=
587 '/home/autotest/tests/crash_log_recursion_test'): 587 os.path.join(self.autodir, 'tests/crash_log_recursion_test')):
kmixter1 2011/01/07 01:30:04 This isn't the right change. There is in the base
DaleCurtis 2011/01/07 01:48:57 Symlink seems dirty. I fixed path in .conf file in
588 raise error.TestError('Path to recursion test changed') 588 raise error.TestError('Path to recursion test changed')
589 # Copy crasher_path to a test location with correct mode and a 589 # Copy crasher_path to a test location with correct mode and a
590 # special name to trigger crash log creation. 590 # special name to trigger crash log creation.
591 utils.system('cp -a "%s" "%s"' % (self._crasher_path, 591 utils.system('cp -a "%s" "%s"' % (self._crasher_path,
592 recursion_triggering_crasher)) 592 recursion_triggering_crasher))
593 # Simply completing this command means that we avoided 593 # Simply completing this command means that we avoided
594 # infinite recursion. 594 # infinite recursion.
595 result = self._run_crasher_process( 595 result = self._run_crasher_process(
596 'root', crasher_path=recursion_triggering_crasher) 596 'root', crasher_path=recursion_triggering_crasher)
597 597
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
671 'root_crasher_no_consent', 671 'root_crasher_no_consent',
672 'crash_filtering', 672 'crash_filtering',
673 'max_enqueued_crashes', 673 'max_enqueued_crashes',
674 'core2md_failure', 674 'core2md_failure',
675 'internal_directory_failure', 675 'internal_directory_failure',
676 'crash_logs_creation', 676 'crash_logs_creation',
677 'crash_log_infinite_recursion', 677 'crash_log_infinite_recursion',
678 'core_file_persists_in_debug', 678 'core_file_persists_in_debug',
679 'core_file_removed_in_production'], 679 'core_file_removed_in_production'],
680 initialize_crash_reporter = True) 680 initialize_crash_reporter = True)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698