Make test failure messages more verbose (and thus useful to testing operators).

client/site_tests/factory_ExtDisplay/factory_ExtDisplay.py

+# -*- coding: utf-8 -*-
+#
 # 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.
 
 
 # DESCRIPTION :
 #
 # This is a factory test to test the external display (hdmi/vga/other)
 # UI based heavily on factory_Display/factory_Audio
 
@@ skipping 50 matching lines @@
             gtk.main_quit()
             return
         self._current_subtest = self._subtest_queue.pop()
         name, cfg = self._current_subtest
         self._status_map[name] = ful.ACTIVE
 
     def start_subtest(self):
         subtest_name, subtest_cfg = self._current_subtest
         if 'cfg' in subtest_cfg:
             for cfg in subtest_cfg['cfg']:
-                utils.system(cfg)
+                try:
+                    utils.system(cfg)
+                except error.CmdError:
+                    raise error.TestNAError('Setup failed\n設定失敗\nCmd: %s' % cfg)
                 factory.log("cmd: " + cfg)
         if 'cmd' in subtest_cfg:
             cmd = "%s %s" % (subtest_cfg['cmd'], self._sample)
             factory.log("cmd: " + cmd)
             self._job = utils.BgJob(cmd, stderr_level=logging.DEBUG)
         else:
             self._job = None
 
     def finish_subtest(self):
         subtest_name, subtest_cfg = self._current_subtest
         if 'postcfg' in subtest_cfg:
             for cfg in subtest_cfg['postcfg']:
-                utils.system(cfg)
+                try:
+                    utils.system(cfg)
+                except error.CmdError:
+                    raise error.TestNAError('Setup failed\n設定失敗\nCmd: %s' % cfg)
                 factory.log("cmd: " + cfg)
         self.close_bgjob(subtest_cfg)
 
     def key_press_callback(self, widget, event):
         subtest_name, subtest_cfg = self._current_subtest
         if event.keyval == gtk.keysyms.space and not self._started:
             self._prompt_label.set_text(subtest_cfg['msg'])
             self._started = True
             self._test_widget.queue_draw()
         return True
@@ skipping 99 matching lines @@
         ful.run_test_widget(self.job, vbox,
             window_registration_callback=self.register_callbacks)
 
         failed_set = set(name for name, status in self._status_map.items()
                          if status is not ful.PASSED)
         if failed_set:
             raise error.TestFail('some subtests failed (%s)' %
                                  ', '.join(failed_set))
 
         factory.log('%s run_once finished' % self.__class__)