Flake8 fixes

Author: Sara Safavi

planet/api/client.py

@@ -146,7 +146,7 @@ def quick_search(self, request, **kw):
         '''Execute a quick search with the specified request.
 
         :param request: see :ref:`api-search-request`
-        :param \**kw: See Options below
+        :param **kw: See Options below
         :returns: :py:class:`planet.api.models.Items`
         :raises planet.api.exceptions.APIException: On API error.
 

planet/api/dispatch.py

@@ -84,7 +84,7 @@ def rebuild_auth(self, prepared_request, response):
             redir = prepared_request.url
             if not _is_subdomain_of_tld(orig, redir):
                 prepared_request.headers.pop('Authorization')
-                key = re.match('api-key (\S+)', existing_auth)
+                key = re.match(r'api-key (\S+)', existing_auth)
                 if key:
                     prepared_request.prepare_url(
                         prepared_request.url, {

planet/api/downloader.py

@@ -297,7 +297,7 @@ class Downloader(object):
 
     def shutdown(self):
         '''Halt execution.'''
-        raise NotImplemented()
+        raise NotImplementedError()
 
     def stats(self):
         '''Retrieve internal state of the Downloader.
@@ -311,7 +311,7 @@ def stats(self):
         - complete: `int` number of completed downloads
         - pending: `int` number of items awaiting download
         '''
-        raise NotImplemented()
+        raise NotImplementedError()
 
     def activate(self, items, asset_types):
         '''Request activation of specified asset_types for the sequence of
@@ -320,7 +320,7 @@ def activate(self, items, asset_types):
         :param items: a sequence of Item representations.
         :param asset_types list: list of asset-type (str)
         '''
-        raise NotImplemented()
+        raise NotImplementedError()
 
     def download(self, items, asset_types, dest):
         '''Request activation and download of specified asset_types for the
@@ -330,7 +330,7 @@ def download(self, items, asset_types, dest):
         :param asset_types list: list of asset-type (str)
         :param dest str: Download destination directory, must exist.
         '''
-        raise NotImplemented()
+        raise NotImplementedError()
 
     def on_complete(self, item, asset, path=None):
         '''Notification of processing an item's asset, invoked on completion of

planet/api/models.py

@@ -140,7 +140,7 @@ def noop(*a, **kw):
             total += size
             callback(wrote=size, total=total)
         # seems some responses don't have a content-length header
-        if self.size is 0:
+        if self.size == 0:
             self.size = total
         callback(finish=self)
 

planet/api/utils.py

@@ -291,7 +291,7 @@ def probably_wkt(text):
         'POINT', 'LINESTRING', 'POLYGON', 'MULTIPOINT',
         'MULTILINESTRING', 'MULTIPOLYGON', 'GEOMETRYCOLLECTION',
     ])
-    matched = re.match('(\w+)\s*\([^)]+\)', text.strip())
+    matched = re.match(r'(\w+)\s*\([^)]+\)', text.strip())
     if matched:
         valid = matched.group(1).upper() in valid_types
     return valid

planet/scripts/util.py

@@ -35,7 +35,7 @@
 
 def _split(value):
     '''return input split on any whitespace or comma'''
-    return re.split('\s+|,', value)
+    return re.split(r'\s+|,', value)
 
 
 # monkey patch warnings module to hide InsecurePlatformWarning - the warning

tests/test_atomic.py

@@ -46,8 +46,8 @@ def assert_content_is(expected):
 
     # manual discarding
     with atomic_open(outfile, 'w') as fp:
-            fp.write('bazzy')
-            fp.discard()
+        fp.write('bazzy')
+        fp.discard()
     assert_content_is('bar')
     # no tmp files remain
     assert ['foo'] == lsdir()