Produce TRX properties only if Trx is enabled

[Youssef1313]

Please check the following before creating a Pull Request

• If this is a new feature or piece of functionality, have you started a discussion and gotten agreement on it?
• If it fixes a bug or problem, is there an issue to track it? If not, create one first and link it please so there's clear visibility.
• Did you write tests to ensure you code works properly?

[thomhurst]

Thanks @Youssef1313 ! Sorry for the delay. Can you fix the conflicts please?

[thomhurst]

@claude could you rebase this PR and fix the conflicts?

[claude]

Claude encountered an error —— View job

Failed with exit code 128

---

I'll analyze this and get back to you.

[thomhurst]

@copilot can you rebase this PR?

[claude]

Claude encountered an error —— View job

Failed with exit code 128

---

I'll analyze this and get back to you.

[thomhurst]

Does this get called by the non-abstractions Trx package?

[Youssef1313]

No. It's not part of ITrxReportCapability interface. It's read in places that create the Trx-related messages so that they can be skipped if trx isn't enabled.

[thomhurst]

I mean what actually calls enable

[Youssef1313]

Ah, Enable itself is called by the Trx extension (the non-abstractions package):

https://github.com/microsoft/testfx/blob/26b1f1b86bbaa1c524e4b91c6bbd838687ea23cc/src/Platform/Microsoft.Testing.Extensions.TrxReport/TrxDataConsumer.cs#L206

https://github.com/microsoft/testfx/blob/26b1f1b86bbaa1c524e4b91c6bbd838687ea23cc/src/Platform/Microsoft.Testing.Extensions.TrxReport/TrxDataConsumer.cs#L80