cq_stats: only take into account attempts within current date range

cq_stats: only take into account attempts within current date range

This avoids historical issues and outages affecting current stats,
and helps make current regressions and issues more visible, since
they're not obscured by the old ones.

BUG=none

Committed: https://chromium.googlesource.com/infra/infra/+/b7864190186f91567cadbab892ba914a063ca716

[Paweł Hajdan Jr., 2015-05-26 09:35:44]

phajdan.jr@chromium.org changed reviewers:
+ akuegel@chromium.org

[Adrian Kuegel, 2015-05-26 14:40:47]

Sorry for not replying sooner. LGTM with nit.

https://codereview.chromium.org/1158723003/diff/1/infra/tools/cq_stats/test/c...
File infra/tools/cq_stats/test/cq_stats_test.py (right):

https://codereview.chromium.org/1158723003/diff/1/infra/tools/cq_stats/test/c...
infra/tools/cq_stats/test/cq_stats_test.py:372: def
mock_derive_patch_stats(_begin_date, _end_date, patch_id):
nit: I think this should be just begin_date and end_date without leading '_'.

[Paweł Hajdan Jr., 2015-05-27 09:19:56]

https://codereview.chromium.org/1158723003/diff/1/infra/tools/cq_stats/test/c...
File infra/tools/cq_stats/test/cq_stats_test.py (right):

https://codereview.chromium.org/1158723003/diff/1/infra/tools/cq_stats/test/c...
infra/tools/cq_stats/test/cq_stats_test.py:372: def
mock_derive_patch_stats(_begin_date, _end_date, patch_id):
On 2015/05/26 14:40:47, Adrian Kuegel wrote:
> nit: I think this should be just begin_date and end_date without leading '_'.

That triggers pylint warnings. I'm leaving the code as-is.

** Presubmit ERRORS **
Pylint (187 files under infra) (26.35s) failed
************* Module infra.tools.cq_stats.test.cq_stats_test
W:372,32: Unused argument 'begin_date' (unused-argument)
W:372,44: Unused argument 'end_date' (unused-argument)

[Paweł Hajdan Jr., 2015-05-27 09:19:59]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2015-05-27 09:20:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1158723003/1

[commit-bot: I haz the power, 2015-05-27 09:24:05]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/infra/infra/+/b7864190186f91567cadbab892ba9...