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

Issue 624998 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Infra tree is closed because of bad findit tests.

Project Member Reported by martiniss@chromium.org, Jul 1 2016

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Jul 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/7fdfa21aeb6422b315d4a6e3cff60ebf822f1682

commit 7fdfa21aeb6422b315d4a6e3cff60ebf822f1682
Author: Stephen Martinis <martiniss@chromium.org>
Date: Fri Jul 01 00:22:35 2016

Disable failing findit tests.

BUG= 624998 
TBR=katesonia

Change-Id: I46e6f4ff9074badc1c066990aab32f78cd6df454
Reviewed-on: https://chromium-review.googlesource.com/357950
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>

[modify] https://crrev.com/7fdfa21aeb6422b315d4a6e3cff60ebf822f1682/appengine/findit/handlers/crash/test/fracas_dashboard_test.py

Owner: kateso...@chromium.org
Status: Assigned (was: Started)
Please re-enable the tests when they aren't broken :)

Comment 3 by st...@chromium.org, Jul 1 2016

martiniss@: many thanks for disabling the test.

Sharu, the tests seem like flaky, please fix them.
It looks like it has something to do with json imports giving you unicode, which your tests might compare as inequal to your regular strings. But I'm not totally sure.
martiniss@: many thanks!

sorry for the delay, I somehow missed the email(probably because it's right before the holidays ><), upload a fix here https://codereview.chromium.org/2147773002
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 13 2016

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

commit b6b2c25f84c821e3c20feb509c9d366ea30905f2
Author: katesonia <katesonia@chromium.org>
Date: Wed Jul 13 00:11:07 2016

[Findit] Fix flaky dashboard test.

The tests are flaky because of the request_time for testing fracas analyses are not fixed, utcnow() is used for testing, so if the tests are running right pass the midnight of today, the assertion would fail. Fix this by hard-coding the request time of testcases and setting start_date and end_date in the request.

BUG= 624998 ,  626511 

Review-Url: https://codereview.chromium.org/2147773002

[modify] https://crrev.com/b6b2c25f84c821e3c20feb509c9d366ea30905f2/appengine/findit/handlers/crash/test/fracas_dashboard_test.py

Status: Fixed (was: Assigned)

Sign in to add a comment