mastermon: report step metrics

infra/services/mastermon/test/pollers_test.py

 # Copyright (c) 2015 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.
 
 import json
 import os
 import tempfile
 import unittest
 
 import mock
@@ skipping 171 matching lines @@
     self.assertEqual(set(result1), set(pollers.FilePoller.field_keys))
     self.assertEqual(set(result2), set(pollers.FilePoller.field_keys))
 
     data1 = result1.copy()
     data2 = result2.copy()
     data1['random'] = 'value'  # Extra field, should be ignored.
     del data2['project_id']    # Missing field, should become 'unknown'.
     data2['duration_s'] = 5
     data2['pending_s'] = 1
     data2['total_s'] = data2['pending_s'] + data2['duration_s']
+    data2['steps'] = [
+      {'step_name': 'compile', 'duration_s': 2, 'result': 'r1'},
+    ]
     with temporary_directory(prefix='poller-test-') as tempdir:
       filename = self.create_data_file(tempdir, [data1, data2])
       p = pollers.FilePoller(filename, {})
       self.assertTrue(p.poll())
       fake_increment.assert_any_call(result1)
       fake_increment.assert_any_call(result2)
       fake_add.assert_any_call(data2['duration_s'], result2)
       fake_add.assert_any_call(data2['pending_s'], result2)
       fake_add.assert_any_call(data2['total_s'], result2)
       self.assertFalse(os.path.isfile(filename))
       # Make sure the rotated file is still there - for debugging.
       self.assertTrue(os.path.isfile(pollers.rotated_filename(filename)))
 
   def test_file_has_bad_data(self):
     """Mostly a smoke test: don't crash on bad data."""
     with temporary_directory(prefix='poller-test-') as tempdir:
       filename = self.create_data_file(tempdir, [])
       with open(filename, 'a') as f:
         f.write('}')
       p = pollers.FilePoller(filename, {})
       self.assertTrue(p.poll())
       self.assertFalse(os.path.isfile(filename))
       # Make sure the rotated file is still there - for debugging.
       self.assertTrue(os.path.isfile(pollers.rotated_filename(filename)))
 
   def test_safe_remove_error(self):
     """Smoke test: the function should not raise an exception."""
     pollers.safe_remove('nonexistent-file')