commit f8d6798e5e1c9b4156e9d38ffee549b74a5077f3
author Allen Li <ayatane@chromium.org> Mon Oct 09 18:45:59 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Wed Oct 11 18:32:44 2017 -0700
tree a08aa3314064bf9a3997b8c160ae774eefe755a8
parent 61f718776b4ab10dfb326e208654b928cdbf2ab3

sysmon: Make a copy of dict before modifying it.

I was wondering why the age metric does not exist yet.  Now I know.
And knowing is half the battle.

BUG=None
TEST=None

Change-Id: I9d75ad204c970bc5d314acf724d89f55fc1a2607
Reviewed-on: https://chromium-review.googlesource.com/707678
Commit-Ready: Allen Li <ayatane@chromium.org>
Tested-by: Allen Li <ayatane@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

scripts/sysmon/puppet_metrics_unittest.py

diff --git a/scripts/sysmon/puppet_metrics_unittest.py b/scripts/sysmon/puppet_metrics_unittest.py
index 8556527..648f3c7 100644
--- a/scripts/sysmon/puppet_metrics_unittest.py
+++ b/scripts/sysmon/puppet_metrics_unittest.py
@@ -128,8 +128,9 @@
   def test_collect(self):
     with open(self.tempfile, 'w') as f:
       f.write(_SUMMARY)
-    with mock.patch.object(puppet_metrics, 'LAST_RUN_FILE', self.tempfile):
-      puppet_metrics.collect_puppet_summary()
+    with mock.patch('time.time', return_value=1500000000):
+      with mock.patch.object(puppet_metrics, 'LAST_RUN_FILE', self.tempfile):
+        puppet_metrics.collect_puppet_summary()
 
     setter = self.store.set
     calls = [
@@ -187,6 +188,8 @@
                   0.001235189, enforce_ge=mock.ANY),
         mock.call('puppet/times', ('filebucket',), None,
                   0.000341392, enforce_ge=mock.ANY),
+        mock.call('puppet/age', (), None,
+                  20329.0, enforce_ge=mock.ANY),
     ]
     setter.assert_has_calls(calls)
     self.assertEqual(len(setter.mock_calls), len(calls))