pk/atst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Allow drafts to be saved without pe_number

Browse Source
This commit is contained in:
richard-dds 2018-11-05 11:44:59 -05:00
parent 5d4f867342
commit 6a39ce2a52
2 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
atst/routes/requests/financial_verification.py
View File

@ -173,7 +173,7 @@ class SaveFinancialVerificationDraft(FinancialVerificationBase):
if not form.validate_draft(): if not form.validate_draft():
self._raise(form) self._raise(form)
if not self.pe_validator.validate(self.request, form.pe_id): if form.pe_id.data and not self.pe_validator.validate(self.request, form.pe_id):
valid = False valid = False
if form.task_order.number.data and not self.task_order_validator.validate( if form.task_order.number.data and not self.task_order_validator.validate(

14
tests/routes/test_financial_verification.py
View File

@ -111,6 +111,20 @@ def test_update_fv_invalid_task_order_number(fv_data):
update_fv.execute() update_fv.execute()
def test_draft_without_pe_id(fv_data):
request = RequestFactory.create()
user = UserFactory.create()
data = {"request-uii_ids": "1234"}
assert SaveFinancialVerificationDraft(
PENumberValidator(),
TaskOrderNumberValidator(),
user,
request,
data,
is_extended=False,
).execute()
def test_update_fv_extended(fv_data, e_fv_data): def test_update_fv_extended(fv_data, e_fv_data):
request = RequestFactory.create() request = RequestFactory.create()
user = UserFactory.create() user = UserFactory.create()