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

Merge pull request #424 from dod-ccpo/bugfix-workspace-link

Browse Source 
Fix a bug that was crashing the requests index
This commit is contained in:
richard-dds 2018-11-01 16:12:59 -04:00 committed by GitHub
commit 64f6393678
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -60,7 +60,7 @@ class RequestsIndex(object):
def _workspace_link_for_request(self, request):
if request.is_approved:
return url_for(
"workspaces.workspace_projects", workspace_id=request.workspace_id
"workspaces.workspace_projects", workspace_id=request.workspace.id
)
else:
return None

3
tests/routes/test_requests_index.py
View File

@ -9,6 +9,8 @@ def test_action_required_mission_owner():
creator = UserFactory.create()
requests = RequestFactory.create_batch(5, creator=creator)
Requests.submit(requests[0])
Requests.approve_and_create_workspace(requests[1])
context = RequestsIndex(creator).execute()
assert context["requests"][0]["action_required"] == False
@ -18,6 +20,7 @@ def test_action_required_ccpo():
creator = UserFactory.create()
requests = RequestFactory.create_batch(5, creator=creator)
Requests.submit(requests[0])
Requests.approve_and_create_workspace(requests[1])
ccpo = UserFactory.from_atat_role("ccpo")
context = RequestsIndex(ccpo).execute()