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

Issue 2990213002: fix DDC buildbot so it is green (Closed)

Created:
3 years, 4 months ago by Jennifer Messerly
Modified:
3 years, 4 months ago
Reviewers:
vsm
CC:
dev-compiler+reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix DDC buildbot so it is green This is an attempt to get DDC's bot (https://travis-ci.org/dart-lang/sdk) green and restore test coverage, without affecting the Analyzer --strong buildbot. DDC and Analyzer can generally use the same status entries, as they are running nearly identical strong mode analysis. This fixes DDC's test builder to understand status codes in the same way analyzer does, as that was causing mismatch in statuses. This does not fix the problem that we're skipping tests instead of recording the status. So there is a bit of test coverage lost, but it restores most of DDC's coverage. We can tackle the skips either in a follow up, or as we migrate tests from *_strong to *_2. This is a partial revert of "Revert status file changes from 58a6227198d2aa927df19f1b4862f8d0017d6da4" in commit 7be946cf3d387fd74eb837211bc22c88288a6d20. Committed: https://github.com/dart-lang/sdk/commit/676cf46c447577b71cae8c61f5f050837eedd038

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -27 lines) Patch
M pkg/dev_compiler/test/codegen_test.dart View 3 chunks +10 lines, -3 lines 0 comments Download
M tests/language_2/language_2.status View 1 chunk +1 line, -1 line 0 comments Download
M tests/language_strong/language_strong.status View 2 chunks +1 line, -9 lines 0 comments Download
M tests/lib_strong/lib_strong.status View 3 chunks +9 lines, -14 lines 0 comments Download

Messages

Total messages: 7 (5 generated)
Jennifer Messerly
I am going to TBR this as it should only affect DDC, and only serve ...
3 years, 4 months ago (2017-08-03 02:57:41 UTC) #5
Jennifer Messerly
3 years, 4 months ago (2017-08-03 02:58:04 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
676cf46c447577b71cae8c61f5f050837eedd038 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698