Reland of expect_tests > coverage: no data, no fail (patchset #1 id:1 of https://codereview.chromium

Reland of expect_tests > coverage: no data, no fail (patchset #1 id:1 of https://codereview.chromium.org/2036823002/ )

R=tandrii@chromium.org
BUG=616748
Committed: https://chromium.googlesource.com/infra/testing/expect_tests/+/fca8c5fcc668484ca919278928fbdc2d71f84a0a

[Sergiy Byelozyorov, 2016-06-02 18:49:29]

Description was changed from

==========
Fixes for my previous CL... wish we had tests for expect_tests

I broke expect_tests in http://crrev.com/2036823002. This is fix.

R=tandrii@chromium.org
BUG=616748
==========

to

==========
Fixes for my previous CL... wish I knew how to run tests for expect_tests

I broke expect_tests in http://crrev.com/2036823002. This is fix.

R=tandrii@chromium.org
BUG=616748
==========

[tandrii(chromium), 2016-06-02 18:53:00]

https://codereview.chromium.org/2031013002/diff/1/expect_tests/cover.py
File expect_tests/cover.py (right):

https://codereview.chromium.org/2031013002/diff/1/expect_tests/cover.py#newco...
expect_tests/cover.py:119: if ce.message == 'No data to report.':
how about something like this:
  coverage_percent = 100.0
  if verbose:
    print 'No data to report, granting 100%'

[Sergiy Byelozyorov, 2016-06-02 18:54:00]

Description was changed from

==========
Fixes for my previous CL... wish I knew how to run tests for expect_tests

I broke expect_tests in http://crrev.com/2036823002. This is fix.

R=tandrii@chromium.org
BUG=616748
==========

to

==========
expect_tests > coverage: no data, no fail

R=tandrii@chromium.org
BUG=616748
==========

[Sergiy Byelozyorov, 2016-06-02 18:54:23]

Description was changed from

==========
expect_tests > coverage: no data, no fail

R=tandrii@chromium.org
BUG=616748
==========

to

==========
expect_tests > coverage: no data, no fail

This is a reland of https://codereview.chromium.org/2036823002/.

R=tandrii@chromium.org
BUG=616748
==========

[Sergiy Byelozyorov, 2016-06-02 18:55:22]

Patchset #1 (id:1) has been deleted

[Sergiy Byelozyorov, 2016-06-02 18:58:34]

On 2016/06/02 18:53:00, tandrii(chromium) wrote:
> https://codereview.chromium.org/2031013002/diff/1/expect_tests/cover.py
> File expect_tests/cover.py (right):
> 
>
https://codereview.chromium.org/2031013002/diff/1/expect_tests/cover.py#newco...
> expect_tests/cover.py:119: if ce.message == 'No data to report.':
> how about something like this:
>   coverage_percent = 100.0
>   if verbose:
>     print 'No data to report, granting 100%'

Done.

[Sergiy Byelozyorov, 2016-06-02 19:00:48]

Description was changed from

==========
expect_tests > coverage: no data, no fail

This is a reland of https://codereview.chromium.org/2036823002/.

R=tandrii@chromium.org
BUG=616748
==========

to

==========
Reland of expect_tests > coverage: no data, no fail (patchset #1 id:1 of
https://codereview.chromium.org/2036823002/ )

R=tandrii@chromium.org
BUG=616748
==========

[tandrii(chromium), 2016-06-02 19:04:45]

lgtm

https://codereview.chromium.org/2031013002/diff/70001/expect_tests/cover.py
File expect_tests/cover.py (right):

https://codereview.chromium.org/2031013002/diff/70001/expect_tests/cover.py#n...
expect_tests/cover.py:124: print 'No data to report, setting coverage percentage
to 100%'
nit: remove percentage.

[Sergiy Byelozyorov, 2016-06-02 19:05:06]

https://codereview.chromium.org/2031013002/diff/70001/expect_tests/cover.py
File expect_tests/cover.py (right):

https://codereview.chromium.org/2031013002/diff/70001/expect_tests/cover.py#n...
expect_tests/cover.py:124: print 'No data to report, setting coverage percentage
to 100%'
On 2016/06/02 19:04:45, tandrii(chromium) wrote:
> nit: remove percentage.

Done.

[Sergiy Byelozyorov, 2016-06-02 19:05:09]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2016-06-02 19:05:20]

CQ is trying da patch.
Reference: 2031013002/90001

[commit-bot: I haz the power, 2016-06-02 19:05:26]

Description was changed from

==========
Reland of expect_tests > coverage: no data, no fail (patchset #1 id:1 of
https://codereview.chromium.org/2036823002/ )

R=tandrii@chromium.org
BUG=616748
==========

to

==========
Reland of expect_tests > coverage: no data, no fail (patchset #1 id:1 of
https://codereview.chromium.org/2036823002/ )

R=tandrii@chromium.org
BUG=616748

Committed:
https://chromium.googlesource.com/infra/testing/expect_tests/+/fca8c5fcc66848...
==========

[commit-bot: I haz the power, 2016-06-02 19:05:28]

Committed patchset #5 (id:90001) as
https://chromium.googlesource.com/infra/testing/expect_tests/+/fca8c5fcc66848...