Guard against Array status in test results (#8637)

Tooling runners can return `status` as an Array (e.g., `["pass"]`)
inside individual test result objects. Use `Array().first` to safely
unwrap, matching the existing pattern for the `version` field.

Closes #8634
Closes #8635

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: iHiD

app/models/submission/test_run.rb

@@ -74,7 +74,7 @@ class TestResult
     def to_h
       {
         name: test[:name].to_s,
-        status: test[:status].try(&:to_sym),
+        status: Array(test[:status]).first.try(&:to_sym),
         test_code: test[:test_code],
         message: test[:message],
         message_html: Ansi::RenderHTML.(test[:message]),

test/models/submission/test_run_test.rb

@@ -235,6 +235,12 @@ class Submission::TestRunTest < ActiveSupport::TestCase
     assert_equal 3, tr.version
   end
 
+  test "handles test result status as array" do
+    tests = [{ 'name' => 'test1', 'status' => ['pass'] }]
+    tr = create(:submission_test_run, raw_results: { version: 2, status: 'pass', tests: tests })
+    assert_equal :pass, tr.test_results.first.to_h[:status]
+  end
+
   test "truncate message" do
     message = 'a' * 66_000
     test_run = create(:submission_test_run, raw_results: { message: })