Requests pending CCPO apprval should not be given workspaces

atst/routes/requests/financial_verification.py

@@ -241,7 +241,7 @@ def update_financial_verification(request_id):
             extended=is_extended,
         )
 
-    if updated_request.is_pending_ccpo_approval:
+    if updated_request.is_approved:
         workspace = Requests.approve_and_create_workspace(updated_request)
         return redirect(
             url_for(

tests/routes/test_financial_verification.py

@@ -10,20 +10,16 @@ from atst.routes.requests.financial_verification import (
     SaveFinancialVerificationDraft,
 )
 
-from tests.mocks import MOCK_REQUEST, MOCK_USER, MOCK_VALID_PE_ID
+from tests.mocks import MOCK_VALID_PE_ID
-from tests.factories import (
+from tests.factories import RequestFactory, UserFactory
-    PENumberFactory,
-    RequestFactory,
-    UserFactory,
-    RequestStatusEventFactory,
-    RequestReviewFactory,
-)
 from atst.forms.exceptions import FormValidationError
 from atst.domain.requests.financial_verification import (
     PENumberValidator,
     TaskOrderNumberValidator,
 )
 from atst.utils import pick
+from atst.models.request_status_event import RequestStatus
+from atst.domain.requests.query import RequestsQuery
 
 
 @pytest.fixture
@@ -347,3 +343,18 @@ def test_manual_task_order_triggers_extended_form(
         follow_redirects=False,
     )
     assert "extended" in response.headers["Location"]
+
+
+def test_manual_to_does_not_trigger_approval(client, user_session, fv_data, e_fv_data):
+    user = UserFactory.create()
+    request = RequestFactory.create(creator=user)
+    data = {**fv_data, **e_fv_data, "request-pe_id": "0101228N"}
+    user_session(user)
+    client.post(
+        url_for("requests.financial_verification", request_id=request.id, extended=True),
+        data=data,
+        follow_redirects=True
+    )
+
+    updated_request = RequestsQuery.get(request.id)
+    assert updated_request.status != RequestStatus.APPROVED