- Return exit code 253 in case of API errors during loading. This

runtime/tests/vm/dart/snapshot_version_test.dart

Index: runtime/tests/vm/dart/snapshot_version_test.dart
===================================================================
--- runtime/tests/vm/dart/snapshot_version_test.dart	(revision 0)
+++ runtime/tests/vm/dart/snapshot_version_test.dart	(working copy)
@@ -0,0 +1,14 @@
+// Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+import "dart:io";
+import "package:expect/expect.dart";
+
+main() {
+  var result = Process.runSync(Platform.executable,
+      [Platform.script.resolve('./bad_snapshot').toFilePath()]);

[ricow1, 2015/02/05 14:00:42]

do you need ./ here? (not sure how that works on windows)

[Ivan Posva, 2015/02/05 14:26:24]

As far as I remember / is the right thing to use for URI resolution on Windows.
I prefer the ./ to make it absolutely clear that this file is in the current
directory.

+  print("=== stdout ===\n ${result.stdout}");

[ricow1, 2015/02/05 14:00:42]

remove the prints?

[Ivan Posva, 2015/02/05 14:26:24]

The prints make it much easier to diagnose in case of failure.

+  print("=== stderr ===\n ${result.stderr}");
+  Expect.equals(result.exitCode, 253);
+}