TestURLs coverage for project app #2229
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds test coverage for most of the URLs in the
project
app.See the previous pull requests #1695, #1922, #2108 for background.
With this change, the bulk of site functionality should now be included in the TestURLs framework. A few small apps are left to add:
oauth
(should be easy)sso
(should be easy but I'm not sure this is enabled in test configs)training
(awaiting pull create training basics --> implement take training feature and resume training feature for user #1951)And a number of URLs across the site are skipped due to a lack of fixture data:
console
edit_submission
copyedit_submission
awaiting_authors
publish_submission
event_agreement_detail
event_agreement_delete
event_agreement_new_version
project
serve_document
published_project_request_access
search
redirect_challenge_project