Smoke test for tracing infrastructure in PyAuto

chrome/test/functional/tracing/tracing_smoke_test.py

Index: chrome/test/functional/tracing/tracing_smoke_test.py
diff --git a/chrome/test/functional/tracing/tracing_smoke_test.py b/chrome/test/functional/tracing/tracing_smoke_test.py
new file mode 100755
index 0000000000000000000000000000000000000000..42fd573d1f5dd756698e8d604af48851cf94ef4a
--- /dev/null
+++ b/chrome/test/functional/tracing/tracing_smoke_test.py
@@ -0,0 +1,56 @@
+#!/usr/bin/env python
+# Copyright (c) 2011 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 unittest

[Nirnimesh, 2012/07/31 20:05:57]

unused?

+
+import pyauto_tracing
+import pyauto
+import tracer
+
+
+class TracingSmokeTest(pyauto.PyUITest):
+  """Test basic functionality of the tracing API."""
+  def setUp(self):
+    super(TracingSmokeTest, self).setUp()
+    self._tracer_factory = tracer.TracerFactory(self)
+
+  def testGetData(self):
+    """Check that we can find a CrBrowserMain thread."""
+    tracer = self._tracer_factory.Produce()
+    tracer.BeginTracing()
+    model = tracer.EndTracing()
+    self.assertEqual(1, len(model.FindAllThreadsNamed('CrBrowserMain')))
+
+  def testMultipleTraces(self):
+    """Check that we can run multiple traces on the same tracer."""
+    tracer = self._tracer_factory.Produce()
+    tracer.BeginTracing()
+    model1 = tracer.EndTracing()
+    tracer.BeginTracing()
+    model2 = tracer.EndTracing()
+    del tracer

[Nirnimesh, 2012/07/31 20:05:57]

Why do you have to explicitly destroy it?

[Russ Harmon, 2012/07/31 20:54:18]

In order to implicitly test that a TraceModel will stay valid after the Tracer
which created it is deleted.

[Russ Harmon, 2012/07/31 22:24:55]

I just added an explicit test for this validity condition and removed the del
statements from the other tests.

+    self.assertEqual(1, len(model1.FindAllThreadsNamed('CrBrowserMain')))
+    self.assertEqual(1, len(model2.FindAllThreadsNamed('CrBrowserMain')))
+
+  def testMultipleTracers(self):
+    """Check that we can run multiple traces with multiple tracers."""
+    tracer1 = self._tracer_factory.Produce()
+    tracer2 = self._tracer_factory.Produce()
+    del self._tracer_factory
+    # I don't know what will happen if you try to nest calls to beginTracing.

[Nirnimesh, 2012/07/31 20:05:57]

Do not use first person language in code. ie rephrase to remove "I" (since it's
not clear who it refers to)

+    tracer1.BeginTracing()
+    model1 = tracer1.EndTracing()
+    del tracer1
+    tracer2.BeginTracing()
+    model2 = tracer2.EndTracing()
+    del tracer2
+    self.assertEqual(1, len(model1.FindAllThreadsNamed('CrBrowserMain')))
+    del model1
+    self.assertEqual(1, len(model2.FindAllThreadsNamed('CrBrowserMain')))
+    del model2
+
+
+if __name__ == '__main__':
+  pyauto_tracing.Main()