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

Unified Diff: pkg/dev_compiler/test/not_yet_strong_tests.dart

Issue 2861783002: fix more mixin tests to work in strong mode (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/dev_compiler/test/browser/language_tests.js ('k') | pkg/dev_compiler/tool/run.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/dev_compiler/test/not_yet_strong_tests.dart
diff --git a/pkg/dev_compiler/test/not_yet_strong_tests.dart b/pkg/dev_compiler/test/not_yet_strong_tests.dart
index 7f851b37e978c3b27684d8df9df2853c48955e02..34795adcfd4a92e048147c5154f7caddde42a647 100644
--- a/pkg/dev_compiler/test/not_yet_strong_tests.dart
+++ b/pkg/dev_compiler/test/not_yet_strong_tests.dart
@@ -1085,9 +1085,6 @@ final notYetStrongTests = new HashSet<String>.from([
'language/mixin_black_listed_test_08_multi',
'language/mixin_cyclic_test_01_multi',
'language/mixin_cyclic_test_02_multi',
- 'language/mixin_extends_field_test',
- 'language/mixin_extends_method_test',
- 'language/mixin_field_test',
'language/mixin_forwarding_constructor4_test_01_multi',
'language/mixin_forwarding_constructor4_test_02_multi',
'language/mixin_forwarding_constructor4_test_03_multi',
@@ -1204,32 +1201,18 @@ final notYetStrongTests = new HashSet<String>.from([
'language/mixin_invalid_inheritance2_test_01_multi',
'language/mixin_invalid_inheritance2_test_02_multi',
'language/mixin_invalid_inheritance2_test_03_multi',
- 'language/mixin_lib_extends_field_test',
- 'language/mixin_lib_extends_method_test',
- 'language/mixin_method_test',
'language/mixin_super_bound2_test_01_multi',
'language/mixin_super_bound2_test_none_multi',
- 'language/mixin_super_bound_test',
- 'language/mixin_super_constructor2_test',
- 'language/mixin_super_constructor_default_test',
+ 'language/mixin_super_bound_test', // tests static errors
'language/mixin_super_constructor_named_test_01_multi',
- 'language/mixin_super_constructor_named_test_none_multi',
'language/mixin_super_constructor_positionals_test_01_multi',
- 'language/mixin_super_constructor_positionals_test_none_multi',
- 'language/mixin_super_constructor_test',
'language/mixin_super_test',
- 'language/mixin_type_parameter4_test',
'language/mixin_type_parameters_errors_test_01_multi',
'language/mixin_type_parameters_errors_test_02_multi',
'language/mixin_type_parameters_errors_test_03_multi',
'language/mixin_type_parameters_errors_test_04_multi',
'language/mixin_type_parameters_errors_test_05_multi',
- 'language/mixin_type_parameters_mixin_extends_test',
- 'language/mixin_type_parameters_mixin_test',
- 'language/mixin_type_parameters_simple_test',
- 'language/mixin_type_parameters_super_extends_test',
- 'language/mixin_type_parameters_super_test',
- 'language/mixin_with_two_implicit_constructors_test',
+ 'language/mixin_with_two_implicit_constructors_test', // tests static errors
'language/multiline_newline_test_01_multi',
'language/multiline_newline_test_02_multi',
'language/multiline_newline_test_03_multi',
« no previous file with comments | « pkg/dev_compiler/test/browser/language_tests.js ('k') | pkg/dev_compiler/tool/run.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698