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

Fix a bug that was crashing the requests index

Browse Source
This commit is contained in:
richard-dds
2018-11-01 15:51:21 -04:00
parent 9c22a57780
commit ff0c4ffb69
2 changed files with 4 additions and 1 deletions
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): def _workspace_link_for_request(self, request):
if request.is_approved: if request.is_approved:
return url_for( return url_for(
"workspaces.workspace_projects", workspace_id=request.workspace_id "workspaces.workspace_projects", workspace_id=request.workspace.id
) )
else: else:
return None return None

3
tests/routes/test_requests_index.py
View File

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