Update schema to require budget field on /can-funding-budgets (#3336)

* feat: make budget required on post can funding budgets

* chore: remove service handling

* chore: remove unit test

backend/openapi.yml

@@ -3048,6 +3048,7 @@ components:
       required:
         - fiscal_year
         - can_id
+        - budget
     FundingBudget:
       description: Funding Budget Object
       type: object

backend/ops_api/ops/schemas/cans.py

@@ -112,7 +112,7 @@ class FundingBudgetSchema(Schema):
 class CreateUpdateFundingBudgetSchema(Schema):
     fiscal_year = fields.Integer(required=True)
     can_id = fields.Integer(required=True)
-    budget = fields.Float(load_default=None)
+    budget = fields.Float(required=True)
     notes = fields.String(load_default=None)
 
 

backend/ops_api/tests/ops/can/test_can_funding_budget.py

@@ -102,6 +102,15 @@ def test_service_create_funding_budget(loaded_db):
     loaded_db.commit()
 
 
+def test_funding_budget_post_400_missing_budget(budget_team_auth_client):
+    response = budget_team_auth_client.post(
+        "/api/v1/can-funding-budgets/", json={"can_id": 500, "fiscal_year": 2024, "notes": "This is a note"}
+    )
+
+    assert response.status_code == 400
+    assert response.json["budget"][0] == "Missing data for required field."
+
+
 # Testing updating CANs by PATCH
 @pytest.mark.usefixtures("app_ctx")
 def test_funding_budget_patch(budget_team_auth_client, mocker):