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

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

Issue 285693005: Fix default value of stored output in skipping tests (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 6 years, 7 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 | no next file » | 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 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 } 58 }
59 if (createJsDeps) { 59 if (createJsDeps) {
60 testJsDeps = _createFile(testJsDepsFilePath); 60 testJsDeps = _createFile(testJsDepsFilePath);
61 var path = suite.TestUtils.absolutePath(new Path(tempDir.path)) 61 var path = suite.TestUtils.absolutePath(new Path(tempDir.path))
62 .append("test.dart"); 62 .append("test.dart");
63 _writeToFile(testJsDeps, "file://$path"); 63 _writeToFile(testJsDeps, "file://$path");
64 } 64 }
65 if (createCachedOutput) { 65 if (createCachedOutput) {
66 testCachedOutput = _createFile(testCachedOutputPath); 66 testCachedOutput = _createFile(testCachedOutputPath);
67 var content = JSON.encode({ 67 var content = JSON.encode({
68 'stdout': '', 68 'stdout': [],
69 'stderr': '', 69 'stderr': [],
70 'exitCode': 0, 70 'exitCode': 0,
71 'timedOut': false 71 'timedOut': false
72 }); 72 });
73 _writeToFile(testCachedOutput, content); 73 _writeToFile(testCachedOutput, content);
74 } 74 }
75 75
76 } 76 }
77 77
78 void cleanup() { 78 void cleanup() {
79 if (testJs != null) testJs.deleteSync(); 79 if (testJs != null) testJs.deleteSync();
(...skipping 193 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 cleanup(); 273 cleanup();
274 throw error; 274 throw error;
275 }).then((_) { 275 }).then((_) {
276 cleanup(); 276 cleanup();
277 }); 277 });
278 } 278 }
279 // We need to wait some time to make sure that the files we 'touch' get a 279 // We need to wait some time to make sure that the files we 'touch' get a
280 // bigger timestamp than the old ones 280 // bigger timestamp than the old ones
281 new Timer(new Duration(seconds: 1), touchFilesAndRunTests); 281 new Timer(new Duration(seconds: 1), touchFilesAndRunTests);
282 } 282 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698