New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 730657 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 0
Type: ----



Sign in to add a comment

Add unittest to triage_flake_analysis.py

Project Member Reported by st...@chromium.org, Jun 7 2017

Issue description

There is some "# pragma: no cover" in that file, we should get rid of them and add unittest instead.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 9 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/0369c963be6f51c7db6af3c848839655018e52a3

commit 0369c963be6f51c7db6af3c848839655018e52a3
Author: Jeffrey Li <lijeffrey@chromium.org>
Date: Fri Jun 09 22:09:55 2017

[Findit] Flake Analyzer - Adding unit tests for triage flake results

Bug:  730657 
Change-Id: If8cd2284b335f6e260de3b6ed7eb32b6045780db
Reviewed-on: https://chromium-review.googlesource.com/528664
Commit-Queue: Jeffrey Li <lijeffrey@chromium.org>
Reviewed-by: Chan Li <chanli@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/0369c963be6f51c7db6af3c848839655018e52a3/appengine/findit/handlers/flake/triage_flake_analysis.py
[add] https://crrev.com/0369c963be6f51c7db6af3c848839655018e52a3/appengine/findit/handlers/flake/test/triage_flake_analysis_test.py

Status: Fixed (was: Started)

Sign in to add a comment