Automated URL route testing, URL redirects and some minor refactor tidy ups #621
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit adds a set of url routes that redirect from the old Dashboard to the new views. It also removes the url route for the coverage page.
Using the Django test framework this commit adds some basic automated testing of the Dashboard URL routes, contains some fixes to the view functions to properly return HTTP 404 errors, and updates the documentation (including a small fix for the file paths). A small fix was also added to pytest.ini.
This commit moves the file path constants from config.py into the ui settings module so that it is more consistent with Django's settings architecture.
After moving configuration constants into settings.py the config.py module was somewhat poorly named, this commit changes the name and refactors the other code to use the new name.
Bjwebb
approved these changes
Nov 7, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 PR mainly adds URL redirects for the previous
.htmlsite structure and includes automated URL route testing for both the new pages and the redirects. It also includes a few commits aimed at tidying up the code and documentation.