Autolab
Autolab copied to clipboard
Maintenance of autolab.rake
Current autolab.rake file has several issues
- It populates fake submissions (
load_submissions_for) with old submission path- Should update
s.filenameto"#{i.to_s}_#{a.handin_filename}"andsubmission_pathtoFile.join(assessment_handin_dir, user.email, s.filename)
- Should update
-
delete_course(called bydepopulatetask) uses removed methodFile.exists?, should useFile.exist?instead -
populatetask isn't idempotent, will fail on future runs due to e.g. existing users - Minor style issues, e.g.
if User.where( ... ).firstcould be replaced withif User.exists?( ... )