Ensures technology roadmap status transitions go through the proper review path with reviewer identity.
status
update_roadmap()
review_roadmap()
reviewed_by
update_roadmap(id, status=...)
ValueError
Test verifying update_roadmap rejects status transitions while review_roadmap works normally.
- Remove status from update_roadmap() allowed fields - Status changes must go through review_roadmap() with reviewed_by - Add test verifying update_roadmap rejects status transitions
Summary
Ensures technology roadmap status transitions go through the proper review path with reviewer identity.
Changes
statusfromupdate_roadmap()allowed fieldsreview_roadmap()which requiresreviewed_byupdate_roadmap(id, status=...)now raisesValueErrorTests
Test verifying update_roadmap rejects status transitions while review_roadmap works normally.