fedora-hubs

Clone

#420 Fix PEP8 failures in the tests
Merged 6 years ago by abompard. Opened 6 years ago by ryanlerch.
ryanlerch/fedora-hubs fix-pep8-E722  into  develop

file modified
+1 -1 
@@ -60,7 +60,7 @@ 
 

          with hubs.app.app.app_context():  # so url_for works
 

              widget_cache_work(widget_idx, fn_name)
 

          session.commit()  # transaction is committed here
 

-     except:
 

+     except Exception:
 

          session.rollback()  # rolls back the transaction
 

          raise
 

      finally:

file modified
+1 -1 
@@ -69,7 +69,7 @@ 
 

              link = data['link']
 

              icon = data['secondary_icon']
 

              dom_id = data['dom_id']
 

-         except:
 

+         except Exception:
 

              return flask.abort(400)
 

          notification = hubs.models.SavedNotification(
 

              username=user.username,

The Style tests were failing on E722 due to two lines
using bare excepts. This commit fixes this issue

Signed-off-by: Ryan Lerch rlerch@redhat.com

Pull-Request has been merged by abompard
6 years ago
Metadata
None
No Tags
Changes Summary 2
+1 -1
file changed
hubs/backend/worker.py
+1 -1
file changed
hubs/views/user.py
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.