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

Unified Diff: tests/compiler/dart2js/serialization_analysis_test.dart

Issue 1870133002: Refactor serialization test files. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Fix Created 4 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 | « no previous file | tests/compiler/dart2js/serialization_impact_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js/serialization_analysis_test.dart
diff --git a/tests/compiler/dart2js/serialization_analysis_test.dart b/tests/compiler/dart2js/serialization_analysis_test.dart
index 373fb89e5bb98fd572e595c458243612f3747e9a..39c46de2e241238b484c9166e5e3970c977c9d24 100644
--- a/tests/compiler/dart2js/serialization_analysis_test.dart
+++ b/tests/compiler/dart2js/serialization_analysis_test.dart
@@ -14,192 +14,7 @@ import 'package:compiler/src/compiler.dart';
import 'package:compiler/src/filenames.dart';
import 'memory_compiler.dart';
import 'serialization_helper.dart';
-
-const List<Test> TESTS = const <Test>[
- const Test(const {
- 'main.dart': 'main() {}'
- }),
-
- const Test(const {
- 'main.dart': 'main() => print("Hello World");'
- }),
-
- const Test(const {
- 'main.dart': 'main() => print("Hello World", 0);'
- },
- expectedWarningCount: 1,
- expectedInfoCount: 1),
-
- const Test(const {
- 'main.dart': r'''
-main() {
- String text = "Hello World";
- print('$text');
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-main() {
- String text = "Hello World";
- print('$text', text);
-}'''
- },
- expectedWarningCount: 1,
- expectedInfoCount: 1),
-
- const Test(const {
- 'main.dart': r'''
-main(List<String> arguments) {
- print(arguments);
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-main(List<String> arguments) {
- for (int i = 0; i < arguments.length; i++) {
- print(arguments[i]);
- }
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-main(List<String> arguments) {
- for (String argument in arguments) {
- print(argument);
- }
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-class Class {}
-main() {
- print(new Class());
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-class Class implements Function {}
-main() {
- print(new Class());
-}'''
- },
- expectedWarningCount: 1),
-
- const Test(const {
- 'main.dart': r'''
-class Class implements Function {
- call() {}
-}
-main() {
- print(new Class()());
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-class Class implements Comparable<Class> {
- int compareTo(Class other) => 0;
-}
-main() {
- print(new Class());
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-class Class implements Comparable<Class, Class> {
- int compareTo(other) => 0;
-}
-main() {
- print(new Class());
-}'''
- },
- expectedWarningCount: 1),
-
- const Test(const {
- 'main.dart': r'''
-class Class implements Comparable<Class> {
- int compareTo(String other) => 0;
-}
-main() {
- print(new Class().compareTo(null));
-}'''
- },
- expectedWarningCount: 1,
- expectedInfoCount: 1),
-
- const Test(const {
- 'main.dart': r'''
-class Class implements Comparable {
- bool compareTo(a, b) => true;
-}
-main() {
- print(new Class().compareTo(null, null));
-}'''
- },
- expectedWarningCount: 1,
- expectedInfoCount: 1),
-
- const Test(const {
- 'main.dart': r'''
-import 'dart:math';
-
-class MyRandom implements Random {
- int nextInt(int max) {
- return max.length;
- }
- bool nextBool() => true;
- double nextDouble() => 0.0;
-}
-main() {
- new MyRandom().nextInt(0);
-}'''
- },
- expectedWarningCount: 1,
- expectedInfoCount: 0),
-
- const Test(const {
- 'main.dart': r'''
-import 'dart:math';
-
-class MyRandom implements Random {
- int nextInt(int max) {
- return max.length;
- }
- bool nextBool() => true;
- double nextDouble() => 0.0;
-}
-main() {
- new MyRandom();
-}'''
- }),
-
- const Test(const {
- 'main.dart': r'''
-import 'dart:math';
-
-class MyRandom implements Random {
- int nextInt(int max) {
- return max.length;
- }
- bool nextBool() => true;
- double nextDouble() => 0.0;
-}
-main() {
- // Invocation of `MyRandom.nextInt` is only detected knowing the actual
- // implementation class for `List` and the world impact of its `shuffle`
- // method.
- [].shuffle(new MyRandom());
-}'''
- },
- expectedWarningCount: 1,
- expectedInfoCount: 0),
-];
+import 'serialization_test_data.dart';
main(List<String> arguments) {
asyncTest(() async {
@@ -217,20 +32,6 @@ main(List<String> arguments) {
});
}
-class Test {
- final Map sourceFiles;
- final int expectedErrorCount;
- final int expectedWarningCount;
- final int expectedHintCount;
- final int expectedInfoCount;
-
- const Test(this.sourceFiles, {
- this.expectedErrorCount: 0,
- this.expectedWarningCount: 0,
- this.expectedHintCount: 0,
- this.expectedInfoCount: 0});
-}
-
Future analyze(String serializedData, Uri entryPoint, Test test) async {
DiagnosticCollector diagnosticCollector = new DiagnosticCollector();
await runCompiler(
« no previous file with comments | « no previous file | tests/compiler/dart2js/serialization_impact_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698