fixes pylint

Author: pcrespov

services/web/server/src/simcore_service_webserver/application_settings.py

@@ -411,7 +411,7 @@ def _enable_only_if_dev_features_allowed(cls, v, info: ValidationInfo):
 
         return (
             None
-            if info.field_name and is_nullable(cls.model_fields[info.field_name])
+            if info.field_name and is_nullable(dict(cls.model_fields)[info.field_name])
             else False
         )
 

services/web/server/src/simcore_service_webserver/application_settings_utils.py

@@ -202,7 +202,7 @@ def convert_to_environ_vars(  # noqa: C901, PLR0915, PLR0912
     def _set_if_disabled(field_name, section):
         # Assumes that by default is enabled
         enabled = section.get("enabled", True)
-        field = ApplicationSettings.model_fields[field_name]
+        field = dict(ApplicationSettings.model_fields)[field_name]
         if not enabled:
             envs[field_name] = "null" if is_nullable(field) else "0"
         elif get_type(field) == bool:

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

@@ -89,7 +89,7 @@ def test_auto_compute_gravatar__deprecated(fake_profile_get: MyProfileGet):
 
     assert (
         "gravatar_id" not in data
-    ), f"{MyProfileGet.model_fields['gravatar_id'].deprecated=}"
+    ), f"{dict(MyProfileGet.model_fields)['gravatar_id'].deprecated=}"
     assert data["id"] == profile.id
     assert data["first_name"] == profile.first_name
     assert data["last_name"] == profile.last_name

services/web/server/tests/unit/with_dbs/04/licenses/test_licensed_items_repository.py

@@ -114,18 +114,16 @@ async def test_licensed_items_db_trash(
         )
         licensed_item_ids.append(licensed_item_db.licensed_item_id)
 
-    licensed_item_id1, licensed_item_id2 = licensed_item_ids
-
     # Trash one licensed item
     trashing_at = arrow.now().datetime
     trashed_item = await _licensed_items_repository.update(
         client.app,
-        licensed_item_id=licensed_item_id1,
+        licensed_item_id=licensed_item_ids[0],
         product_name=osparc_product_name,
         updates=LicensedItemUpdateDB(trash=True),
     )
 
-    assert trashed_item.licensed_item_id == licensed_item_id1
+    assert trashed_item.licensed_item_id == licensed_item_ids[0]
     assert trashed_item.trashed
     assert trashing_at < trashed_item.trashed
     assert trashed_item.trashed < arrow.now().datetime
@@ -152,7 +150,7 @@ async def test_licensed_items_db_trash(
         trashed="exclude",
     )
     assert total_count == 1
-    assert items[0].licensed_item_id == licensed_item_id2
+    assert items[0].licensed_item_id == licensed_item_ids[1]
     assert items[0].trashed is None
 
     # List with filter_trashed all

services/web/server/tests/unit/with_dbs/04/wallets/payments/test_payments.py

@@ -362,7 +362,7 @@ async def test_payment_not_found(
 
 
 def test_payment_transaction_state_and_literals_are_in_sync():
-    state_literals = PaymentTransaction.model_fields["state"].annotation
+    state_literals = dict(PaymentTransaction.model_fields)["state"].annotation
     assert (
         TypeAdapter(list[state_literals]).validate_python(
             [f"{s}" for s in PaymentTransactionState]