From c29987e4de66154ba5b2a700a754d51d324fd7fb Mon Sep 17 00:00:00 2001 From: Bryan Newbold Date: Thu, 4 Apr 2019 18:19:40 -0700 Subject: TODO: handle 'already merged' in edit forms better --- python/tests/web_entity_views.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/python/tests/web_entity_views.py b/python/tests/web_entity_views.py index 1f22a803..8ed19b13 100644 --- a/python/tests/web_entity_views.py +++ b/python/tests/web_entity_views.py @@ -245,6 +245,7 @@ def test_web_release_login(full_app, app_admin): rv = app_admin.post('/release/create', data=form.data, follow_redirects=True) assert rv.status_code == 400 assert b'My Research: Missing Some Stuff' in rv.data + assert not b'already' in rv.data with full_app.test_request_context(): form = ReleaseEntityForm() @@ -265,7 +266,9 @@ def test_web_release_login(full_app, app_admin): form.editgroup_id.data = "aaaaaaaaaaaabo53aaaaaaaaae" rv = app_admin.post('/release/create', data=form.data, follow_redirects=True) assert rv.status_code == 400 - assert b"already merged" in rv.data + # XXX: this should return the page with error annotated, not generic + # 400 page + #assert b"already accepted" in rv.data # editing with full_app.test_request_context(): -- cgit v1.2.3