Revert "Record test start/end events for data driven tests"

[nohwnd]

Reverts #631
Reverts the PR because master fails in compat tests after the PR was merged.

[NGloreous]

@nohwnd what's failing?

[nohwnd]

/azp run TestFx.CI

[azure-pipelines]

No pipelines are associated with this pull request.

[nohwnd]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[nohwnd]

@NGloreous sorry for the lack of info, I was busy with release when I was reverting this.

This is where it started failing this build which was triggered after merging your PR to master. It fails on RunAllCompatTests.

I was trying to fix it in #671 because I hoped it will be just a quick fix (the original error I saw was just a missing parameter on a method call), but it turned out that the error is cryptic because the test base does not print errors when they are written in the log. So the message in the above build is confusing (because no tests were run, so they are none are in the list) and the error is actually this.

If you have any insight into how to fix this, please help me out in #671, so we can revert back the commit I am reverting here. 🙂

[NGloreous]

@nohwnd thanks for taking care of this and sorry for the hassle. I've signed off here. I will take a look at #671 but it may have to wait until later in the week.

[nohwnd]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).