Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(136)

Issue 109893004: Attempt to deflake our status files (Closed)

Created:
7 years ago by kustermann
Modified:
7 years ago
Reviewers:
Bill Hesse, floitsch
CC:
reviews_dartlang.org, Søren Gjesse, kasperl
Visibility:
Public.

Description

Attempt to deflake our status files R=whesse@google.com Committed: https://code.google.com/p/dart/source/detail?r=31022

Patch Set 1 #

Patch Set 2 : #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -139 lines) Patch
M dart/pkg/pkg.status View 1 7 chunks +21 lines, -42 lines 2 comments Download
M dart/tests/co19/co19-analyzer.status View 2 chunks +1 line, -2 lines 0 comments Download
M dart/tests/co19/co19-co19.status View 5 chunks +3 lines, -10 lines 0 comments Download
M dart/tests/co19/co19-dart2js.status View 7 chunks +6 lines, -14 lines 4 comments Download
M dart/tests/co19/co19-runtime.status View 6 chunks +1 line, -11 lines 0 comments Download
M dart/tests/corelib/corelib.status View 2 chunks +2 lines, -14 lines 0 comments Download
M dart/tests/html/html.status View 1 3 chunks +4 lines, -9 lines 0 comments Download
M dart/tests/isolate/isolate.status View 1 1 chunk +0 lines, -3 lines 0 comments Download
M dart/tests/language/language.status View 1 chunk +0 lines, -2 lines 0 comments Download
M dart/tests/language/language_dart2js.status View 1 chunk +1 line, -1 line 0 comments Download
M dart/tests/lib/analyzer/analyze_library.status View 2 chunks +0 lines, -12 lines 0 comments Download
M dart/tests/lib/lib.status View 2 chunks +0 lines, -2 lines 0 comments Download
M dart/tests/standalone/standalone.status View 1 3 chunks +1 line, -17 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
kustermann
This is based on a (semi)automated tool which goes through all test results of all ...
7 years ago (2013-12-10 14:33:53 UTC) #1
kustermann
7 years ago (2013-12-10 14:51:03 UTC) #2
Bill Hesse
This LGTM. Maybe we could skip changing the polymer files in this run, since I ...
7 years ago (2013-12-10 14:59:15 UTC) #3
kustermann
Committed patchset #2 manually as r31022 (presubmit successful).
7 years ago (2013-12-10 15:37:51 UTC) #4
kustermann
7 years ago (2013-12-10 15:58:20 UTC) #5
Message was sent while issue was closed.
https://codereview.chromium.org/109893004/diff/20001/dart/pkg/pkg.status
File dart/pkg/pkg.status (left):

https://codereview.chromium.org/109893004/diff/20001/dart/pkg/pkg.status#oldc...
dart/pkg/pkg.status:274: barback/test/too_many_open_files_test: Pass, Slow, Fail
# 14220
On 2013/12/10 14:59:16, Bill Hesse wrote:
> How does the tool know whether to remove "Slow"?  I would leave it in.

That was my fault (the tool writes out something else, and I incorporate the
changes into the status file).

https://codereview.chromium.org/109893004/diff/20001/dart/tests/co19/co19-dar...
File dart/tests/co19/co19-dart2js.status (left):

https://codereview.chromium.org/109893004/diff/20001/dart/tests/co19/co19-dar...
dart/tests/co19/co19-dart2js.status:256:
LibTest/async/Future/catchError_A03_t05: Pass, RuntimeError # Issue 14712
On 2013/12/10 14:59:16, Bill Hesse wrote:
> Will we edit all the bugs whose entries in the status file we change?  Waiting
> first to see if the status change is ok?

We could do that, but I think we will do that anyway at some points for *ALL*
bugs mentioned in status files (there are tons of bugs that have been closed,
but status files still refer to them ...).

https://codereview.chromium.org/109893004/diff/20001/dart/tests/co19/co19-dar...
File dart/tests/co19/co19-dart2js.status (right):

https://codereview.chromium.org/109893004/diff/20001/dart/tests/co19/co19-dar...
dart/tests/co19/co19-dart2js.status:69: LibTest/isolate/SendPort/send_A02_t05:
RuntimeError # TODO(ahe): Please triage this failure.
On 2013/12/10 14:59:16, Bill Hesse wrote:
> We should handle the TODO at this point.

I don't think we should. If we have a test and dart2js fails that test, it's up
to the dart2js team to triage the issue (and possibly fix it).

Powered by Google App Engine
This is Rietveld 408576698