diff --git a/atst/routes/requests/financial_verification.py b/atst/routes/requests/financial_verification.py index cd2caf0f..67008734 100644 --- a/atst/routes/requests/financial_verification.py +++ b/atst/routes/requests/financial_verification.py @@ -88,20 +88,6 @@ class FinancialVerificationBase(object): else: return None - def _apply_pe_number_error(self, field): - suggestion = self.pe_validator.suggest_pe_id(field.data) - error_str = ( - "We couldn't find that PE number. {}" - "If you have double checked it you can submit anyway. " - "Your request will need to go through a manual review." - ).format('Did you mean "{}"? '.format(suggestion) if suggestion else "") - field.errors += (error_str,) - field.errors = list(field.errors) - - def _apply_task_order_number_error(self, field): - field.errors += ("Task Order number not found",) - field.errors = list(field.errors) - def _raise(self, form): form.reset() raise FormValidationError(form) @@ -191,7 +177,9 @@ class SaveFinancialVerificationDraft(FinancialVerificationBase): if not self.pe_validator.validate(self.request, form.pe_id): valid = False - if form.task_order.number.data and not self.task_order_validator.validate(form.task_order.number): + if form.task_order.number.data and not self.task_order_validator.validate( + form.task_order.number + ): valid = False attachment = self._process_attachment(self.is_extended, form) diff --git a/tests/routes/test_financial_verification.py b/tests/routes/test_financial_verification.py index 1a167ba3..72be9d82 100644 --- a/tests/routes/test_financial_verification.py +++ b/tests/routes/test_financial_verification.py @@ -328,7 +328,9 @@ def test_get_fv_form_route(client, user_session, fv_data): assert response.status_code == 200 -def test_manual_task_order_triggers_extended_form(client, user_session, fv_data, e_fv_data): +def test_manual_task_order_triggers_extended_form( + client, user_session, fv_data, e_fv_data +): user = UserFactory.create() request = RequestFactory.create(creator=user)