test: add MSW handler and non-budget-team test for pending requisitions

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: josbell

backend/ops_api/Pipfile.lock

@@ -93,14 +93,25 @@ def test_get_pending_requisitions_excludes_completed(
 
 
 def test_get_pending_requisitions_filters_by_budget_team_role(client, test_budget_team_requisition_step, loaded_db):
-    """Only BUDGET_TEAM role users get results."""
-    # Non-budget-team user should get empty list
-    # Note: This test would need proper auth setup for a non-budget-team user
-    # For now, just verify the endpoint requires auth
+    """Unauthenticated users get 401."""
     response = client.get("/api/v1/procurement-tracker-steps/pending-requisitions/")
     assert response.status_code == 401  # Unauthenticated
 
 
+def test_get_pending_requisitions_non_budget_team_gets_empty_list(
+    basic_user_auth_client, test_budget_team_requisition_step, loaded_db
+):
+    """Authenticated non-budget-team users get empty list, not 401."""
+    # Make request as basic user (not budget team)
+    response = basic_user_auth_client.get("/api/v1/procurement-tracker-steps/pending-requisitions/")
+    assert response.status_code == 200
+
+    data = response.json
+    assert isinstance(data, list)
+    # Non-budget-team user should see empty list
+    assert len(data) == 0
+
+
 def test_get_pending_requisitions_includes_agreement_data(
     budget_team_auth_client, test_budget_team_requisition_step, loaded_db
 ):

backend/ops_api/tests/ops/procurement_tracker/test_budget_team_requisition_review_card.py

@@ -10,6 +10,12 @@ import { FontAwesomeIcon } from "@fortawesome/react-fontawesome";
 /**
  * BudgetTeamRequisitionReviewCard component for displaying pending budget team requisition reviews
  * Appears after Division Director approval, prompts budget team to enter requisition details
+ *
+ * NOTE: This component does NOT use the shared ReviewCard base component because it serves a different
+ * purpose than change request cards. ReviewCard is designed for inline approve/reject actions on
+ * change requests, while this card (like PreAwardReviewCard) is an informational card that links
+ * to a detail page. The card structure is intentionally customized to show budget-specific metadata.
+ *
  * @component
  * @param {Object} props - Properties passed to component
  * @param {number} props.agreementId - The ID of the agreement

frontend/src/components/ChangeRequests/BudgetTeamRequisitionReviewCard/BudgetTeamRequisitionReviewCard.jsx

@@ -10,6 +10,12 @@ import { FontAwesomeIcon } from "@fortawesome/react-fontawesome";
 
 /**
  * PreAwardReviewCard component for displaying pending pre-award approval requests
+ *
+ * NOTE: This component does NOT use the shared ReviewCard base component because it serves a different
+ * purpose than change request cards. ReviewCard is designed for inline approve/reject actions on
+ * change requests, while this card (like BudgetTeamRequisitionReviewCard) is an informational card
+ * that links to a detail/approval page.
+ *
  * @component
  * @param {Object} props - Properties passed to component
  * @param {number} props.agreementId - The ID of the agreement