Open damianhxy opened 6 months ago
Current autolab.rake file has several issues
autolab.rake
load_submissions_for
s.filename
"#{i.to_s}_#{a.handin_filename}"
submission_path
File.join(assessment_handin_dir, user.email, s.filename)
delete_course
depopulate
File.exists?
File.exist?
populate
if User.where( ... ).first
if User.exists?( ... )
Current
autolab.rake
file has several issuesload_submissions_for
) with old submission paths.filename
to"#{i.to_s}_#{a.handin_filename}"
andsubmission_path
toFile.join(assessment_handin_dir, user.email, s.filename)
delete_course
(called bydepopulate
task) uses removed methodFile.exists?
, should useFile.exist?
insteadpopulate
task isn't idempotent, will fail on future runs due to e.g. existing usersif User.where( ... ).first
could be replaced withif User.exists?( ... )