Silence pub --version tests

tools/observatory_tool.py

 #!/usr/bin/env python
 # 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.
 """Helper for building and deploying Observatory"""
 
 import argparse
 import os
 import platform
 import shutil
@@ skipping 23 matching lines @@
   result.add_argument("--package-root", help="package root", default=None)
   result.add_argument("--dart-executable", help="dart executable", default=None)
   result.add_argument("--pub-executable", help="pub executable", default=None)
   result.add_argument("--directory", help="observatory root", default=None)
   result.add_argument("--command", help="[get, build, deploy]", default=None)
   result.add_argument("--silent", help="silence all output", default=False)
   result.add_argument("--sdk", help="Use prebuilt sdk", default=False)
   return result
 
 def ProcessOptions(options, args):
-  # Required options.
-  if options.command is None or options.directory is None:
-    return False
+  with open(os.devnull, 'wb') as silent_sink:
+    # Required options.
+    if options.command is None or options.directory is None:
+      return False
 
-  # Set a default value for pub_snapshot.
-  options.pub_snapshot = None
+    # Set a default value for pub_snapshot.
+    options.pub_snapshot = None
 
-  # If we have a working pub executable, try and use that.
-  # TODO(whesse): Drop the pub-executable option if it isn't used.
-  if options.pub_executable is not None:
-    try:
-      if 0 == subprocess.call([options.pub_executable, '--version']):
-        return True
-    except OSError as e:
-      pass
-  options.pub_executable = None
+    # If we have a working pub executable, try and use that.
+    # TODO(whesse): Drop the pub-executable option if it isn't used.
+    if options.pub_executable is not None:
+      try:
+        if 0 == subprocess.call([options.pub_executable, '--version'],
+                                stdout=silent_sink,
+                                stderr=silent_sink):
+          return True
+      except OSError as e:
+        pass
+    options.pub_executable = None
 
-  if options.sdk is not None and utils.CheckedInSdkCheckExecutable():
-    # Use the checked in pub executable.
-    options.pub_snapshot = os.path.join(utils.CheckedInSdkPath(),
-                                        'bin',
-                                        'snapshots',
-                                        'pub.dart.snapshot');
-    try:
-      if 0 == subprocess.call([utils.CheckedInSdkExecutable(),
-                               options.pub_snapshot,
-                               '--version']):
-        return True
-    except OSError as e:
-      pass
-  options.pub_snapshot = None
+    if options.sdk is not None and utils.CheckedInSdkCheckExecutable():
+      # Use the checked in pub executable.
+      options.pub_snapshot = os.path.join(utils.CheckedInSdkPath(),
+                                          'bin',
+                                          'snapshots',
+                                          'pub.dart.snapshot');
+      try:
+        if 0 == subprocess.call([utils.CheckedInSdkExecutable(),
+                                 options.pub_snapshot,
+                                 '--version'],
+                                 stdout=silent_sink,
+                                 stderr=silent_sink):
+          return True
+      except OSError as e:
+        pass
+    options.pub_snapshot = None
 
-  # We need a dart executable and a package root.
-  return (options.package_root is not None and
-          options.dart_executable is not None)
+    # We need a dart executable and a package root.
+    return (options.package_root is not None and
+            options.dart_executable is not None)

[Bob Nystrom, 2015/09/29 16:06:57]

Drive-by! Instead of making an explicit null sink, I've used this before:

try:
  subprocess.check_output(args, stderr=subprocess.STDOUT)
except subprocess.CalledProcessError as error:
  ...

check_output consumes and returns stdout. If you redirect stderr to that, it
consumes all output. It raises a CalledProcessError error if the process exits
with a non-zero exit code.

 
 def ChangeDirectory(directory):
   os.chdir(directory);
 
 def DisplayBootstrapWarning():
   print """\
 
 
 WARNING: Your system cannot run the checked-in Dart SDK. Using the
 bootstrap Dart executable will make debug builds slow.
@@ skipping 83 matching lines @@
   if (options.pub_snapshot != None):
     options.pub_snapshot = os.path.abspath(options.pub_snapshot)
   if len(args) == 1:
     args[0] = os.path.abspath(args[0])
   # Pub must be run from the project's root directory.
   ChangeDirectory(options.directory)
   return ExecuteCommand(options, args)
 
 if __name__ == '__main__':
   sys.exit(main());