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

Unified Diff: tests/standalone/io/test_extension_test.dart

Issue 2771453003: Format all tests. (Closed)
Patch Set: Format files Created 3 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
Index: tests/standalone/io/test_extension_test.dart
diff --git a/tests/standalone/io/test_extension_test.dart b/tests/standalone/io/test_extension_test.dart
index cf98925d5dbf2f69323949eb1fae05d5eec3abef..c2c094862352fe77205b86712d768d9851549327 100644
--- a/tests/standalone/io/test_extension_test.dart
+++ b/tests/standalone/io/test_extension_test.dart
@@ -33,7 +33,7 @@ List<String> getExtensionNames(String arch) {
case 'macos':
return ['libtest_extension.dylib', 'libtest_extension$arch.dylib'];
case 'windows':
- return ['test_extension.dll','test_extension$arch.dll'];
+ return ['test_extension.dll', 'test_extension$arch.dll'];
default:
Expect.fail('Unknown operating system ${Platform.operatingSystem}');
}
@@ -84,7 +84,7 @@ Future testExtension(bool withArchSuffix) {
// Copy test_extension shared library, test_extension.dart and
// test_extension_tester.dart to the temporary test directory.
return copyFileToDirectory(getExtensionPath(buildDirectory, fileNames[0]),
- join(testDirectory, fileNames[1])).then((_) {
+ join(testDirectory, fileNames[1])).then((_) {
var extensionDartFile = join(scriptDirectory, 'test_extension.dart');
return copyFileToDirectory(extensionDartFile, testDirectory);
}).then((_) {
@@ -94,19 +94,21 @@ Future testExtension(bool withArchSuffix) {
}).then((_) {
var script = join(testDirectory, 'test_extension_tester.dart');
return Process.run(Platform.executable, [script]);
- })..then((ProcessResult result) {
- if (result.exitCode != 0) {
- print('Subprocess failed with exit code ${result.exitCode}');
- print('stdout:');
- print('${result.stdout}');
- print('stderr:');
- print('${result.stderr}');
- }
- Expect.equals(0, result.exitCode);
- tempDirectory.deleteSync(recursive: true);
- })..catchError((_) {
- tempDirectory.deleteSync(recursive: true);
- });
+ })
+ ..then((ProcessResult result) {
+ if (result.exitCode != 0) {
+ print('Subprocess failed with exit code ${result.exitCode}');
+ print('stdout:');
+ print('${result.stdout}');
+ print('stderr:');
+ print('${result.stderr}');
+ }
+ Expect.equals(0, result.exitCode);
+ tempDirectory.deleteSync(recursive: true);
+ })
+ ..catchError((_) {
+ tempDirectory.deleteSync(recursive: true);
+ });
}
Future testWithArchSuffix() {

Powered by Google App Engine
This is Rietveld 408576698