Add unittest for gtest_test_results.

Add unittest for gtest_test_results.

BUG=383100
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=276573

[tonyg, 2014-06-11 23:08:51]

lgtm

Incredible, thank you :)

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
File tools/telemetry/telemetry/results/base_test_results_unittest.py (right):

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
tools/telemetry/telemetry/results/base_test_results_unittest.py:18: def
CreateException(self):
I know you're just moving this. But while you're here, can you convert it to an
IntentionalException. 

https://code.google.com/p/chromium/codesearch#chromium/src/tools/telemetry/te...

That exception doesn't print stacks that confuse sheriffs in unittests results.

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
File tools/telemetry/telemetry/results/gtest_test_results_unittest.py (right):

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
tools/telemetry/telemetry/results/gtest_test_results_unittest.py:20: 
nit: 2 blank lines between top level definitions

http://google-styleguide.googlecode.com/svn/trunk/pyguide.html#Blank_Lines

[nednguyen, 2014-06-11 23:30:36]

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
File tools/telemetry/telemetry/results/base_test_results_unittest.py (right):

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
tools/telemetry/telemetry/results/base_test_results_unittest.py:18: def
CreateException(self):
On 2014/06/11 23:08:51, tonyg wrote:
> I know you're just moving this. But while you're here, can you convert it to
an
> IntentionalException. 
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/tools/telemetry/te...
> 
> That exception doesn't print stacks that confuse sheriffs in unittests
results.

Done.

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
File tools/telemetry/telemetry/results/gtest_test_results_unittest.py (right):

https://codereview.chromium.org/332513002/diff/1/tools/telemetry/telemetry/re...
tools/telemetry/telemetry/results/gtest_test_results_unittest.py:20: 
On 2014/06/11 23:08:51, tonyg wrote:
> nit: 2 blank lines between top level definitions
> 
> http://google-styleguide.googlecode.com/svn/trunk/pyguide.html#Blank_Lines

Done.

[nednguyen, 2014-06-11 23:30:49]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2014-06-11 23:32:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nednguyen@google.com/332513002/20001

[commit-bot: I haz the power, 2014-06-12 01:01:13]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/15774)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/18906)

[commit-bot: I haz the power, 2014-06-12 01:07:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-12 01:07:17]

Try jobs failed on following builders:
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)

[nednguyen, 2014-06-12 03:34:44]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2014-06-12 03:36:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nednguyen@google.com/332513002/20001

[commit-bot: I haz the power, 2014-06-12 07:57:54]

Change committed as 276573