Merge pull request #215 from developmentseed/bug/191-bands

Bug/191 bands

Author: drewbo

landsat/landsat.py

@@ -216,7 +216,7 @@ def args_options():
                                  help="Provide Full sceneID, e.g. LC81660392014196LGN00")
 
     parser_download.add_argument('-b', '--bands', help='If you specify bands, landsat-util will try to download '
-                                 'the band from S3. If the band does not exist, an error is returned', default='')
+                                 'the band from S3. If the band does not exist, an error is returned', default=None)
     parser_download.add_argument('-d', '--dest', help='Destination path')
     parser_download.add_argument('-p', '--process', help='Process the image after download', action='store_true')
     parser_download.add_argument('--pansharpen', action='store_true',

tests/test_download.py

@@ -108,7 +108,7 @@ def test_remote_file_size(self):
         url = self.d.google_storage_url(self.d.scene_interpreter(self.scene))
         size = self.d.get_remote_file_size(url)
 
-        self.assertEqual(self.scene_size, size)
+        self.assertAlmostEqual(self.scene_size, size)
 
     def test_google_storage_url(self):
         sat = self.d.scene_interpreter(self.scene)

tests/test_landsat.py

@@ -109,7 +109,7 @@ def test_download_correct_zip(self, mock_downloader):
         args = ['download', 'LC80010092015051LGN00', '-d', self.mock_path]
         output = landsat.main(self.parser.parse_args(args))
         mock_downloader.assert_called_with(download_dir=self.mock_path, usgs_pass=None, usgs_user=None)
-        mock_downloader.return_value.download.assert_called_with(['LC80010092015051LGN00'], [])
+        mock_downloader.return_value.download.assert_called_with(['LC80010092015051LGN00'], None)
         self.assertEquals(output, ['Download Completed', 0])
 
     @mock.patch('landsat.landsat.process_image')