TRX Reporter for MultiNode tests

[valdisz]

Test reporter in TRX format for MulitNode tests. This will allow Azure DevOps and other tools, who understands TRX, to see actual test outcome.

It is possible to use either TeamCity or TRX reporter depending on parameter value:
-Dmulitnode.reporter=teamcity or -Dmulitnode.reporter=trx

-Dmultinode.teamcity switch is removed and build scripts are updated accordingly.

PR is related to #3855

[valdisz]

Right now PR exists to verify that something will be reported to DevOps during a build.

Please do not look into it yet :)

[valdisz]

@Aaronontheweb Incrementalist is failing :( Can you direct into a possible issue? Before I will spend some hours on it.

[Aaronontheweb]

Yep, I'd be happy to take a look

…

Sent from my iPhone

On Sep 5, 2019, at 3:21 PM, Valdis Zobēla ***@***.***> wrote: @Aaronontheweb Incrementalist is failing :( Can you direct into a possible issue? Before I will spend some hours on it. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[valdisz]

@Aaronontheweb seems that build related token does not allow to publish test results. I can verify that this is really so, then we have 2 possibilities:

1. Create access token with needed permissions
2. Generate xunit test report style instead of direct reporting to devops

What you will choose?

[Aaronontheweb]

@valdisz I'd just emit an XUnit-style report and I can come along later and output those TRX files up to Azure DevOps, which is what we do here:

https://github.com/akkadotnet/akka.net/blob/dev/build.fsx#L232

This forces XUnit to emit a TRX file, which can be read and understood by Azure DevOps.

And here is where we tell Azure DevOps how to consume it:

akka.net/build-system/azure-pipeline.template.yaml

Lines 34 to 39 in 37be6a5

	- task: PublishTestResults@2
	inputs:
	testRunner: VSTest
	testResultsFiles: '**/*.trx' #TestResults folder usually
	testRunTitle: ${{ parameters.name }}
	mergeTestResults: true

So all we really need is just a way to output a .TRX file - does that make sense?

[valdisz]

@Aaronontheweb you can look on this.

[Aaronontheweb]

@valdisz will do

[Aaronontheweb]

Have some changes.

I don't think this could should ship in separate projects - could you please put all of it under the Akka.MultiNodeTestRunner.Shared package instead? In its own namespace?

[Aaronontheweb]

@valdisz thanks for answering my question about the test structure - will review this again once the other changes are implemented.

[Aaronontheweb]

LGTM