Merge pull request #15 from davidwessman/rails-failure-screenshot

Adds support for Rails failure screenshots

lib/minitest/junit.rb

@@ -62,6 +62,7 @@ def format(result, parent = nil)
         testcase['file'] = relative_to_cwd(result.source_location.first)
         testcase['line'] = result.source_location.last
         testcase['assertions'] = result.assertions
+
         if result.skipped?
           skipped = Ox::Element.new('skipped')
           skipped['message'] = result
@@ -76,6 +77,16 @@ def format(result, parent = nil)
           end
         end
 
+        # Minitest 5.19 supports metadata
+        # Rails 7.1 adds `failure_screenshot_path` to metadata
+        # Output according to Gitlab format
+        # https://docs.gitlab.com/ee/ci/testing/unit_test_reports.html#view-junit-screenshots-on-gitlab
+        if result.respond_to?("metadata") && result.metadata[:failure_screenshot_path]
+          screenshot = Ox::Element.new("system-out")
+          screenshot << "[[ATTACHMENT|#{result.metadata[:failure_screenshot_path]}]]"
+          testcase << screenshot
+        end
+
         testcase
       end
 

test/reporter_test.rb

@@ -49,8 +49,9 @@ def test_formats_each_failed_result_with_a_formatter
     results.each do |result|
       parsed_report.xpath("//testcase[@name='#{result.name}']").any?
     end
-    # Check if some testcase has a failure
+    # Check if some testcase has a failure and screenshot path
     assert parsed_report.xpath("//testcase//failure").any?
+    assert parsed_report.xpath("//testcase//system-out").any?
   end
 
   def test_xml_nodes_has_file_and_line_attributes
@@ -93,6 +94,11 @@ def create_test_result(name: FakeTestName, methodname: 'test_method_name', succe
         end
       end.new
     end
+
+    if failures.positive?
+      test.metadata[:failure_screenshot_path] = '/tmp/screenshot.png'
+    end
+
     Minitest::Result.from test
   end