TEST: Add module assert_helper #17
Closed
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
Implement re-usable code used for asserting tests into module assert_helper
Submission date is now set when the task files are uploaded. Works for needs help and ready to mark.
* Use Latex for Portfolio generation. * Include details on task - outcome alignment * Add in comments * Add in times assessed
Existing stats incorrectly included counts of task stats that included withdrawn students as not started tasks.
* Create a new file partial to add submitted files into latex * Include student uploaded files in portfolio
* Ensure students in units without LOs can add files to portfolios * Fix alerts from showing * Include tutorial code in filters
Ensure that file renaming keeps checking for 000.type... rather than 000-type format inputs. (should not be needed)
Accidental character deletion... Should have been .nil?
Ensure that each project can only have one task for each task definition.
Rails validations do not guarantee safety from duplicate row creation which is impacting on task creation. The validations remain in the rails model, but now there is also the DB index to ensure that a Task has a unique `project_id`,`task_definition_id` combination.
Make it easier in some cases to identify tasks/projects from the logger info.
Log rotation in production is done using logrotate, and should not be done by the rails logger. This fix ensures that the Doubtfire logger uses the default rotate settings that should write all log messages to one file.
* Add additional details to task process fails. * Use student.username (code is invalid)
* Remove the old portfolio creation with pdftk * Use new latex portfolio creation with latex * Switch to include all pages from pdfs until length can be checked
ghost
closed this
This pull request was closed.
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.
Implement re-usable code used for asserting tests into module assert_helper