adam/meal-order-manager
1
0
Fork 0
mirror of https://github.com/Sea-Haven-Industries/meal-order-manager.git synced 2026-10-07 16:18:54 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

style(python): apply Ruff formatting

Browse source
This commit is contained in:
Adam Moussa 2026-08-03 14:24:54 -04:00
parent 308f46eafd
commit 7e4bccc804
No known key found for this signature in database
3 changed files with 15 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
functions/submit_order/handler.py
View file

@ -220,9 +220,7 @@ def handle_publish_settings():
return response( return response(
200, 200,
{ {
"bulk_discount_percent": float( "bulk_discount_percent": float(settings.get("bulk_discount_percent", 0)),
settings.get("bulk_discount_percent", 0)
),
"company_subsidy_percent": float( "company_subsidy_percent": float(
settings.get("company_subsidy_percent", 0) settings.get("company_subsidy_percent", 0)
), ),
@ -236,14 +234,14 @@ def handle_publish_menu(event):
body = json.loads(event.get("body", "{}")) body = json.loads(event.get("body", "{}"))
except json.JSONDecodeError: except json.JSONDecodeError:
return response(400, {"error": "Invalid JSON"}) return response(400, {"error": "Invalid JSON"})
if not isinstance(body, dict): if not isinstance(body, dict):
return response(400, {"error": "Request body must be a JSON object"}) return response(400, {"error": "Request body must be a JSON object"})
meals = body.get("meals") meals = body.get("meals")
if not isinstance(meals, list) or not meals: if not isinstance(meals, list) or not meals:
return response(400, {"error": "At least one meal is required"}) return response(400, {"error": "At least one meal is required"})
for meal in meals: for meal in meals:
price = meal.get("price") if isinstance(meal, dict) else None price = meal.get("price") if isinstance(meal, dict) else None
if ( if (
@ -256,7 +254,7 @@ def handle_publish_menu(event):
return response( return response(
400, {"error": "Each meal requires a name and non-negative price"} 400, {"error": "Each meal requires a name and non-negative price"}
) )
week = current_week() week = current_week()
menu = { menu = {
"scraped_at": body.get("scraped_at"), "scraped_at": body.get("scraped_at"),

14
scripts/upload_menu.py
View file

@ -39,7 +39,7 @@ def signed_request(
SigV4Auth(credentials.get_frozen_credentials(), "execute-api", region).add_auth( SigV4Auth(credentials.get_frozen_credentials(), "execute-api", region).add_auth(
request request
) )
prepared = request.prepare() prepared = request.prepare()
http_request = urllib.request.Request( http_request = urllib.request.Request(
prepared.url, prepared.url,
@ -52,13 +52,13 @@ def signed_request(
return json.loads(response.read()) return json.loads(response.read())
except urllib.error.HTTPError as exc: except urllib.error.HTTPError as exc:
detail = exc.read().decode(errors="replace") detail = exc.read().decode(errors="replace")
raise RuntimeError(f"Publication API returned HTTP {exc.code}: {detail}") from exc raise RuntimeError(
f"Publication API returned HTTP {exc.code}: {detail}"
) from exc
def get_settings(api_url: str, region: str) -> dict: def get_settings(api_url: str, region: str) -> dict:
return signed_request( return signed_request(api_url, "/api/publish/settings", method="GET", region=region)
api_url, "/api/publish/settings", method="GET", region=region
)
def publish_menu(api_url: str, region: str) -> dict: def publish_menu(api_url: str, region: str) -> dict:
@ -81,9 +81,7 @@ def main():
parser = argparse.ArgumentParser() parser = argparse.ArgumentParser()
parser.add_argument("action", choices=("settings", "publish")) parser.add_argument("action", choices=("settings", "publish"))
parser.add_argument("--api-url", required=True) parser.add_argument("--api-url", required=True)
parser.add_argument( parser.add_argument("--region", default=os.environ.get("AWS_REGION", "us-east-1"))
"--region", default=os.environ.get("AWS_REGION", "us-east-1")
)
args = parser.parse_args() args = parser.parse_args()
try: try:

8
tests/test_generate_form.py
View file

@ -189,16 +189,16 @@ class TestGenerateFormStructural:
if route == "/api/publish/settings" if route == "/api/publish/settings"
else " AdminOrders:" else " AdminOrders:"
) )
route_config = template.split(f"Path: {route}", 1)[1].split( route_config = template.split(f"Path: {route}", 1)[1].split(next_event, 1)[
next_event, 1 0
)[0] ]
assert "Authorizer: AWS_IAM" in route_config assert "Authorizer: AWS_IAM" in route_config
assert route in script assert route in script
assert "scripts/upload_menu.py settings" in workflow assert "scripts/upload_menu.py settings" in workflow
assert "scripts/upload_menu.py publish" in workflow assert "scripts/upload_menu.py publish" in workflow
assert "aws dynamodb" not in workflow assert "aws dynamodb" not in workflow
assert "boto3.resource(\"dynamodb\")" not in script assert 'boto3.resource("dynamodb")' not in script
def test_local_and_google_render(self): def test_local_and_google_render(self):
local = _render(google=False) local = _render(google=False)