Fix flake8 rules

Skyscanner · Apr 22, 2020 · 6b5a15e · 6b5a15e
1 parent 6f84af4
commit 6b5a15e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/skyscanner/skyscanner.py b/skyscanner/skyscanner.py
@@ -45,6 +45,7 @@ def configure_logger(log_level=logging.WARN):
     logger.addHandler(sa)
     return logger
 
+
 log = configure_logger()
 STRICT, GRACEFUL, IGNORE = 'strict', 'graceful', 'ignore'
 
@@ -92,7 +93,7 @@ def __init__(self, api_key, response_format='json'):
         if response_format.lower() not in self._SUPPORTED_FORMATS:
             raise ValueError(
                 'Unknown response format: {}'.format(response_format) +
-                ', supported formats are '.format(
+                + ', supported formats are '.format(
                     ', '.join(self._SUPPORTED_FORMATS)
                 )
             )

diff --git a/tests/test_skyscanner.py b/tests/test_skyscanner.py
@@ -343,8 +343,8 @@ def test_default_resp_callback_xml(self):
                               t._default_resp_callback,
                               FakeResponse(content='invalid XML')
                               )
-        except:
-            self.assertRaises(Exception,
+        except Exception as e:
+            self.assertRaises(e,
                               t._default_resp_callback,
                               FakeResponse(content='invalid XML')
                               )
@@ -840,5 +840,6 @@ def test_get_result_xml(self):
         self.assertTrue(
             len(self.result.findall('./Itineraries/ItineraryApiDto')) > 0)
 
+
 if __name__ == '__main__':
     unittest.main()