Merge pull request #505 from MongoEngine/remove_tests_warnings

Changed: Tests: treat all warnings as errors (#498)

Author: insspb

pyproject.toml

@@ -123,6 +123,8 @@ use_parentheses = true
 addopts = "--cov=flask_mongoengine --cov-config=setup.cfg"
 testpaths = ["tests"]
 filterwarnings = [
+  "error",
+  "ignore::ResourceWarning",
   "ignore::DeprecationWarning:flask_mongoengine",
   "ignore::DeprecationWarning:tests",
   ]

tests/test_forms_v2.py

@@ -2,6 +2,7 @@
 import json
 
 import pytest
+from bson.json_util import RELAXED_JSON_OPTIONS
 from werkzeug.datastructures import MultiDict
 
 wtforms = pytest.importorskip("wtforms")
@@ -160,7 +161,7 @@ class AllFieldsModel(db.Document):
     form.save()
 
     obj = AllFieldsModel.objects.get(id=form.instance.pk)
-    object_dict = json.loads(obj.to_json())
+    object_dict = json.loads(obj.to_json(json_options=RELAXED_JSON_OPTIONS))
     object_dict.pop("_id")
 
     assert object_dict == {

tests/test_json_app.py

@@ -18,7 +18,7 @@ def as_pymongo():
     @app.route("/aggregate")
     def aggregate():
         return flask.jsonify(
-            result=Todo.objects().aggregate({"$match": {"title": {"$ne": "lksdjh"}}})
+            result=Todo.objects().aggregate([{"$match": {"title": {"$ne": "lksdjh"}}}])
         )
 
     @app.route("/add", methods=["POST"])