Skip to content

Commit c10d7d0

Browse files
authored
⬆️♻️ Fix/test models (#6758)
1 parent f442e63 commit c10d7d0

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

packages/models-library/src/models_library/rest_base.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from pydantic import BaseModel, Extra
1+
from pydantic import BaseModel, ConfigDict
22

33

44
class RequestParameters(BaseModel):
@@ -15,5 +15,4 @@ def as_params(self, **export_options) -> dict[str, str]:
1515
class StrictRequestParameters(RequestParameters):
1616
"""Use a base class for context, path and query parameters"""
1717

18-
class Config:
19-
extra = Extra.forbid # strict
18+
model_config = ConfigDict(extra="forbid")

packages/models-library/src/models_library/rest_ordering.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ class _OrderBy(OrderBy):
7272
}
7373
]
7474
},
75-
validate_assignment=True, # Necessary to run _check_ordering_field_and_map in defaults and assignments
75+
# Necessary to run _check_ordering_field_and_map in defaults and assignments
76+
validate_assignment=True,
7677
validate_default=True,
7778
)
7879

@@ -91,9 +92,9 @@ def _check_ordering_field_and_map(cls, v):
9192

9293
assert "json_schema_extra" in _OrderBy.model_config # nosec
9394
assert isinstance(_OrderBy.model_config["json_schema_extra"], dict) # nosec
94-
assert isinstance(
95+
assert isinstance( # nosec
9596
_OrderBy.model_config["json_schema_extra"]["examples"], list
96-
) # nosec
97+
)
9798
order_by_example = _OrderBy.model_config["json_schema_extra"]["examples"][0]
9899
order_by_example_json = json_dumps(order_by_example)
99100
assert _OrderBy.model_validate(order_by_example), "Example is invalid" # nosec

packages/models-library/tests/test_rest_ordering.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ def test_ordering_query_model_class__fails_with_invalid_direction():
106106

107107
error = err_info.value.errors()[0]
108108

109-
assert error["type"] == "type_error.enum"
109+
assert error["type"] == "enum"
110110
assert error["loc"] == ("order_by", "direction")
111111

112112

@@ -137,7 +137,7 @@ def test_ordering_query_model_class__defaults():
137137

138138
error = err_info.value.errors()[0]
139139
assert error["loc"] == ("order_by", "field")
140-
assert error["type"] == "value_error.missing"
140+
assert error["type"] == "missing"
141141

142142

143143
def test_ordering_query_model_with_map():

0 commit comments

Comments
 (0)