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

Unified Diff: tests/compiler/dart2js/serialization/compilation_test_helper.dart

Issue 2156033002: Split out long-running serialization test and add timings to output (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 4 years, 5 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
Index: tests/compiler/dart2js/serialization/compilation_test_helper.dart
diff --git a/tests/compiler/dart2js/serialization/compilation_test_helper.dart b/tests/compiler/dart2js/serialization/compilation_test_helper.dart
index 4605e8e87fd12617a11d5b480a84181d427d4f94..4445f9dcfedbb0b3c33a4ad1bca2b27a02bae788 100644
--- a/tests/compiler/dart2js/serialization/compilation_test_helper.dart
+++ b/tests/compiler/dart2js/serialization/compilation_test_helper.dart
@@ -28,6 +28,7 @@ main(List<String> args) {
Uri entryPoint = Uri.parse('memory:main.dart');
await arguments.forEachTest(serializedData, TESTS, compile);
}
+ printMeasurementResults();
});
}
@@ -40,20 +41,20 @@ Future compile(
bool verbose: false}) async {
String testDescription = test != null ? test.name : '${entryPoint}';
String id = index != null ? '$index: ' : '';
- print('------------------------------------------------------------------');
- print('compile ${id}${testDescription}');
- print('------------------------------------------------------------------');
+ String title = '${id}${testDescription}';
OutputCollector outputCollector = new OutputCollector();
- List<String> options = [];
- if (test.checkedMode) {
- options.add(Flags.enableCheckedMode);
- }
- await runCompiler(
- entryPoint: entryPoint,
- memorySourceFiles: sourceFiles,
- resolutionInputs: resolutionInputs,
- options: options,
- outputProvider: outputCollector);
+ await measure(title, 'compile', () async {
+ List<String> options = [];
+ if (test.checkedMode) {
+ options.add(Flags.enableCheckedMode);
+ }
+ await runCompiler(
+ entryPoint: entryPoint,
+ memorySourceFiles: sourceFiles,
+ resolutionInputs: resolutionInputs,
+ options: options,
+ outputProvider: outputCollector);
+ });
if (verbose) {
print(outputCollector.getOutput('', 'js'));
}
« no previous file with comments | « tests/compiler/dart2js/serialization/analysis_test_helper.dart ('k') | tests/compiler/dart2js/serialization/helper.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698