Skip to content

Commit efefd0a

Browse files
upgrade deprecated
1 parent 6eba301 commit efefd0a

File tree

6 files changed

+15
-16
lines changed

6 files changed

+15
-16
lines changed

services/web/server/src/simcore_service_webserver/dynamic_scheduler/settings.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
from typing import Final
22

33
from aiohttp import web
4-
from pydantic import Field, NonNegativeInt
4+
from pydantic import AliasChoices, Field, NonNegativeInt
55
from settings_library.base import BaseCustomSettings
66
from settings_library.utils_service import MixinServiceSettings
77

@@ -23,10 +23,10 @@ class DynamicSchedulerSettings(BaseCustomSettings, MixinServiceSettings):
2323
"- director-v* requests save_state and uses a 01:00:00 timeout"
2424
"The +10 seconds is used to make sure the director replies"
2525
),
26-
envs=[
26+
validation_alias=AliasChoices(
2727
"DIRECTOR_V2_STOP_SERVICE_TIMEOUT",
2828
"DYNAMIC_SCHEDULER_STOP_SERVICE_TIMEOUT",
29-
],
29+
),
3030
)
3131

3232

services/web/server/src/simcore_service_webserver/users/_notifications_handlers.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ async def mark_notification_as_read(request: web.Request) -> web.Response:
106106
# NOTE: only the user's notifications can be patched
107107
key = get_notification_key(req_ctx.user_id)
108108
all_user_notifications: list[UserNotification] = [
109-
UserNotification.parse_raw(x)
109+
UserNotification.model_validate_json(x)
110110
for x in await handle_redis_returns_union_types(redis_client.lrange(key, 0, -1))
111111
]
112112
for k, user_notification in enumerate(all_user_notifications):
@@ -130,7 +130,9 @@ async def list_user_permissions(request: web.Request) -> web.Response:
130130
)
131131
return envelope_json_response(
132132
[
133-
PermissionGet.construct(_fields_set=p.__fields_set__, **p.model_dump())
133+
PermissionGet.model_construct(
134+
_fields_set=p.model_fields_set, **p.model_dump()
135+
)
134136
for p in list_permissions
135137
]
136138
)

services/web/server/src/simcore_service_webserver/users/_preferences_models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ class BillingCenterUsageColumnOrderFrontendUserPreference(FrontendUserPreference
132132
]
133133

134134
_PREFERENCE_NAME_TO_IDENTIFIER_MAPPING: dict[PreferenceName, PreferenceIdentifier] = {
135-
p.get_preference_name(): p.__fields__["preference_identifier"].default
135+
p.get_preference_name(): p.model_fields["preference_identifier"].default
136136
for p in ALL_FRONTEND_PREFERENCES
137137
}
138138
_PREFERENCE_IDENTIFIER_TO_NAME_MAPPING: dict[PreferenceIdentifier, PreferenceName] = {

services/web/server/src/simcore_service_webserver/users/_schemas.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
from models_library.api_schemas_webserver._base import InputSchema, OutputSchema
1313
from models_library.emails import LowerCaseEmailStr
1414
from models_library.products import ProductName
15-
from pydantic import ConfigDict, field_validator, model_validator, Field, field_validator
15+
from pydantic import ConfigDict, Field, field_validator, model_validator
1616
from simcore_postgres_database.models.users import UserStatus
1717

1818

@@ -74,10 +74,7 @@ class PreUserProfile(InputSchema):
7474
description="Keeps extra information provided in the request form. At most MAX_NUM_EXTRAS fields",
7575
)
7676

77-
model_config = ConfigDict(
78-
str_strip_whitespace=True,
79-
str_max_length=200
80-
)
77+
model_config = ConfigDict(str_strip_whitespace=True, str_max_length=200)
8178

8279
@model_validator(mode="before")
8380
@classmethod
@@ -93,8 +90,8 @@ def _preprocess_aliases_and_extras(cls, values):
9390
# collect extras
9491
extra_fields = {}
9592
field_names_and_aliases = (
96-
set(cls.__fields__.keys())
97-
| {f.alias for f in cls.__fields__.values() if f.alias}
93+
set(cls.model_fields.keys())
94+
| {f.alias for f in cls.model_fields.values() if f.alias}
9895
| set(alias_by_priority)
9996
)
10097
for key, value in values.items():

services/web/server/tests/unit/isolated/test_login_settings.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,6 @@ def test_smtp_settings(mock_env_devel_environment: dict[str, Any]):
137137
def test_product_login_settings_in_plugin_settings():
138138
# pylint: disable=no-member
139139
customizable_attributes = set(ProductLoginSettingsDict.__annotations__.keys())
140-
settings_atrributes = set(LoginSettingsForProduct.__fields__.keys())
140+
settings_atrributes = set(LoginSettingsForProduct.model_fields.keys())
141141

142142
assert customizable_attributes.issubset(settings_atrributes)

services/web/server/tests/unit/with_dbs/03/test_users.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ async def test_update_profile(
149149
data, _ = await assert_status(resp, status.HTTP_200_OK)
150150

151151
# This is a PUT! i.e. full replace of profile variable fields!
152-
assert data["first_name"] == ProfileUpdate.__fields__["first_name"].default
152+
assert data["first_name"] == ProfileUpdate.model_fields["first_name"].default
153153
assert data["last_name"] == "Foo"
154154
assert data["role"] == user_role.name
155155

@@ -381,7 +381,7 @@ def test_preuserprofile_parse_model_without_extras(
381381
account_request_form: dict[str, Any]
382382
):
383383
required = {
384-
f.alias or f.name for f in PreUserProfile.__fields__.values() if f.required
384+
f.alias or f.name for f in PreUserProfile.model_fields.values() if f.required
385385
}
386386
data = {k: account_request_form[k] for k in required}
387387
assert not PreUserProfile(**data).extras

0 commit comments

Comments
 (0)