Omit null optional fields from Modal worker payloads (#469)

Author: MaxGhenis

projects/policyengine-api-simulation/src/modal/gateway/endpoints.py

@@ -100,6 +100,7 @@ def _build_budget_window_parent_payload(
     payload = request.model_dump(
         exclude={"telemetry"},
         mode="json",
+        exclude_none=True,
     )
     payload["version"] = resolved_version
     if request.telemetry is not None:
@@ -162,6 +163,7 @@ async def submit_simulation(request: SimulationRequest):
     payload = request.model_dump(
         exclude={"version", "telemetry"},
         mode="json",
+        exclude_none=True,
     )
     run_id = request.telemetry.run_id if request.telemetry else None
     if request.telemetry is not None:

projects/policyengine-api-simulation/tests/gateway/test_endpoints.py

@@ -167,6 +167,8 @@ def test__given_no_data_value__then_routes_to_run_simulation(
             "policyengine-simulation-us1-500-0-uk2-66-0",
             "run_simulation",
         )
+        assert "time_period" not in mock_modal["func"].last_payload
+        assert "data_version" not in mock_modal["func"].last_payload
 
     def test__given_submission__then_returns_job_id_and_poll_url(
         self, mock_modal, client: TestClient