keep method public

Author: Ben Lerner

app/models/graders/sandbox_grader.rb

@@ -83,28 +83,8 @@ def to_s
     end
   end
 
-  protected
-
-  def get_grading_script(sub)
-    JSON.load(File.open(self.upload.extracted_files.to_h {|v| [v[:path], v[:full_path]]}["grading_script.json"]))
-  end
-
-  def generate_files_hash(sub)
-    files = {
-      submission: {
-        url: sub.upload.url,
-        mime_type: sub.upload.read_metadata[:mimetype],
-        should_replace_paths: false
-      },
-      grader: {
-        url: self.upload.url,
-        mime_type: self.upload.read_metadata[:mimetype],
-        should_replace_paths: false
-      }
-    }
-  end
-
-  def postprocess_orca_response(grade, response)
+    def postprocess_orca_response(grade, response)
+    Audit.log("In SandboxGrader(#{self.id}).postprocess_orca_response(#{grade.id}, #{response.inspect})"
     sub = grade.submission
     prefix = "Assignment #{assignment.id}, submission #{sub.id}"
     if response['errors'].present?
@@ -163,13 +143,36 @@ def postprocess_orca_response(grade, response)
             InlineComment.import ics
           end
         when "plaintext"
+          Audit.log("Not yet implemented: plaintext response for SandboxGrader(#{self.id}).postprocess_orca_response")
         end
       rescue Exception => e
         Audit.log("#{prefix}: #{self.response_type} error: #{e}")
         record_compile_error(sub, grade)
       end
     end
   end
+
+  protected
+
+  def get_grading_script(sub)
+    JSON.load(File.open(self.upload.extracted_files.to_h {|v| [v[:path], v[:full_path]]}["grading_script.json"]))
+  end
+
+  def generate_files_hash(sub)
+    files = {
+      submission: {
+        url: sub.upload.url,
+        mime_type: sub.upload.read_metadata[:mimetype],
+        should_replace_paths: false
+      },
+      grader: {
+        url: self.upload.url,
+        mime_type: self.upload.read_metadata[:mimetype],
+        should_replace_paths: false
+      }
+    }
+  end
+
 
   def do_grading(assignment, sub)
     # Nothing to do, since Orca will handle it!

app/views/graders/_enable_orca.html.erb

@@ -1,10 +1,12 @@
 <% if current_user.site_admin? %>
+  <% if !local_assigns[:force_orca] %>
   <tr class="form-group">
     <td class="col-sm-4"><%= f.label "orca_status", "Use Orca? (Experimental: do not use yet!)" %></td>
     <td><%= f.check_box :orca_status, checked: !!f.object.orca_status,
       data: {toggle: "toggle", on: "Yes", off: "No"} %></td>
     <td></td>
   </tr>
+  <% end %>
   <% if f.object.orca_status  %>
     <tr>
       <td><strong>Orca Grader Build</strong></td>

app/views/graders/_sandbox_grader.html.erb

@@ -23,6 +23,7 @@
   </tr>
   <%= render "graders/grader_file_picker", f: f, id: f.object.id.to_s,
       label: "Grading file:", schema: "sandbox-schema" %>
+  <%= render partial: "graders/enable_orca", locals: { f: f, force_orca: true } %>
   <!--
   <tr class="form-group">
     <td class="col-sm-2"><%= f.label "extra_upload_file", "Extra file:" %></td>