implements system-out and system-err output formating using example-m…

…etadata

example/spec/example_spec.rb

@@ -1,3 +1,4 @@
+require "spec_helper"
 require_relative "shared_examples"
 
 describe "some example specs" do
@@ -42,4 +43,9 @@
   end
 
   it_should_behave_like "shared examples"
+
+  it "can capture stdout and stderr" do
+    $stdout.puts "Test"
+    $stderr.puts "Bar"
+  end
 end

example/spec/spec_helper.rb

@@ -0,0 +1,15 @@
+RSpec.configure do |config|
+  # register around filter that captures stderr and stdout
+  config.around(:each) do |example|
+    $stdout = StringIO.new
+    $stderr = StringIO.new
+
+    example.run
+
+    example.metadata[:stdout] = $stdout.string
+    example.metadata[:stderr] = $stderr.string
+
+    $stdout = STDOUT
+    $stderr = STDERR
+  end
+end

lib/rspec_junit_formatter.rb

@@ -71,9 +71,24 @@ def xml_dump_example(example)
     output << %{ time="#{escape("%.6f" % duration_for(example))}"}
     output << %{>}
     yield if block_given?
+    xml_dump_output(example)
     output << %{</testcase>\n}
   end
 
+  def xml_dump_output(example)
+    if (stdout = stdout_for(example)) && !stdout.empty?
+      output << %{<system-out>}
+      output << escape(stdout)
+      output << %{</system-out>}
+    end
+
+    if (stderr = stderr_for(example)) && !stderr.empty?
+      output << %{<system-err>}
+      output << escape(stderr)
+      output << %{</system-err>}
+    end
+  end
+
   # Inversion of character range from https://www.w3.org/TR/xml/#charsets
   ILLEGAL_REGEXP = Regexp.new(
     "[^" <<

lib/rspec_junit_formatter/rspec2.rb

@@ -68,4 +68,12 @@ def find_shared_group(example)
   def group_and_parent_groups(example)
     example.example_group.parent_groups + [example.example_group]
   end
+
+  def stdout_for(example)
+    example.metadata[:stdout]
+  end
+
+  def stderr_for(example)
+    example.metadata[:stderr]
+  end
 end

lib/rspec_junit_formatter/rspec3.rb

@@ -99,6 +99,14 @@ def strip_diff_colors(string)
     #
     string.sub(STRIP_DIFF_COLORS_BLOCK_REGEXP) { |match| match.gsub(STRIP_DIFF_COLORS_CODES_REGEXP, "".freeze) }
   end
+
+  def stdout_for(example)
+    example.metadata[:stdout]
+  end
+
+  def stderr_for(example)
+    example.metadata[:stderr]
+  end
 end
 
 # rspec-core 3.0.x forgot to mark this as a module function which causes:

spec/rspec_junit_formatter_spec.rb

@@ -13,7 +13,7 @@
 
   let(:testsuite) { doc.xpath("/testsuite").first }
   let(:testcases) { doc.xpath("/testsuite/testcase") }
-  let(:successful_testcases) { doc.xpath("/testsuite/testcase[count(*)=0]") }
+  let(:successful_testcases) { doc.xpath("/testsuite/testcase[not(failure) and not(skipped)]") }
   let(:pending_testcases) { doc.xpath("/testsuite/testcase[skipped]") }
   let(:failed_testcases) { doc.xpath("/testsuite/testcase[failure]") }
   let(:shared_testcases) { doc.xpath("/testsuite/testcase[contains(@name, 'shared example')]") }
@@ -29,7 +29,7 @@
     expect(testsuite).not_to be(nil)
 
     expect(testsuite["name"]).to eql("rspec")
-    expect(testsuite["tests"]).to eql("11")
+    expect(testsuite["tests"]).to eql("12")
     expect(testsuite["skipped"]).to eql("1")
     expect(testsuite["failures"]).to eql("8")
     expect(testsuite["errors"]).to eql("0")
@@ -39,7 +39,7 @@
 
     # it has some test cases
 
-    expect(testcases.size).to eql(11)
+    expect(testcases.size).to eql(12)
 
     testcases.each do |testcase|
       expect(testcase["classname"]).to eql("spec.example_spec")
@@ -49,11 +49,14 @@
 
     # it has successful test cases
 
-    expect(successful_testcases.size).to eql(2)
+    expect(successful_testcases.size).to eql(3)
 
     successful_testcases.each do |testcase|
       expect(testcase).not_to be(nil)
-      expect(testcase.children).to be_empty
+      # test results that capture stdout / stderr are not 'empty'
+      unless (testcase["name"]) =~ /capture stdout and stderr/
+        expect(testcase.children).to be_empty
+      end
     end
 
     # it has pending test cases
@@ -117,6 +120,10 @@
     # it correctly escapes reserved xml characters
 
     expect(doc.xpath("//testcase[contains(@name, 'html')]").first[:name]).to eql(%{some example specs escapes <html tags='correctly' and="such &amp; such">})
+
+    # it correctly captures stdout / stderr output
+    expect(doc.xpath("//testcase/system-out").text).to eql("Test\n")
+    expect(doc.xpath("//testcase/system-err").text).to eql("Bar\n")
   end
 
   context "when $TEST_ENV_NUMBER is set" do