backend/lint: Do not shadow certain builtins

This commit enables ruff's flake8-builtin linter that emits warnings
when builtin functions are shadowed. This is useful for builtins like
"dict", "list", or "str" which we use often.

Given the nature of this program we historically rely a lot on the usage
of "id", "hash", and "filter" as variable names which also shadow Python
builtins. For now let's ignore those, we have not used any of them in
our code and the impact to the codebase would be considerable. This
might be revisited in the future.

2 files changed, 8 insertions, 8 deletions

diff --git a/tests/api/test_comic.py b/tests/api/test_comic.py
index 8555a5a..dcc5822 100644
--- a/tests/api/test_comic.py
+++ b/tests/api/test_comic.py
@@ -1097,7 +1097,7 @@ async def test_upsert_comic_tags_uses_existing(upsert_comics, empty_comic):
 
 
 @pytest.mark.parametrize(
-    "key,list",
+    "key,items",
     [
         ("artists", ["arty", "farty"]),
         ("tags", ["alien:medium", "human:tiny"]),
@@ -1116,11 +1116,11 @@ async def test_upsert_comic_tags_uses_existing(upsert_comics, empty_comic):
     ],
 )
 @pytest.mark.anyio
-async def test_upsert_comic_creates(upsert_comics, empty_comic, key, list):
+async def test_upsert_comic_creates(upsert_comics, empty_comic, key, items):
     original_comic = await DB.add(empty_comic)
 
     input = {
-        key: {"names": list, "options": {"onMissing": "CREATE"}},
+        key: {"names": items, "options": {"onMissing": "CREATE"}},
     }
     response = Response(await upsert_comics(original_comic.id, input))
     response.assert_is("UpsertSuccess")
@@ -1128,7 +1128,7 @@ async def test_upsert_comic_creates(upsert_comics, empty_comic, key, list):
     comic = await DB.get(Comic, original_comic.id, full=True)
     assert comic is not None
 
-    assert set(list) == set([o.name for o in getattr(comic, key)])
+    assert set(items) == set([o.name for o in getattr(comic, key)])
 
 
 @pytest.mark.anyio
diff --git a/tests/api/test_db.py b/tests/api/test_db.py
index 1405c23..b030035 100644
--- a/tests/api/test_db.py
+++ b/tests/api/test_db.py
@@ -67,8 +67,8 @@ async def test_models_retained_when_clearing_association(
     comic = await DB.add(comic)
 
     async with database.session() as s:
-        object = await s.get(Comic, comic.id)
-        setattr(object, key, [])
+        obj = await s.get(Comic, comic.id)
+        setattr(obj, key, [])
         await s.commit()
 
     assert await DB.get(assoccls, (comic.id, model.id)) is None
@@ -87,8 +87,8 @@ async def test_models_retained_when_clearing_comictag(empty_comic):
     await DB.add(ct)
 
     async with database.session() as s:
-        object = await s.get(Comic, comic.id)
-        object.tags = []
+        obj = await s.get(Comic, comic.id)
+        obj.tags = []
         await s.commit()
 
     assert await DB.get(ComicTag, (comic.id, ct.namespace_id, ct.tag_id)) is None