Rename process_log_utils and related things.

scripts/slave/unittests/performance_log_processor_test.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
-"""Unit tests for annotated log parsers (aka log processors) used by runtest.py.
-
-The classes tested here reside in process_log_utils.py.
-
-The script runtest.py has the option to parse test output locally and send
-results to the master via annotator steps. This file tests those parsers.
-"""
+"""Unit tests for annotated log processor classes used by runtest.py."""
 
 import json
 import os
 import unittest
 
 import test_env  # pylint: disable=W0403,W0611
 
-from slave import process_log_utils
+from slave import performance_log_processor
 
-# These should be the same as the constants used in process_log_utils.
+# These should be the same as the constants used in performance_log_processor.
 # See: http://docs.buildbot.net/current/developer/results.html
 SUCCESS, WARNINGS, FAILURE, SKIPPED, EXCEPTION, RETRY = range(6)
 
 # Custom percentile numbers to use in the tests below.
 TEST_PERCENTILES = [.05, .3, .8]
 
 
 class LogProcessorTest(unittest.TestCase):
   """Base class for log processor unit tests. Contains common operations."""
 
@@ skipping 101 matching lines @@
   """Test case for basic functionality of GraphingLogProcessor class."""
 
   def testSummary(self):
     """Tests the output of "summary" files, which contain per-graph data."""
     input_files = ['graphing_processor.log']
     output_files = ['%s-summary.dat' % graph for graph in ('commit_charge',
         'ws_final_total', 'vm_final_browser', 'vm_final_total',
         'ws_final_browser', 'processes', 'artificial_graph')]
 
     self._ConstructParseAndCheckJSON(input_files, output_files, None,
-        process_log_utils.GraphingLogProcessor)
+        performance_log_processor.GraphingLogProcessor)
 
   def testGraphList(self):
     """Tests the output of "graphs.dat" files, which contains a graph list."""
     input_files = ['graphing_processor.log']
     graphfile = 'graphs.dat'
     output_files = [graphfile]
 
     logs = self._ConstructParseAndCheckLogfiles(input_files, output_files,
-        process_log_utils.GraphingLogProcessor)
+        performance_log_processor.GraphingLogProcessor)
 
     actual = json.loads('\n'.join(logs[graphfile]))
     expected = json.load(open(
         os.path.join(test_env.DATA_PATH, 'graphing_processor-graphs.dat')))
 
     self.assertEqual(len(actual), len(expected))
 
     for graph in expected:
       self.assertTrue(graph['name'] in actual)
       for element in graph:
         self.assertEqual(actual[graph['name']][element], graph[element])
 
   def testHistogramGeometricMeanAndStandardDeviation(self):
     input_files = ['graphing_processor.log']
     summary_file = 'hist1-summary.dat'
     output_files = [summary_file]
 
     logs = self._ConstructParseAndCheckLogfiles(input_files, output_files,
-        process_log_utils.GraphingLogProcessor)
+        performance_log_processor.GraphingLogProcessor)
 
     actual = json.loads('\n'.join(logs[summary_file]))
     expected = json.load(open(
         os.path.join(test_env.DATA_PATH, summary_file)))
 
     self.assertEqual(actual, expected, 'Filename %s did not contain expected '
         'data.' % summary_file)
 
   def testHistogramPercentiles(self):
     input_files = ['graphing_processor.log']
     summary_files = ['hist1_%s-summary.dat' % str(p) for p in TEST_PERCENTILES]
     output_files = summary_files
 
     logs = self._ConstructParseAndCheckLogfiles(input_files, output_files,
-        process_log_utils.GraphingLogProcessor)
+        performance_log_processor.GraphingLogProcessor)
 
     for filename in output_files:
       actual = json.loads('\n'.join(logs[filename]))
       expected = json.load(open(os.path.join(test_env.DATA_PATH, filename)))
       self.assertEqual(actual, expected, 'Filename %s did not contain expected '
           'data.' % filename)
 
 
 class GraphingLogProcessorPerfTest(LogProcessorTest):
   """Another test case for the GraphingLogProcessor class.
 
   The tests in this test case compare results against the contents of a
   perf expectations file.
   """
 
   def _TestPerfExpectations(self, perf_expectations_file):
     perf_expectations_path = os.path.join(
         test_env.DATA_PATH, perf_expectations_file)
 
     input_file = 'graphing_processor.log'
     graph_file = 'graphs.dat'
 
     parser = self._ConstructDefaultProcessor(
-        process_log_utils.GraphingLogProcessor,
+        performance_log_processor.GraphingLogProcessor,
         factory_properties={'expectations': True, 'perf_id': 'tester'},
         perf_expectations_path=perf_expectations_path)
 
     self._ProcessLog(parser, input_file)
 
     actual = json.loads('\n'.join(parser.PerformanceLogs()[graph_file]))
     expected = json.load(open(
         os.path.join(test_env.DATA_PATH, 'graphing_processor-graphs.dat')))
 
     self.assertEqual(len(actual), len(expected))
@@ skipping 139 matching lines @@
     expected = ('PERF_IMPROVE: vm_final_browser/1t_vm_b (inf%)')
     self.assertEqual(expected, step.PerformanceSummary()[0])
     self.assertEqual(WARNINGS, step.evaluateCommand('mycommand'))
 
 
 class GraphingPageCyclerLogProcessorPerfTest(LogProcessorTest):
   """Unit tests for the GraphingPageCyclerLogProcessor class."""
 
   def testPageCycler(self):
     parser = self._ConstructDefaultProcessor(
-        process_log_utils.GraphingPageCyclerLogProcessor)
+        performance_log_processor.GraphingPageCyclerLogProcessor)
     self._ProcessLog(parser, 'page_cycler.log')
 
     expected = 't: 2.32k'
     self.assertEqual(expected, parser.PerformanceSummary()[0])
 
 
 if __name__ == '__main__':
   unittest.main()