commit 38372f4a5b40482d04ffd8b613c5666c0f136eee
author Ralph Nathan <ralphnathan@chromium.org> Tue May 19 15:56:25 2015 -0700
committer ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Thu May 28 02:09:33 2015 +0000
tree 460a458f86a5e983b8c7dfdbaffb9a19ed44b489
parent f2904ae8cee464b319fe32d1504634a60ac95392

logging: Change default log level of cros commands.

Change the default log level of cros commands to notice from
info.

Change operation.ProgressBar() to not display a progress bar if
not in a terminal.

BUG=none
TEST=unittests

Change-Id: I6762525b5688904b7d80d18bc9fa293ac77efcbb
Reviewed-on: https://chromium-review.googlesource.com/272268
Reviewed-by: Yiming Chen <yimingc@chromium.org>
Commit-Queue: Ralph Nathan <ralphnathan@chromium.org>
Trybot-Ready: Ralph Nathan <ralphnathan@chromium.org>
Tested-by: Ralph Nathan <ralphnathan@chromium.org>

scripts/cros_unittest.py

diff --git a/scripts/cros_unittest.py b/scripts/cros_unittest.py
index ffe9bad..4a46b28 100644
--- a/scripts/cros_unittest.py
+++ b/scripts/cros_unittest.py
@@ -50,8 +50,8 @@
     arg_parser.assert_called_with(caching=True, default_log_level='notice')
 
   def testDefaultLogLevelCros(self):
-    """Test that the default log level is not set to notice for cros tools."""
+    """Test that the default log level is set to notice for cros tools."""
     arg_parser = self.PatchObject(commandline, 'ArgumentParser',
                                   return_value=commandline.ArgumentParser())
     cros.GetOptions({})
-    arg_parser.assert_called_with(caching=True)
+    arg_parser.assert_called_with(caching=True, default_log_level='notice')