razor
razor copied to clipboard
Add missing integration test for telemetry testing
Currently Diagnostic integration tests are skipped due to https://github.com/dotnet/razor/issues/8150
We'll want an integration test that validates the telemetry events we are interested are fired when expected.
/cc @ryzngard
Just my personal opinion, but tests for telemetry seem unnecessary to me, and I can't think how we would do them with our current testing tech without comprising the quality of the code.
I would say that integration tests specifically for telemetry are not needed, but adding telemetry checks to integration tests would be useful to validate we expect certain things to be fired