Skip to content
Snippets Groups Projects
Commit 510b3b94 authored by Florian Schröder's avatar Florian Schröder
Browse files

Add ManageEnv class and refactor stop_env function

A new class called ManageEnv was introduced, encapsulating manager_id, env_id, and a reason. The asynchronous function stop_env in the game_server.py file was refactored to take an instance of this class as an argument instead of three separate arguments. Additionally, a trailing slash was added to the stop_env request URL in the overcooked_gui.py file.
parent 7d9a5b37
No related branches found
No related tags found
No related merge requests found
Pipeline #45660 passed
......@@ -675,6 +675,12 @@ class CreateEnvironmentConfig(BaseModel):
seed: int
class ManageEnv(BaseModel):
manager_id: str
env_id: str
reason: str
class AdditionalPlayer(BaseModel):
manager_id: str
env_id: str
......@@ -697,8 +703,10 @@ async def additional_player(creation: AdditionalPlayer) -> dict[str, PlayerInfo]
@app.post("/manage/stop_env/")
async def stop_env(manager_id: str, env_id: str, reason: str) -> str:
accept = environment_handler.stop_env(manager_id, env_id, reason)
async def stop_env(manage_env: ManageEnv) -> str:
accept = environment_handler.stop_env(
manage_env.manager_id, manage_env.env_id, manage_env.reason
)
if accept:
raise HTTPException(
status_code=403 if accept == 1 else 409,
......
......@@ -860,7 +860,7 @@ class PyGameGUI:
def finished_button_press(self):
requests.post(
f"{self.request_url}/manage/stop_env",
f"{self.request_url}/manage/stop_env/",
json={
"manager_id": self.manager_id,
"env_id": self.current_env_id,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment