Doubled up execution starting/finished messages

Author: bradwilson

src/xunit.runner.visualstudio/VsTestRunner.cs

@@ -583,13 +583,6 @@ await DiscoverTestsInAssembly(
 					executionOptions.SetDisableParallelization(true);
 				}
 
-				reporterMessageHandler.OnMessage(new TestAssemblyExecutionStarting
-				{
-					Assembly = runInfo.Assembly,
-					ExecutionOptions = executionOptions,
-					Seed = executionOptions.GetSeed(),
-				});
-
 				using var vsExecutionSink = new VsExecutionSink(reporterMessageHandler, frameworkHandle, logger, testCasesMap, () => cancelled);
 				var executionSinkOptions = new ExecutionSinkOptions
 				{
@@ -614,12 +607,6 @@ await DiscoverTestsInAssembly(
 
 				resultsSink.Finished.WaitOne();
 
-				reporterMessageHandler.OnMessage(new TestAssemblyExecutionFinished
-				{
-					Assembly = runInfo.Assembly,
-					ExecutionOptions = executionOptions,
-					ExecutionSummary = resultsSink.ExecutionSummary,
-				});
 				if ((resultsSink.ExecutionSummary.Failed != 0 || resultsSink.ExecutionSummary.Errors != 0) && executionOptions.GetStopOnTestFailOrDefault())
 				{
 					logger.Log("Canceling due to test failure...");