flake8 pass

- fix test_empty_embed not actually reporting on empty embeds

litecord/blueprints/auth.py

@@ -64,7 +64,7 @@ async def register():
 
     j = await request.get_json()
 
-    if not "password" in j:
+    if "password" not in j:
         # we need a password to generate a token.
         # passwords are optional, so
         j["password"] = "default_password"

litecord/blueprints/users.py

@@ -259,7 +259,7 @@ async def patch_me():
             user_id,
         )
 
-    if user["email"] is None and not "new_password" in j:
+    if user["email"] is None and "new_password" not in j:
         raise BadRequest("missing password", {"password": "Please set a password."})
 
     if "new_password" in j and j["new_password"]:

litecord/enums.py

@@ -33,7 +33,7 @@ class EasyEnum(Enum):
 
 class Flags:
     """Construct a class that represents a bitfield.
-    
+
     You can use it like this:
         >>> class MyField(Flags):
                 field_1 = 1

litecord/pubsub/dispatcher.py

@@ -25,17 +25,20 @@ from logbook import Logger
 log = Logger(__name__)
 
 
+def _identity(_self, x):
+    return x
+
+
 class Dispatcher:
     """Pub/Sub backend dispatcher.
-    
+
     This just declares functions all Dispatcher subclasses
     can implement. This does not mean all Dispatcher
     subclasses have them implemented.
     """
 
-    # the _ parameter is for (self)
+    KEY_TYPE = _identity
-    KEY_TYPE = lambda _, x: x
+    VAL_TYPE = _identity
-    VAL_TYPE = lambda _, x: x
 
     def __init__(self, main):
         #: main EventDispatcher

litecord/utils.py

@@ -45,7 +45,7 @@ async def task_wrapper(name: str, coro):
         await coro
     except asyncio.CancelledError:
         pass
-    except:
+    except Exception:
         log.exception("{} task error", name)
 
 
@@ -201,7 +201,7 @@ def to_update(j: dict, orig: dict, field: str) -> bool:
 
 async def search_result_from_list(rows: List) -> Dict[str, Any]:
     """Generate the end result of the search query, given a list of rows.
-    
+
     Each row must contain:
      - A bigint on `current_id`
      - An int (?) on `total_results`

manage/cmd/migration/command.py

@@ -170,7 +170,7 @@ async def apply_migration(app, migration: Migration) -> bool:
         log.info("applied {} {}", migration.id, migration.name)
 
         return True
-    except:
+    except Exception:
         log.exception("failed to run migration, rollbacking log")
         await _delete_log(app, migration.id)
 

tests/test_embeds.py

@@ -30,20 +30,16 @@ def valid(embed: dict):
     try:
         validate_embed(embed)
         return True
-    except:
+    except Exception:
         return False
 
 
 def invalid(embed):
-    try:
+    return not valid(embed)
-        validate_embed(embed)
-        return False
-    except:
-        return True
 
 
 def test_empty_embed():
-    valid({})
+    assert valid({})
 
 
 def test_basic_embed():