Defining the test expectations object

tools/telemetry/telemetry/test.py

Index: tools/telemetry/telemetry/test.py
diff --git a/tools/telemetry/telemetry/test.py b/tools/telemetry/telemetry/test.py
index f48a8823193c9c1cb83a2e4949fad32af8443575..9883c2fec111a4e7f78bfda10607f4e2cf3c9e28 100644
--- a/tools/telemetry/telemetry/test.py
+++ b/tools/telemetry/telemetry/test.py
@@ -25,6 +25,9 @@ class Test(object):
 
     test = self.test()
     ps = self.CreatePageSet(options)
+    expect = self.SetExpectations(ps)

[dtu, 2013/07/16 09:50:49]

Better to pass in ps.expectations instead of returning it, to make the
definition side cleaner.

def SetExpectations(expect):
  expect.Fail(123456, ['win'], 'file:///*.html')

instead of

def SetExpectations(ps):
  ps.expectations.Fail(123456, ['win'], 'file:///*.html')
  return ps.expectations

+    if expect:
+      ps.expectations = expect
     results = page_runner.Run(test, ps, options)
     results.PrintSummary()
     return len(results.failures) + len(results.errors)