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

Side by Side Diff: tests/standalone/io/skipping_dart2js_compilations_test.dart

Issue 19589002: Fixed skipping_dart2js_compilations_test after changing Command() constructor (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | tests/standalone/io/test_runner_test.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 /* 5 /*
6 * This test makes sure that the "skipping Dart2Js compilations if the output is 6 * This test makes sure that the "skipping Dart2Js compilations if the output is
7 * already up to date" feature does work as it should. 7 * already up to date" feature does work as it should.
8 * Therefore this test ensures that compilations are only skipped if the last 8 * Therefore this test ensures that compilations are only skipped if the last
9 * modified date of the output of a dart2js compilation is newer than 9 * modified date of the output of a dart2js compilation is newer than
10 * - the the dart application to compile (including it's dependencies) 10 * - the the dart application to compile (including it's dependencies)
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
147 runner.TestCase makeTestCase(String testName, FileUtils fileUtils) { 147 runner.TestCase makeTestCase(String testName, FileUtils fileUtils) {
148 var config = new options.TestOptionsParser().parse(['--timeout', '2'])[0]; 148 var config = new options.TestOptionsParser().parse(['--timeout', '2'])[0];
149 var scriptDirPath = new Path(Platform.script).directoryPath; 149 var scriptDirPath = new Path(Platform.script).directoryPath;
150 var createFileScript = scriptDirPath. 150 var createFileScript = scriptDirPath.
151 append('skipping_dart2js_compilations_helper.dart').toNativePath(); 151 append('skipping_dart2js_compilations_helper.dart').toNativePath();
152 var executable = Platform.executable; 152 var executable = Platform.executable;
153 var arguments = [createFileScript, fileUtils.scriptOutputPath.toNativePath()]; 153 var arguments = [createFileScript, fileUtils.scriptOutputPath.toNativePath()];
154 var bootstrapDeps = [ 154 var bootstrapDeps = [
155 Uri.parse("file://${fileUtils.testSnapshotFilePath}")]; 155 Uri.parse("file://${fileUtils.testSnapshotFilePath}")];
156 var commands = [new runner.CompilationCommand( 156 var commands = [new runner.CompilationCommand(
157 'dart2js',
157 fileUtils.testJsFilePath.toNativePath(), 158 fileUtils.testJsFilePath.toNativePath(),
158 false, 159 false,
159 bootstrapDeps, 160 bootstrapDeps,
160 executable, 161 executable,
161 arguments)]; 162 arguments)];
162 return new runner.TestCase( 163 return new runner.TestCase(
163 testName, 164 testName,
164 commands, 165 commands,
165 config, 166 config,
166 (_) {}, 167 (_) {},
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
240 cleanup(); 241 cleanup();
241 throw error; 242 throw error;
242 }).then((_) { 243 }).then((_) {
243 cleanup(); 244 cleanup();
244 }); 245 });
245 } 246 }
246 // We need to wait some time to make sure that the files we 'touch' get a 247 // We need to wait some time to make sure that the files we 'touch' get a
247 // bigger timestamp than the old ones 248 // bigger timestamp than the old ones
248 new Timer(new Duration(seconds: 1), touchFilesAndRunTests); 249 new Timer(new Duration(seconds: 1), touchFilesAndRunTests);
249 } 250 }
OLDNEW
« no previous file with comments | « no previous file | tests/standalone/io/test_runner_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698