fix flake8 violations, mostly trivial

Author: ischneider

planet/api/_fatomic.py

@@ -52,7 +52,8 @@ def atomic_open(filename, mode, *args, **kwargs):
                                     dir=os.path.dirname(filename),
                                     suffix='.tmp',
                                     delete=False)
-    _discard = [False]
+    # track: explicitly discarded, normal/abnormal completion
+    _discard = [None]
     try:
         if mode[0] == 'a':
             try:
@@ -63,17 +64,18 @@ def atomic_open(filename, mode, *args, **kwargs):
                     pass
 
         def discard(self, _discard=_discard):
+            # explicit discard
             _discard[0] = True
         f.discard = types.MethodType(discard, f)
         yield f
-    # ischneider addition to fatomic - catch/raise any exception thrown in with
+        # normal completion
+        if not _discard[0]:
+            _discard[0] = False
     # block and force discarding
-    except:
-        _discard = [True]
-        raise
     finally:
         f.close()
-        if _discard[0]:
+        # if we didn't complete or were aborted, delete
+        if _discard[0] is None or _discard[0]:
             os.unlink(f.name)
         else:
             _replace_file(f.name, filename)

planet/api/downloader.py

@@ -19,7 +19,7 @@
 from planet.api.exceptions import RequestCancelled
 try:
     import Queue as queue
-except:
+except ImportError:
     # renamed in 3
     import queue
 

planet/scripts/item_asset_types.py

@@ -43,20 +43,16 @@ def _get_json_or_raise(url, timeout=0.7):
 def get_item_types():
     global _item_types
     if _item_types is None:
-        try:
-            data = _get_json_or_raise(ITEM_TYPE_URL)
-            _item_types = [it['id'] for it in data['item_types']]
-        except:
-            _item_types = DEFAULT_ITEM_TYPES
+        _item_types = DEFAULT_ITEM_TYPES
+        data = _get_json_or_raise(ITEM_TYPE_URL)
+        _item_types = [it['id'] for it in data['item_types']]
     return _item_types
 
 
 def get_asset_types():
     global _asset_types
     if _asset_types is None:
-        try:
-            data = _get_json_or_raise(ASSET_TYPE_URL)
-            _asset_types = [a['id'] for a in data['asset_types']]
-        except:
-            _asset_types = DEFAULT_ASSET_TYPES
+        _asset_types = DEFAULT_ASSET_TYPES
+        data = _get_json_or_raise(ASSET_TYPE_URL)
+        _asset_types = [a['id'] for a in data['asset_types']]
     return _asset_types

tests/test_atomic.py

@@ -37,7 +37,7 @@ def assert_content_is(expected):
         with atomic_open(outfile, 'w') as fp:
             fp.write('bazzy')
             raise Exception('drat')
-    except:
+    except Exception:
         assert_content_is('bar')
     else:
         assert False

tests/test_v1_cli.py

@@ -39,7 +39,7 @@ def assert_success(result, expected_output, exit_code=0):
             pass
     try:
         assert json.loads(result.output) == expected_output
-    except:
+    except ValueError:
         assert result.output == expected_output