Feat: Update ITestResultSeralizer to accept messages.

[Siphonophora]

Pass messages to seralizer as discussed in spekt/junit.testlogger#33

[codecov]

Codecov Report

Merging #19 (9cba3ff) into master (f5704c0) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           19        19           
  Lines          502       503    +1     
=========================================
+ Hits           502       503    +1     

Impacted Files	Coverage Δ
src/TestLogger/Core/TestRunCompleteWorkflow.cs	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5704c0...9cba3ff. Read the comment docs.

[Siphonophora]

@codito So, this is pretty simple. I was a bit surprised by the two tests that were failing. I think those are just windows specific issues.

Let me know if you think I should add tests to this. The part I care about for junit will be covered by the unit test that is currently failing in spekt/junit.testlogger#33

[codito]

👍 thanks! Will add a windows CI soon.

[codito]

It will be good to add a test. May be we can update the JsonTestResultSerializer output schema to include messages: https://github.com/spekt/testlogger/blob/master/test/TestLogger.UnitTests/TestDoubles/JsonTestResultSerializer.cs#L20, this will allow us to assert the output content in TestRunCompleteWorkflowTests.

[Siphonophora]

How about this as an option. Its simpler than putting messages in the JSON. Any downside to this?

[codito]

JsonTestResultSerializer is also part of the json test logger which is used in Acceptance Tests. With this approach, we may not be able to validate messages as part of an acceptance test (in future).

[Siphonophora]

Yep makes sense. We should be all set now.

Note, I dropped traits from the sample json, since they aren't in the Test model.

[codito]

Looks perfect! Thanks.