diff options
author | Frederick Muriuki Muriithi | 2022-11-15 04:22:17 +0300 |
---|---|---|
committer | Frederick Muriuki Muriithi | 2022-11-15 04:22:17 +0300 |
commit | b0d4aeb29c059c6dea85a7993149fa1e0697f702 (patch) | |
tree | 301bd3db1bb25cf622c7f83c6773e1ac5711ae7e /gn3 | |
parent | 9aee64c5bccae917ab0e65c882be8f442fc0f5ca (diff) | |
download | genenetwork3-b0d4aeb29c059c6dea85a7993149fa1e0697f702.tar.gz |
pylint: Fix linting errors.
Diffstat (limited to 'gn3')
-rw-r--r-- | gn3/auth/authorisation/groups.py | 1 | ||||
-rw-r--r-- | gn3/auth/db.py | 18 |
2 files changed, 14 insertions, 5 deletions
diff --git a/gn3/auth/authorisation/groups.py b/gn3/auth/authorisation/groups.py index ad30763..5290196 100644 --- a/gn3/auth/authorisation/groups.py +++ b/gn3/auth/authorisation/groups.py @@ -8,6 +8,7 @@ from . import authorised_p ("create-group",), success_message="Successfully created group.", error_message="Failed to create group.") def create_group(conn, group_name): + """Create a group""" with db.cursor(conn) as cursor: group_id = uuid.uuid4() cursor.execute( diff --git a/gn3/auth/db.py b/gn3/auth/db.py index c0d0415..e732a03 100644 --- a/gn3/auth/db.py +++ b/gn3/auth/db.py @@ -4,16 +4,24 @@ import contextlib @contextlib.contextmanager def connection(db_path: str): - connection = sqlite3.connect(db_path) + """Create the connection to the auth database.""" + conn = sqlite3.connect(db_path) try: - yield connection + yield conn + except: # pylint: disable=bare-except + conn.rollback() finally: - connection.close() + conn.commit() + conn.close() @contextlib.contextmanager -def cursor(connection): - cur = connection.cursor() +def cursor(conn): + """Get a cursor from the given connection to the auth database.""" + cur = conn.cursor() try: yield cur + except: # pylint: disable=bare-except + conn.rollback() finally: + conn.commit() cur.close() |