Some hardcoded (previsously made optional) directives were brought back

[pedrolimasi]

Brief Issue Summary

Commit 4293388 brought back the ctest hardcoded directives '-T', 'test', '--output-on-failure' which were made optional on PR #1729. Some of the code we wrote was kept, though. Should I interpret this as an accident?

Quick fix

It seems from my side that fixing it will just be a matter of deleting them from here:

vscode-cmake-tools/src/ctest.ts

Lines 366 to 367 in b71fb38

	ctestArgs = [`-j${this.ws.config.numCTestJobs}`, '-C', configuration, '-T', 'test', '--output-on-failure'].concat(
	this.ws.config.ctestDefaultArgs, this.ws.config.ctestArgs);

I would've submit a pull request, but if this is all it takes to fix it you guys should get to it much faster than I

[bobbrow]

It looks like an incorrect merge happened. Thanks for spotting this!