UPD use enum for patch request action types

api/patch_request.py

@@ -8,7 +8,7 @@ from starlette import status
 
 from api import db, get_conn
 from model.mail_requests import APIPatchRequestAction, PatchRequestFormRequest, InformalPatchRequestFormRequest, \
-    PatchRequestAction
+    PatchRequestAction, PatchActionType
 from model.settings import settings
 from model.wapi.cntl import APIToken, Mgr
 from model.wapi.nd import Port, Module, Room, Bldg, Site
@@ -131,7 +131,7 @@ def validate_patch_request_actions(conn, user, actions: list[APIPatchRequestActi
                 {'idx': 'bcd_list', 'name': 'nd.bcd.list', 'old': {'name': action.bcd_name}},
             ], user=user.login_name)
             internal_action.bcd = Port(**bcd_result['bcd_list'][0])
-        elif action.action.lower() == 'patch':
+        elif action.action == PatchActionType.PATCH:
             raise HTTPException(
                 status_code=status.HTTP_400_BAD_REQUEST,
                 detail='BCD is required for patch action'

model/mail_requests.py

 from typing import Optional
 
 from pydantic import BaseModel, Field
+from enum import Enum
 
 from model.wapi.nd import BCD, Bldg, Module, Room, Port, Site
 
 
+class PatchActionType(Enum):
+    PATCH = 'patch'
+    UNPATCH = 'unpatch'
+    CHANGE_INSERT = 'change_insert'
+
+
 class APIPatchRequestAction(BaseModel):
-    action: str = Field()
+    action: PatchActionType = Field()
     bcd_name: Optional[str] = Field(default=None)
     bcd_tagged: Optional[bool] = Field(default=False)
     building_gpk: str = Field()