Skip to content

Commit

Permalink
Update old mixin tests. Remove the --super-mixins flag from the tests.
Browse files Browse the repository at this point in the history
Bug: http://dartbug.com/34781
Change-Id: Ib07e1927e7de35ff0c48514b9e89da65a5f192dd
Reviewed-on: https://dart-review.googlesource.com/c/79700
Commit-Queue: Lasse R.H. Nielsen <[email protected]>
Reviewed-by: Leaf Petersen <[email protected]>
  • Loading branch information
lrhn authored and [email protected] committed Oct 23, 2018
1 parent a394e4f commit 03765ed
Show file tree
Hide file tree
Showing 20 changed files with 99 additions and 409 deletions.
3 changes: 1 addition & 2 deletions tests/language_2/bug32372_test.dart
Original file line number Diff line number Diff line change
@@ -1,15 +1,14 @@
// Copyright (c) 2018, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
// SharedOptions=--supermixin

import "package:expect/expect.dart";

class A extends Object with B<String>, C {}

class B<T> {}

class C<T> extends B<T> {
mixin C<T> on B<T> {
get t => T;
}

Expand Down
3 changes: 1 addition & 2 deletions tests/language_2/issue32353_2_test.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// Copyright (c) 2018, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
// SharedOptions=--supermixin

// The following compile-time error is expected:
//
Expand All @@ -14,7 +13,7 @@ class A {}

class B<X, Y> {}

class C<X> extends B<X, A> {}
mixin C<X> on B<X, A> {}

class /*@compile-error=unspecified*/ D<X, Y> extends B<X, Y> with C {}

Expand Down
5 changes: 2 additions & 3 deletions tests/language_2/issue32353_test.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// Copyright (c) 2018, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
// SharedOptions=--supermixin

import "package:expect/expect.dart";

Expand All @@ -23,8 +22,8 @@ abstract class FileSystemEntity implements io_FileSystemEntity {}
abstract class ForwardingFileSystemEntity<T extends FileSystemEntity,
D extends io_FileSystemEntity> implements FileSystemEntity {}

abstract class ForwardingDirectory<T extends Directory>
extends ForwardingFileSystemEntity<T, io_Directory> implements Directory {
mixin ForwardingDirectory<T extends Directory>
on ForwardingFileSystemEntity<T, io_Directory> implements Directory {
get t => T;
}

Expand Down
8 changes: 3 additions & 5 deletions tests/language_2/issue34404_flutter_test.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// Copyright (c) 2018, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
// SharedOptions=--supermixin

// This test case is a reduction of some Flutter code. We wish to verify that
// the class _DismissibleState doesn't have any type inference errors.
Expand All @@ -21,13 +20,12 @@ abstract class Diagnosticable {}

class Dismissible extends StatefulWidget {}

abstract class TickerProviderStateMixin<T extends StatefulWidget>
extends State<T> implements TickerProvider {}
mixin TickerProviderStateMixin<T extends StatefulWidget> on State<T>
implements TickerProvider {}

abstract class TickerProvider {}

abstract class AutomaticKeepAliveClientMixin<T extends StatefulWidget>
extends State<T> {}
mixin AutomaticKeepAliveClientMixin<T extends StatefulWidget> on State<T> {}

main() {
new _DismissibleState();
Expand Down
31 changes: 2 additions & 29 deletions tests/language_2/language_2_analyzer.status
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ accessor_conflict_import_prefixed2_test: CompileTimeError # Issue 25626
accessor_conflict_import_prefixed_test: CompileTimeError # Issue 25626
accessor_conflict_import_test: CompileTimeError # Issue 25626
additional_interface_adds_optional_args_test: CompileTimeError # Issue #30568
bug32372_test: Skip
cascaded_forwarding_stubs_test: CompileTimeError # Issue 34329
config_import_corelib_test: CompileTimeError, StaticWarning, OK # failing-by-design: Will never pass, see Issue #34332
conflicting_generic_interfaces_hierarchy_loop_infinite_test: Skip # Issue #34333 (loops forever)
Expand Down Expand Up @@ -70,40 +69,14 @@ issue31596_super_test/05: CompileTimeError # Issue #31596
issue31596_super_test/none: CompileTimeError # Issue #31596
issue31596_tearoff_test: CompileTimeError # Issue #31596
issue31596_test: CompileTimeError # Issue #31596
issue32353_2_test: Skip
issue32353_test: Skip
issue34404_flutter_test: Skip
issue34498_test: MissingCompileTimeError # Issue 34500
large_class_declaration_test: Slow, Pass
malformed2_test: Pass, MissingCompileTimeError # Flaky: issue 31056.
mixin_declaration/mixin_declaration_factory_test/02: Crash # Issue 34809
mixin_declaration/mixin_declaration_invalid_superinvocation_test/10: CompileTimeError # Issue 30552
mixin_method_override_test/01: MissingCompileTimeError
mixin_of_mixin_test: Skip
mixin_super_2_test: Skip
mixin_super_2_test/01: MissingCompileTimeError
mixin_super_2_test/03: MissingCompileTimeError
mixin_super_test: Skip
mixin_super_use_test: Skip
mixin_superclass_test: Skip
mixin_supertype_subclass2_test: Skip
mixin_supertype_subclass2_test/02: MissingStaticWarning # Issue 25614
mixin_supertype_subclass2_test/05: MissingStaticWarning # Issue 25614
mixin_supertype_subclass3_test: Skip
mixin_supertype_subclass3_test/02: MissingStaticWarning # Issue 25614
mixin_supertype_subclass3_test/05: MissingStaticWarning # Issue 25614
mixin_supertype_subclass4_test: Skip
mixin_supertype_subclass4_test/01: MissingStaticWarning # Issue 25614
mixin_supertype_subclass4_test/02: MissingStaticWarning # Issue 25614
mixin_supertype_subclass4_test/03: MissingStaticWarning # Issue 25614
mixin_supertype_subclass4_test/04: MissingStaticWarning # Issue 25614
mixin_supertype_subclass4_test/05: MissingStaticWarning # Issue 25614
mixin_supertype_subclass_test: Skip
mixin_supertype_subclass_test/02: MissingCompileTimeError, MissingStaticWarning # failing-by-design, see Issue #34376
mixin_supertype_subclass_test/05: MissingCompileTimeError, MissingStaticWarning # failing-by-design, see Issue #34376
mixin_type_parameter_inference_error_test: Skip
mixin_type_parameter_inference_previous_mixin_test: Skip
mixin_type_parameter_inference_test: Skip
mixin_super_2_test: CompileTimeError
mixin_super_use_test: CompileTimeError
mock_writable_final_private_field_test: CompileTimeError # failing-by-design, see Issue #34377
multiple_interface_inheritance_test: CompileTimeError # Issue 30552
nested_generic_closure_test: CompileTimeError # Issue #28515
Expand Down
68 changes: 0 additions & 68 deletions tests/language_2/language_2_dart2js.status
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ bit_operations_test: RuntimeError, OK # non JS number semantics
bit_operations_test/03: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
bit_operations_test/04: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
bit_operations_test/none: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
bug32372_test: CompileTimeError
call_method_as_cast_test/06: RuntimeError
call_method_implicit_tear_off_implements_function_test/05: RuntimeError
call_method_implicit_tear_off_implements_function_test/06: RuntimeError
Expand Down Expand Up @@ -47,29 +46,12 @@ int64_literal_test/17: RuntimeError, OK # Error if web int literal cannot be rep
int64_literal_test/19: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
int64_literal_test/none: RuntimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
issue23244_test: RuntimeError # Isolates - enum canonicalization - Issue 23244
issue32353_test: CompileTimeError
issue34404_flutter_test: CompileTimeError # --supermixin not supported
library_env_test/has_mirror_support: RuntimeError, OK
library_env_test/has_no_html_support: RuntimeError, OK
list_test: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
mint_arithmetic_test: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
mint_compares_test: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
mint_identical_test: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
mixin_type_parameter_inference_error_test/none: CompileTimeError
mixin_type_parameter_inference_previous_mixin_test/01: CompileTimeError
mixin_type_parameter_inference_previous_mixin_test/02: CompileTimeError
mixin_type_parameter_inference_previous_mixin_test/05: CompileTimeError
mixin_type_parameter_inference_previous_mixin_test/none: CompileTimeError
mixin_type_parameter_inference_test/01: CompileTimeError
mixin_type_parameter_inference_test/02: CompileTimeError
mixin_type_parameter_inference_test/03: CompileTimeError
mixin_type_parameter_inference_test/08: CompileTimeError
mixin_type_parameter_inference_test/09: CompileTimeError
mixin_type_parameter_inference_test/10: CompileTimeError
mixin_type_parameter_inference_test/12: CompileTimeError
mixin_type_parameter_inference_test/13: CompileTimeError
mixin_type_parameter_inference_test/16: CompileTimeError
mixin_type_parameter_inference_test/none: CompileTimeError
number_identity_test: CompileTimeError, OK # Error if web int literal cannot be represented exactly, see http://dartbug.com/33351
partial_instantiation_eager_bounds_check_test: RuntimeError # Issue #34295
partial_tearoff_instantiation_test/05: Pass # for the wrong reason.
Expand Down Expand Up @@ -346,29 +328,6 @@ mixin_mixin7_test: RuntimeError
mixin_mixin_bound2_test: RuntimeError
mixin_mixin_bound_test: RuntimeError
mixin_mixin_test: RuntimeError
mixin_of_mixin_test/none: CompileTimeError
mixin_super_2_test/none: CompileTimeError
mixin_super_test: CompileTimeError
mixin_super_use_test: CompileTimeError
mixin_superclass_test: CompileTimeError
mixin_supertype_subclass2_test/01: CompileTimeError
mixin_supertype_subclass2_test/02: CompileTimeError
mixin_supertype_subclass2_test/03: CompileTimeError
mixin_supertype_subclass2_test/04: CompileTimeError
mixin_supertype_subclass2_test/05: CompileTimeError
mixin_supertype_subclass2_test/none: CompileTimeError
mixin_supertype_subclass3_test/01: CompileTimeError
mixin_supertype_subclass3_test/02: CompileTimeError
mixin_supertype_subclass3_test/03: CompileTimeError
mixin_supertype_subclass3_test/04: CompileTimeError
mixin_supertype_subclass3_test/05: CompileTimeError
mixin_supertype_subclass3_test/none: CompileTimeError
mixin_supertype_subclass4_test/01: CompileTimeError
mixin_supertype_subclass4_test/02: CompileTimeError
mixin_supertype_subclass4_test/03: CompileTimeError
mixin_supertype_subclass4_test/04: CompileTimeError
mixin_supertype_subclass4_test/05: CompileTimeError
mixin_supertype_subclass4_test/none: CompileTimeError
mixin_type_parameters_super_test: RuntimeError
mock_writable_final_field_test: RuntimeError # Issue 30847
mock_writable_final_private_field_test: RuntimeError # Issue 17526, 30847
Expand Down Expand Up @@ -608,33 +567,6 @@ mixin_illegal_superclass_test/28: MissingCompileTimeError
mixin_illegal_superclass_test/29: MissingCompileTimeError
mixin_illegal_superclass_test/30: MissingCompileTimeError
mixin_method_override_test/G5: Skip # Issue 34354
mixin_of_mixin_test/none: CompileTimeError
mixin_super_2_test/none: CompileTimeError
mixin_super_test: CompileTimeError
mixin_super_use_test: CompileTimeError
mixin_superclass_test: CompileTimeError
mixin_supertype_subclass2_test/01: CompileTimeError
mixin_supertype_subclass2_test/02: CompileTimeError
mixin_supertype_subclass2_test/03: CompileTimeError
mixin_supertype_subclass2_test/04: CompileTimeError
mixin_supertype_subclass2_test/05: CompileTimeError
mixin_supertype_subclass2_test/none: CompileTimeError
mixin_supertype_subclass3_test/01: CompileTimeError
mixin_supertype_subclass3_test/02: CompileTimeError
mixin_supertype_subclass3_test/03: CompileTimeError
mixin_supertype_subclass3_test/04: CompileTimeError
mixin_supertype_subclass3_test/05: CompileTimeError
mixin_supertype_subclass3_test/none: CompileTimeError
mixin_supertype_subclass4_test/01: CompileTimeError
mixin_supertype_subclass4_test/02: CompileTimeError
mixin_supertype_subclass4_test/03: CompileTimeError
mixin_supertype_subclass4_test/04: CompileTimeError
mixin_supertype_subclass4_test/05: CompileTimeError
mixin_supertype_subclass4_test/none: CompileTimeError
mixin_supertype_subclass_test/01: CompileTimeError
mixin_supertype_subclass_test/03: CompileTimeError
mixin_supertype_subclass_test/04: CompileTimeError
mixin_supertype_subclass_test/none: CompileTimeError
mock_writable_final_field_test: RuntimeError # Issue 30847
mock_writable_final_private_field_test: RuntimeError # Issue 17526, 30847
modulo_test: RuntimeError # non JS number semantics
Expand Down
34 changes: 5 additions & 29 deletions tests/language_2/language_2_dartdevc.status
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ async_star_test/none: RuntimeError
await_future_test: Pass, Timeout # Issue 29920
bit_operations_test: RuntimeError # No bigints on web.
bug32372_test: RuntimeError
bug32372_test: Skip
built_in_identifier_prefix_test: CompileTimeError
built_in_identifier_type_annotation_test/dynamic-funarg: RuntimeError # Issue 28816
built_in_identifier_type_annotation_test/dynamic-funret: RuntimeError # Issue 28816
Expand Down Expand Up @@ -101,12 +100,9 @@ issue31596_super_test/05: CompileTimeError
issue31596_super_test/none: CompileTimeError
issue31596_tearoff_test: CompileTimeError
issue31596_test: CompileTimeError
issue32353_2_test: Skip
issue32353_test: RuntimeError
issue32353_test: Skip
issue32353_test: CompileTimeError # Issue 34846
issue34404_flutter_modified_test: CompileTimeError # DDC doesn't support mixin inference
issue34404_flutter_test: CompileTimeError # DDC doesn't support mixin inference
issue34404_flutter_test: Skip
issue34498_test: MissingCompileTimeError # Issue 34500
issue34635_test: MissingCompileTimeError
issue34636_test: MissingCompileTimeError
Expand Down Expand Up @@ -154,28 +150,12 @@ mixin_declaration/mixin_declaration_inference_valid_C13_test: CompileTimeError #
mixin_declaration/mixin_declaration_inference_valid_mixin_applications_test: CompileTimeError # https://github.com/dart-lang/sdk/issues/34164
mixin_declaration/mixin_declaration_invalid_superinvocation_test/10: CompileTimeError # Analyzer chooses wrong(?) super method.
mixin_method_override_test/01: MissingCompileTimeError
mixin_of_mixin_test: Skip
mixin_super_2_test: Skip
mixin_super_2_test/01: MissingCompileTimeError
mixin_super_2_test/03: MissingCompileTimeError
mixin_super_test: RuntimeError
mixin_super_test: Skip
mixin_super_use_test: RuntimeError
mixin_super_use_test: Skip
mixin_superclass_test: Skip
mixin_supertype_subclass2_test: Skip
mixin_supertype_subclass3_test: Skip
mixin_supertype_subclass4_test: Skip
mixin_supertype_subclass_test: Skip
mixin_supertype_subclass_test/02: MissingCompileTimeError
mixin_supertype_subclass_test/05: MissingCompileTimeError
mixin_type_parameter_inference_error_test: Skip
mixin_type_parameter_inference_previous_mixin_test: Skip
mixin_super_2_test: CompileTimeError # Issue 34806
mixin_super_use_test: CompileTimeError # Issue 34806
mixin_type_parameter_inference_previous_mixin_test/01: CompileTimeError
mixin_type_parameter_inference_previous_mixin_test/02: CompileTimeError
mixin_type_parameter_inference_previous_mixin_test/04: MissingCompileTimeError
mixin_type_parameter_inference_previous_mixin_test/05: RuntimeError
mixin_type_parameter_inference_test: Skip
mixin_type_parameter_inference_test/01: CompileTimeError
mixin_type_parameter_inference_test/02: CompileTimeError
mixin_type_parameter_inference_test/03: CompileTimeError
Expand Down Expand Up @@ -344,12 +324,8 @@ mixin_illegal_superclass_test/27: MissingCompileTimeError
mixin_illegal_superclass_test/28: MissingCompileTimeError
mixin_illegal_superclass_test/29: MissingCompileTimeError
mixin_illegal_superclass_test/30: MissingCompileTimeError
mixin_super_2_test/01: MissingCompileTimeError
mixin_super_2_test/03: MissingCompileTimeError
mixin_super_test: RuntimeError
mixin_super_use_test: RuntimeError
mixin_supertype_subclass_test/02: MissingCompileTimeError
mixin_supertype_subclass_test/05: MissingCompileTimeError
mixin_super_2_test: RuntimeError # Issue 34807
mixin_super_use_test: RuntimeError # Issue 34808
mock_writable_final_private_field_test: RuntimeError
multiline_newline_test/04: MissingCompileTimeError
multiline_newline_test/04r: MissingCompileTimeError
Expand Down
6 changes: 0 additions & 6 deletions tests/language_2/language_2_kernel.status
Original file line number Diff line number Diff line change
Expand Up @@ -296,12 +296,6 @@ super_call4_test/01: MissingCompileTimeError
[ $compiler != dart2js && $compiler != dartdevk && $fasta ]
const_native_factory_test: MissingCompileTimeError # Issue 29763

[ $compiler != dart2js && $fasta ]
mixin_super_2_test/01: MissingCompileTimeError
mixin_super_2_test/03: MissingCompileTimeError
mixin_supertype_subclass_test/02: MissingCompileTimeError
mixin_supertype_subclass_test/05: MissingCompileTimeError

[ $compiler == dartk && $runtime == vm && !$checked && $strong ]
assertion_initializer_const_error2_test/cc01: MissingCompileTimeError # Not reporting failed assert() at compile time.
assertion_initializer_const_error2_test/cc02: MissingCompileTimeError # Not reporting failed assert() at compile time.
Expand Down
56 changes: 0 additions & 56 deletions tests/language_2/mixin_of_mixin_test.dart

This file was deleted.

Loading

0 comments on commit 03765ed

Please sign in to comment.