Add no_auth flag to skip auth checking for buckets that don't require it. Also fix tests

download_from_google_storage.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Download files from Google Storage based on SHA1 sums."""
 
 
 import hashlib
 import optparse
@@ skipping 279 matching lines @@
   parser.add_option('-d', '--directory', action='store_true',
                     help='The target is a directory.  '
                          'Cannot be used with -s/--sha1_file.')
   parser.add_option('-s', '--sha1_file', action='store_true',
                     help='The target is a file containing a sha1 sum.  '
                          'Cannot be used with -d/--directory.')
   parser.add_option('-g', '--config', action='store_true',
                     help='Alias for "gsutil config".  Run this if you want '
                          'to initialize your saved Google Storage '
                          'credentials.')
-  parser.add_option('-p', '--platform', 
+  parser.add_option('-n', '--no_auth', action='store_true',
+                    help='Skip auth checking.  Use if its known that the '

[ghost stip (do not use), 2013/11/19 19:32:37]

it's

+                         'target bucket is a public bucket.')
+  parser.add_option('-p', '--platform',
                     help='A regular expression that is compared against '
                          'Python\'s sys.platform. If this option is specified, '
                          'the download will happen only if there is a match.')
 
   (options, args) = parser.parse_args()
 
   # Make sure we should run at all based on platform matching.
   if options.platform:
     if not re.match(options.platform, sys.platform):
       print('The current platform doesn\'t match "%s", skipping.' %
@@ skipping 47 matching lines @@
     else:
       parser.error('Unreachable state.')
 
   # Check if output file already exists.
   if not options.directory and not options.force and not options.no_resume:
     if os.path.exists(options.output):
       parser.error('Output file %s exists and --no_resume is specified.'
                    % options.output)
 
   # Check we have a valid bucket with valid permissions.
-  base_url, code = check_bucket_permissions(options.bucket, gsutil)
+  if not options.no_auth:
+    base_url, code = check_bucket_permissions(options.bucket, gsutil)
   if code:
     return code
 
   return download_from_google_storage(
       input_filename, base_url, gsutil, options.num_threads, options.directory,
       options.recursive, options.force, options.output, options.ignore_errors,
       options.sha1_file)
 
 
 if __name__ == '__main__':
   sys.exit(main(sys.argv))