Add a unit test for run_test_cases.py.

Add a unit test for run_test_cases.py.

Make Progress to output to stderr, it automatically flushes and it doesn't
interfere with the output, making testing easier and stdout more usable in
general, like if piping the results.

NOTRY=true
R=cmp@chromium.org
BUG=
TEST=new integration test


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=141686

[cmp, 2012-06-12 16:32:10]

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/data/gte...
File tools/isolate/data/gtest_fake/gtest_fake.py (right):

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/data/gte...
tools/isolate/data/gtest_fake/gtest_fake.py:49: 
nit: insert an empty line

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
File tools/isolate/run_test_cases_test.py (right):

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
tools/isolate/run_test_cases_test.py:21: target =  os.path.join(ROOT_DIR,
'data', 'gtest_fake', 'gtest_fake.py')
nit: remove extra space after =

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
tools/isolate/run_test_cases_test.py:32: expected = (
before line 32, can you assert that proc.returncode is the expected value?

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
tools/isolate/run_test_cases_test.py:36: '\n'
why do you switch from using + above to no + on lines 36-39?  you could make
this consistent by using + everywhere.  can you make that change?

[M-A Ruel, 2012-06-12 17:33:48]

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/data/gte...
File tools/isolate/data/gtest_fake/gtest_fake.py (right):

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/data/gte...
tools/isolate/data/gtest_fake/gtest_fake.py:49: 
On 2012/06/12 16:32:10, cmp wrote:
> nit: insert an empty line

done

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
File tools/isolate/run_test_cases_test.py (right):

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
tools/isolate/run_test_cases_test.py:21: target =  os.path.join(ROOT_DIR,
'data', 'gtest_fake', 'gtest_fake.py')
On 2012/06/12 16:32:10, cmp wrote:
> nit: remove extra space after =

I coded that up quickly and it shown. fixed

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
tools/isolate/run_test_cases_test.py:32: expected = (
On 2012/06/12 16:32:10, cmp wrote:
> before line 32, can you assert that proc.returncode is the expected value?

Good idea, fixed.

https://chromiumcodereview.appspot.com/10533085/diff/1/tools/isolate/run_test...
tools/isolate/run_test_cases_test.py:36: '\n'
On 2012/06/12 16:32:10, cmp wrote:
> why do you switch from using + above to no + on lines 36-39?  you could make
> this consistent by using + everywhere.  can you make that change?

I was abusing string concatenation in python. I agree it wasn't readable so I
cleaned that up.

[cmp, 2012-06-12 17:39:54]

lgtm

[commit-bot: I haz the power, 2012-06-12 17:40:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/10533085/5001

[commit-bot: I haz the power, 2012-06-12 17:46:47]

Try job failure for 10533085-5001 (retry) on mac_rel for step "update".
It's a second try, previously, step "update" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

Step "update" is always a major failure.
Look at the try server FAQ for more details.

[M-A Ruel, 2012-06-12 18:05:02]

On 2012/06/12 17:46:47, I haz the power (commit-bot) wrote:
> Try job failure for 10533085-5001 (retry) on mac_rel for step "update".
> It's a second try, previously, step "update" failed.
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...
> 
> Step "update" is always a major failure.
> Look at the try server FAQ for more details.

The "empty file doesn't work on the try server" issue will be fixed by Roger.

[commit-bot: I haz the power, 2012-06-12 18:05:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/10533085/5001

[commit-bot: I haz the power, 2012-06-12 18:07:09]

Change committed as 141686