diff options
author | Ruben Rodriguez Buchillon <coconutruben@chromium.org> | 2018-07-24 13:45:32 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-07-31 16:37:03 -0700 |
commit | 46ba3c02d7e1749818aab5794d4051f111499ab6 (patch) | |
tree | e1ca37517c967c51adcc369534e91788a3ea9937 /extra/usb_power/stats_manager_unittest.py | |
parent | d6f54b711f64b42f9c4e53f5071a65bef9750f89 (diff) | |
download | chrome-ec-46ba3c02d7e1749818aab5794d4051f111499ab6.tar.gz |
stats_manager: remove expliting printing from stats_manager
StatsManager is a library shared across some modules. It shouldn't print
anything itself, but rather use logging & return a formatted summary
that can be printed.
Also take this opportunity to replace majority of explicit print calls
in powerlog with logger module.
BRANCH=None
BUG=chromium:760267
TEST=manual testing, output as expected still for powerlog
Change-Id: I0ed2ccaa45bbd1261119a1446110beadb075c1a2
Signed-off-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1140026
Reviewed-by: Mengqi Guo <mqg@chromium.org>
Diffstat (limited to 'extra/usb_power/stats_manager_unittest.py')
-rw-r--r-- | extra/usb_power/stats_manager_unittest.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/extra/usb_power/stats_manager_unittest.py b/extra/usb_power/stats_manager_unittest.py index 362db3fa77..ed317eca02 100644 --- a/extra/usb_power/stats_manager_unittest.py +++ b/extra/usb_power/stats_manager_unittest.py @@ -76,7 +76,7 @@ class TestStatsManager(unittest.TestCase): self.data.AddValue('test', 250) self.data.SetUnit('test', 'mw') self.data.CalculateStats() - summary_str = self.data._SummaryToString() + summary_str = self.data.SummaryToString() self.assertIn('test_mw', summary_str) def test_DoubleUnitSuffix(self): @@ -84,7 +84,7 @@ class TestStatsManager(unittest.TestCase): self.data.AddValue('test_mw', 250) self.data.SetUnit('test_mw', 'mw') self.data.CalculateStats() - summary_str = self.data._SummaryToString() + summary_str = self.data.SummaryToString() self.assertIn('test_mw', summary_str) self.assertNotIn('test_mw_mw', summary_str) |