Skip to content

Commit b0bf0b2

Browse files
authored
Merge pull request doubtfire-lms#576 from b0ink/fix/skip-empty-overseer
fix: skip overseer if no active steps
2 parents 7a6a20e + d010ef6 commit b0bf0b2

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

app/models/overseer_assessment.rb

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ def self.create_for(task, test_submission)
3232

3333
return nil unless task.overseer_enabled? || test_submission
3434

35+
active_overseer_steps = task.task_definition.overseer_steps.select(&:enabled)
36+
return nil if active_overseer_steps.empty?
37+
3538
docker_image_name_tag = task_definition.docker_image_name_tag || unit.docker_image_name_tag
3639
# assessment_resources_path = task_definition.task_assessment_resources
3740

0 commit comments

Comments
 (0)