added preliminary support for new gtest macro GTEST_SKIP()

[csoltenborn]

Added support for GTEST_SKIP() macro (not yet released) - this takes case of #260

@sixten-hilborn: Would you mind to give it a try? You can download the according build artifacts from here...

As a side note: I have not tested this on the actual adapter, but merely adjusted the output parsers and provided according unit tests. However, the skipped tests have been marked as failures before the patch, but not as test crashes. Is the skipped test indeed marked as "probably CRASHED" for the minimal example you provided, or is there anything special about the test which made you report the issue in the first place?

[codecov]

Codecov Report

Merging #262 into master will increase coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #262      +/-   ##
==========================================
+ Coverage   73.52%   73.63%   +0.11%     
==========================================
  Files          95       95              
  Lines        3565     3580      +15     
  Branches      516      519       +3     
==========================================
+ Hits         2621     2636      +15     
  Misses        765      765              
  Partials      179      179

Flag	Coverage Δ
#Integration	52.7% <17.64%> (-0.2%)	⬇️
#Unit	60.22% <100%> (+0.16%)	⬆️

Impacted Files	Coverage Δ
...Results/StreamingStandardOutputTestResultParser.cs	92.85% <100%> (+0.38%)	⬆️
...Core/TestResults/StandardOutputTestResultParser.cs	96.22% <100%> (+0.39%)	⬆️

[codecov]

Codecov Report

Merging #262 into master will increase coverage by 1.4%.
The diff coverage is 70.83%.

@@            Coverage Diff            @@
##           master     #262     +/-   ##
=========================================
+ Coverage   73.52%   74.92%   +1.4%     
=========================================
  Files          95       95             
  Lines        3565     3605     +40     
  Branches      516      524      +8     
=========================================
+ Hits         2621     2701     +80     
+ Misses        765      725     -40     
  Partials      179      179

Flag	Coverage Δ
#Integration	55.25% <37.5%> (+2.35%)	⬆️
#Unit	60.11% <64.58%> (+0.05%)	⬆️

Impacted Files	Coverage Δ
...Results/StreamingStandardOutputTestResultParser.cs	93.87% <100%> (+1.4%)	⬆️
...leTestAdapter/Core/Runners/SequentialTestRunner.cs	75.89% <100%> (+2.67%)	⬆️
...Core/TestResults/StandardOutputTestResultParser.cs	96.22% <100%> (+0.39%)	⬆️
...gleTestAdapter/Core/Runners/TestResultCollector.cs	88.13% <100%> (+23.72%)	⬆️
...estAdapter/Core/TestResults/XmlTestResultParser.cs	80% <50%> (-12%)	⬇️
...ProcessExecution/DebuggedProcessExecutorFactory.cs	83.33% <0%> (+16.66%)	⬆️
...ocessExecution/FrameworkDebuggedProcessExecutor.cs	68% <0%> (+68%)	⬆️
... and 1 more

[sixten-hilborn]

That was fast! :)
I'll probably not be able to test this until Monday. I'm not sure if this case was reported as crashed, but I definitely had another test reported as crashed. I'll double-check that!

[csoltenborn]

If you give it a try, please use this build...

[sixten-hilborn]

Yep, now they're correctly reported as Skipped !

[csoltenborn]

Great, thanks for testing!