Add aborted tests to TRX summary

[mfkl]

Description

The current TRX report summary is missing information about various test status, such as aborted, timeout, inconclusive, etc.

Full list is here

vstest/src/Microsoft.TestPlatform.Extensions.TrxLogger/ObjectModel/TestRunSummary.cs

Lines 105 to 116 in 458e90e

	_abortedTests = countForNonExistingResults;
	_errorTests = countForNonExistingResults;
	_timeoutTests = countForNonExistingResults;
	_inconclusiveTests = countForNonExistingResults;
	_passedButRunAborted = countForNonExistingResults;
	_notRunnable = countForNonExistingResults;
	_notExecuted = countForNonExistingResults;
	_disconnected = countForNonExistingResults;
	_warning = countForNonExistingResults;
	_completed = countForNonExistingResults;
	_inProgress = countForNonExistingResults;
	_pending = countForNonExistingResults;

Public repro here https://github.com/mfkl/trx-missing-attributes-repro

It would be nice to have this information as part of the TRX summary, which currently only reports failed/passed test information.

I've started by adding the aborted test result info to the TRX summary. If the approach seems fine with you, I'll add the other properties so that the TRX summary is completely filled.

To write the test, I had to modify the following public API src/Microsoft.TestPlatform.ObjectModel/TestOutcome.cs which does not match the internal API src/Microsoft.TestPlatform.Extensions.TrxLogger/ObjectModel/TestOutcome.cs

Related issue

Contribution towards #2506

[mfkl]

@microsoft-github-policy-service agree

[mfkl]

Hi, could anyone please take a look and provide their thoughts on this?

[MarcoRossignoli]

Thanks @mfkl for the PR, TRX is used in more than one tool and we need to ensure that we're not breaking anybody, at the moment we're busy on higher priority tasks, we'll review and let you know soon as possible but we don't have an ETA at the moment. Thanks for the patience.

If you need it and you're in rush you can fork and use your custom implementation.

[mfkl]

Understood, thanks. Will wait for your feedback!

[MarcoRossignoli]

We did some check and for the moment we decided to not take this PR for risky edge cases, we're planning to work on a full review of the TRX format and implementation. Thanks for your contribution.

[topherhansen]

I think it's really annoying that the TRX says that all of these attributes are zero when it's just making it up. When a sessiontimeout happens (I tell it 300 seconds and it's only run half my tests) it lies and says that the total was some small part of my tests.