fixup some server tests

Opentrons · Sep 9, 2024 · 666b3ff · 666b3ff
1 parent 9159f4a
commit 666b3ff
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 11 deletions.
diff --git a/robot-server/tests/service/json_api/test_request.py b/robot-server/tests/service/json_api/test_request.py
@@ -30,17 +30,23 @@ def test_attributes_as_item_model_empty_dict():
         {
             "loc": ("data", "name"),
             "msg": "Field required",
-            "type": "value_error.missing",
+            "type": "missing",
+            "input": {},
+            "url": "https://errors.pydantic.dev/2.9/v/missing",
         },
         {
             "loc": ("data", "quantity"),
             "msg": "Field required",
-            "type": "value_error.missing",
+            "type": "missing",
+            "input": {},
+            "url": "https://errors.pydantic.dev/2.9/v/missing",
         },
         {
             "loc": ("data", "price"),
             "msg": "Field required",
-            "type": "value_error.missing",
+            "type": "missing",
+            "input": {},
+            "url": "https://errors.pydantic.dev/2.9/v/missing",
         },
     ]
 
@@ -54,8 +60,10 @@ def test_attributes_required():
     assert e.value.errors() == [
         {
             "loc": ("data",),
-            "msg": "none is not an allowed value",
-            "type": "type_error.none.not_allowed",
+            "msg": "Input should be a valid dictionary",
+            "input": None,
+            "url": "https://errors.pydantic.dev/2.9/v/dict_type",
+            "type": "dict_type",
         },
     ]
 
@@ -69,8 +77,10 @@ def test_data_required():
     assert e.value.errors() == [
         {
             "loc": ("data",),
-            "msg": "none is not an allowed value",
-            "type": "type_error.none.not_allowed",
+            "msg": "Input should be a valid dictionary",
+            "input": None,
+            "url": "https://errors.pydantic.dev/2.9/v/dict_type",
+            "type": "dict_type",
         },
     ]
 

diff --git a/robot-server/tests/service/json_api/test_resource_links.py b/robot-server/tests/service/json_api/test_resource_links.py
@@ -27,5 +27,11 @@ def test_must_be_self_key_with_string_value():
     with raises(ValidationError) as e:
         ThingWithLink.model_validate(invalid_structure_to_validate)
     assert e.value.errors() == [
-        {"loc": ("links",), "msg": "Field required", "type": "value_error.missing"}
+        {
+            "loc": ("links",),
+            "msg": "Field required",
+            "type": "missing",
+            "input": {"invalid": {"key": "value"}},
+            "url": "https://errors.pydantic.dev/2.9/v/missing",
+        }
     ]
diff --git a/robot-server/tests/service/legacy/models/test_control.py b/robot-server/tests/service/legacy/models/test_control.py
@@ -11,12 +11,12 @@ def test_robot_home_target():
 
 
 def test_robot_move_target_points_too_few():
-    with pytest.raises(ValueError, match="ensure this value has at least 3 items"):
+    with pytest.raises(ValueError, match="List should have at least 3 items"):
         control.RobotMoveTarget(target=control.MotionTarget.pipette, point=[1, 2])
 
 
 def test_robot_move_target_points_too_many():
-    with pytest.raises(ValueError, match="ensure this value has at most 3 items"):
+    with pytest.raises(ValueError, match="List should have at most 3 items"):
         control.RobotMoveTarget(target=control.MotionTarget.pipette, point=[1, 2, 3, 4])
 
 

diff --git a/robot-server/tests/service/legacy/routers/test_settings.py b/robot-server/tests/service/legacy/routers/test_settings.py
@@ -598,7 +598,7 @@ def test_reset_invalid_option(
     assert resp.status_code == 422
     body = resp.json()
     assert "message" in body
-    assert "not a valid enumeration member" in body["message"]
+    assert "Input should be" in body["message"]
 
 
 @pytest.fixture()