test: Clean up site_login.py a bit.

client/site_tests/logging_LogVolume/logging_LogVolume.py

 # Copyright (c) 2010 The Chromium OS 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 logging, os, re, subprocess, utils
 from autotest_lib.client.bin import site_login, site_ui_test, test
 from autotest_lib.client.common_lib import error
 
 
 class logging_LogVolume(site_ui_test.UITest):
     version = 1
 
 
     def log_stateful_used(self):
         output = utils.system_output('df /mnt/stateful_partition/')
         matches = re.search('(\d+)%', output)
         if matches is None:
             error.TestError('df failed')
         self._perf['percent_stateful_used'] = int(matches.group(1))
 
 
     def run_once(self):
-        if not site_login.wait_for_cryptohome():
-            raise error.TestFail('Could not wait for crytohome')
+        site_login.wait_for_cryptohome()
 
         self._perf = {}
         self.log_stateful_used()
         whitelist = open(os.path.join(self.bindir,
                                       'stateful_whitelist.txt'))
         patterns = {}
         for pattern in whitelist.readlines():
             pattern = pattern.strip()
             if pattern == '' or pattern[0] == '#':
                 continue
@@ skipping 35 matching lines @@
         self._perf['percent_unused_patterns'] = \
             int(100 * unmatched_patterns / len(patterns))
 
         self._perf['files_in_stateful_partition'] = stateful_files
 
         self.write_perf_keyval(self._perf)
 
         if unexpected_files > 0:
             raise error.TestError('There were %d unexpected files' %
                                   unexpected_files)