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

Issue 602361 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
not available
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Changing all FlakyTest from android.test.FlakyTest to org.chromium.base.test.util.FlakyTest

Project Member Reported by yolandyan@chromium.org, Apr 11 2016

Issue description

It would be great if we can embed metadata about the test into the FlakyTest and DisabledTest annotations.

For example, crbug link which are already added next to the these annotations as comments
@FlakyTest //  https://crbug.com/579363  => @FlakyTest(message = " https://crbug.com/579363 ")

The main usage for this metadata is that disabled test tracking script would be able to get the bug link associated with the annotation and send them to the tracking dashboard

CL for adding FlakyTest.java: https://codereview.chromium.org/1856203003/
CL for changing all annotations: https://codereview.chromium.org/1856273002/
CL for presubmit checker: https://codereview.chromium.org/1856253002/
 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/daabc6db764177bd31d9ade06be383a4aa3e6617

commit daabc6db764177bd31d9ade06be383a4aa3e6617
Author: yolandyan <yolandyan@google.com>
Date: Mon Apr 18 18:29:39 2016

Change PRESUBMIT.py to make sure the right FlakyTest being used

BUG= 602361 

Review URL: https://codereview.chromium.org/1856253002

Cr-Commit-Position: refs/heads/master@{#387960}

[modify] https://crrev.com/daabc6db764177bd31d9ade06be383a4aa3e6617/PRESUBMIT.py

Status: Fixed (was: Untriaged)

Sign in to add a comment