update hooks and autoformat with newer black

Author: bagerard

.pre-commit-config.yaml

@@ -8,7 +8,7 @@ repos:
       - id: trailing-whitespace
       - id: end-of-file-fixer
   - repo: https://github.com/ambv/black
-    rev: 22.12.0
+    rev: 23.1.0
     hooks:
       - id: black
   - repo: https://github.com/pycqa/flake8
@@ -23,6 +23,6 @@ repos:
       - id: pyupgrade
         args: [--py36-plus]
   - repo: https://github.com/pycqa/isort
-    rev: 5.11.4
+    rev: 5.12.0
     hooks:
       - id: isort

mongoengine/base/document.py

@@ -161,7 +161,6 @@ def __delattr__(self, *args, **kwargs):
     def __setattr__(self, name, value):
         # Handle dynamic data only if an initialised dynamic document
         if self._dynamic and not self._dynamic_lock:
-
             if name not in self._fields_ordered and not name.startswith("_"):
                 DynamicField = _import_class("DynamicField")
                 field = DynamicField(db_field=name, null=True)

mongoengine/base/metaclasses.py

@@ -178,7 +178,6 @@ def __new__(mcs, name, bases, attrs):
                 f.owner_document = new_class
             delete_rule = getattr(f, "reverse_delete_rule", DO_NOTHING)
             if isinstance(f, CachedReferenceField):
-
                 if issubclass(new_class, EmbeddedDocument):
                     raise InvalidDocumentError(
                         "CachedReferenceFields is not allowed in EmbeddedDocuments"

mongoengine/connection.py

@@ -102,7 +102,6 @@ def _get_connection_settings(
 
     resolved_hosts = []
     for entity in conn_host:
-
         # Reject old mongomock integration
         # To be removed in a few versions after 0.27.0
         if entity.startswith("mongomock://") or kwargs.get("is_mock"):

mongoengine/dereference.py

@@ -165,7 +165,6 @@ def _fetch_objects(self, doc_type=None):
         """Fetch all references and convert to their document objects"""
         object_map = {}
         for collection, dbrefs in self.reference_map.items():
-
             # we use getattr instead of hasattr because hasattr swallows any exception under python2
             # so it could hide nasty things without raising exceptions (cfr bug #1688))
             ref_document_cls_exists = getattr(collection, "objects", None) is not None

mongoengine/document.py

@@ -918,7 +918,6 @@ def ensure_indexes(cls):
         # If _cls is being used (for polymorphism), it needs an index,
         # only if another index doesn't begin with _cls
         if index_cls and not cls_indexed and cls._meta.get("allow_inheritance"):
-
             # we shouldn't pass 'cls' to the collection.ensureIndex options
             # because of https://jira.mongodb.org/browse/SERVER-769
             if "cls" in index_opts:

mongoengine/fields.py

@@ -981,7 +981,6 @@ def prepare_query_value(self, op, value):
             self.error("List is too long")
 
         if self.field:
-
             # If the value is iterable and it's not a string nor a
             # BaseDocument, call prepare_query_value for each of its items.
             if (
@@ -2216,7 +2215,6 @@ def __get__(self, instance, owner):
         return value
 
     def __set__(self, instance, value):
-
         if value is None and instance._initialised:
             value = self.generate()
 

mongoengine/queryset/base.py

@@ -1120,7 +1120,6 @@ def order_by(self, *keys):
         new_ordering = queryset._get_order_by(keys)
 
         if queryset._cursor_obj:
-
             # If a cursor object has already been created, apply the sort to it
             if new_ordering:
                 queryset._cursor_obj.sort(new_ordering)

mongoengine/queryset/queryset.py

@@ -87,7 +87,6 @@ def _iter_results(self):
 
         pos = 0
         while True:
-
             # For all positions lower than the length of the current result
             # cache, serve the docs straight from the cache w/o hitting the
             # database.

tests/document/test_instance.py

@@ -1526,7 +1526,6 @@ class Doc(Document):
         assert my_doc.int_field == 1
 
     def test_document_update(self):
-
         # try updating a non-saved document
         with pytest.raises(OperationError):
             person = self.Person(name="dcrosta")

tests/fields/test_dict_field.py

@@ -324,6 +324,7 @@ class Simple(Document):
 
     def test_dictfield_with_referencefield_complex_nesting_cases(self):
         """Ensure complex nesting inside DictField handles dereferencing of ReferenceField(dbref=True | False)"""
+
         # Relates to Issue #1453
         class Doc(Document):
             s = StringField()

tests/fields/test_fields.py

@@ -239,6 +239,7 @@ def test_default_value_is_not_used_when_changing_value_to_empty_list_for_strict_
         self,
     ):
         """List field with default can be set to the empty list (strict)"""
+
         # Issue #1733
         class Doc(Document):
             x = ListField(IntField(), default=lambda: [42])
@@ -253,6 +254,7 @@ def test_default_value_is_not_used_when_changing_value_to_empty_list_for_dyn_doc
         self,
     ):
         """List field with default can be set to the empty list (dynamic)"""
+
         # Issue #1733
         class Doc(DynamicDocument):
             x = ListField(IntField(), default=lambda: [42])

tests/fields/test_file_field.py

@@ -535,7 +535,6 @@ class TestFile(Document):
     @require_pil
     def test_get_image_by_grid_id(self):
         class TestImage(Document):
-
             image1 = ImageField()
             image2 = ImageField()
 

tests/fields/test_geo_fields.py

@@ -369,7 +369,6 @@ class Parent(Document):
         assert len(Location._geo_indices()) == 1
 
     def test_geo_indexes_auto_index(self):
-
         # Test just listing the fields
         class Log(Document):
             location = PointField(auto_index=False)

tests/queryset/test_queryset.py

@@ -604,7 +604,6 @@ class BlogPost(Document):
         assert post.comments[1].votes == 8
 
     def test_update_using_positional_operator_matches_first(self):
-
         # Currently the $ operator only applies to the first matched item in
         # the query
 
@@ -5611,7 +5610,6 @@ class Person(Document):
         # Check that bool(queryset) does not uses the orderby
         qs = Person.objects.order_by("name")
         with query_counter() as q:
-
             if bool(qs):
                 pass
 
@@ -5624,7 +5622,6 @@ class Person(Document):
         # Check that normal query uses orderby
         qs2 = Person.objects.order_by("name")
         with query_counter() as q:
-
             for x in qs2:
                 pass
 
@@ -5648,7 +5645,6 @@ class Person(Document):
         Person(name="A").save()
 
         with query_counter() as q:
-
             if Person.objects:
                 pass
 

tests/test_context_managers.py

@@ -76,7 +76,6 @@ class Group(Document):
         assert 1 == Group.objects.count()
 
         with switch_db(Group, "testdb-1") as Group:
-
             assert 0 == Group.objects.count()
 
             Group(name="hello").save()
@@ -104,7 +103,6 @@ class Group(Document):
         assert 1 == Group.objects.count()
 
         with switch_collection(Group, "group1") as Group:
-
             assert 0 == Group.objects.count()
 
             Group(name="hello - group1").save()

tests/test_signals.py

@@ -97,7 +97,6 @@ def post_bulk_insert(cls, sender, documents, **kwargs):
         Author.id.set_next_value(0)
 
         class Another(Document):
-
             name = StringField()
 
             def __unicode__(self):