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

Issue 2980213002: Re-land beginning Dart 2.0 test migration. (Closed)

Created:
3 years, 5 months ago by Bob Nystrom
Modified:
3 years, 5 months ago
Reviewers:
jcollins
CC:
reviews_dartlang.org, zra, aam
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Re-land beginning Dart 2.0 test migration. This is similar to: 47985d6dbd85c541eb0374d95addf677af51ae5f 452d2d08402d5521ae2d09cd29d2c8147ac1cab2 7af2d86ed3cacec955171311e3660a54bf26a574 Except that it also fixes the status for dart2analyzer. R=jcollins@google.com Committed: https://github.com/dart-lang/sdk/commit/489c96ac2ef264c64ed961223723b0a4779192d8

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+99 lines, -403 lines) Patch
D tests/corelib/apply_test.dart View 1 chunk +0 lines, -76 lines 0 comments Download
A + tests/corelib_2/apply_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A tests/corelib_2/corelib_2.status View 1 chunk +3 lines, -0 lines 0 comments Download
A tests/corelib_2/symbol_map_helper.dart View 1 chunk +15 lines, -0 lines 0 comments Download
D tests/corelib_strong/apply_test.dart View 1 chunk +0 lines, -76 lines 0 comments Download
D tests/language/aborting_switch_case_test.dart View 1 chunk +0 lines, -27 lines 0 comments Download
D tests/language/abstract_beats_arguments2_test.dart View 1 chunk +0 lines, -14 lines 0 comments Download
D tests/language/abstract_beats_arguments_test.dart View 1 chunk +0 lines, -28 lines 0 comments Download
D tests/language/abstract_exact_selector_test.dart View 1 chunk +0 lines, -38 lines 0 comments Download
D tests/language/abstract_getter_test.dart View 1 chunk +0 lines, -27 lines 0 comments Download
A + tests/language_2/aborting_switch_case_test.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A tests/language_2/abstract_beats_arguments_test.dart View 1 chunk +16 lines, -0 lines 0 comments Download
A + tests/language_2/abstract_exact_selector_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
A + tests/language_2/abstract_getter_test.dart View 1 chunk +1 line, -1 line 0 comments Download
A tests/language_2/compiler_annotations.dart View 1 chunk +12 lines, -0 lines 0 comments Download
A tests/language_2/language_2.status View 1 chunk +10 lines, -0 lines 1 comment Download
D tests/language_strong/aborting_switch_case_test.dart View 1 chunk +0 lines, -27 lines 0 comments Download
D tests/language_strong/abstract_exact_selector_test.dart View 1 chunk +0 lines, -38 lines 0 comments Download
D tests/language_strong/abstract_getter_test.dart View 1 chunk +0 lines, -27 lines 0 comments Download
D tests/lib/math/call_cmath_box_failure_path_test.dart View 1 chunk +0 lines, -23 lines 0 comments Download
A tests/lib_2/lib_2.status View 1 chunk +3 lines, -0 lines 0 comments Download
A + tests/lib_2/math/call_cmath_box_failure_path_test.dart View 1 chunk +3 lines, -0 lines 0 comments Download
M tools/bots/ddc_tests.py View 1 chunk +4 lines, -0 lines 0 comments Download
M tools/testing/dart/options.dart View 1 chunk +3 lines, -0 lines 0 comments Download
M tools/testing/dart/test_configurations.dart View 1 chunk +3 lines, -0 lines 0 comments Download
M tools/testing/dart/test_suite.dart View 1 chunk +26 lines, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
Bob Nystrom
This re-reverts the stuff aam@ had to revert on Friday, and also fixes https://github.com/dart-lang/sdk/issues/30171.
3 years, 5 months ago (2017-07-17 17:49:16 UTC) #2
jcollins
lgtm https://codereview.chromium.org/2980213002/diff/1/tests/language_2/language_2.status File tests/language_2/language_2.status (right): https://codereview.chromium.org/2980213002/diff/1/tests/language_2/language_2.status#newcode7 tests/language_2/language_2.status:7: [ $runtime == vm || $compiler == dart2js ...
3 years, 5 months ago (2017-07-17 18:04:03 UTC) #3
Bob Nystrom
On 2017/07/17 18:04:03, jcollins wrote: > lgtm > > https://codereview.chromium.org/2980213002/diff/1/tests/language_2/language_2.status > File tests/language_2/language_2.status (right): > ...
3 years, 5 months ago (2017-07-17 18:17:14 UTC) #4
Bob Nystrom
3 years, 5 months ago (2017-07-17 19:40:24 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
489c96ac2ef264c64ed961223723b0a4779192d8.

Powered by Google App Engine
This is Rietveld 408576698