Fix: Report 0 tests found in case it is not a CatchTest exe.

Author: xkbeyer

TestAdapter/TestDiscoverer.cs

@@ -127,7 +127,8 @@ private static IEnumerable<TestCase> ParseListing( string exeName, IList<string>
             // The first line should be fixed.
             if( !line.MoveNext() || line.Current != "All available test cases:" )
             {
-                throw new Exception( "Unexpected line in catch output: " + line.Current );
+                yield break;
+                //throw new Exception( "Unexpected line in catch output: " + line.Current );
             }
 
             // Split output to groups of lines related to the same test case.

TestAdapterTest/TestTestDiscoverer.cs

@@ -19,8 +19,8 @@ public class TestTestDiscoverer
         // Tests that all the tests in the reference project are found.
         [TestMethod]
         [DeploymentItem( Common.ReferenceExePath )]
-		public void DiscoversAllTests()
-		{
+        public void DiscoversAllTests()
+        {
             // Initialize a mock sink to keep track of the discovered tests.
             MockTestCaseDiscoverySink testSink = new MockTestCaseDiscoverySink();
 
@@ -145,5 +145,26 @@ public void FiltersTestExecutables()
             // There should be no tests, as nothing matches the filter.
             Assert.AreEqual( testSink.Tests.Count, 0 );
         }
+
+        // Tests that a non Catch exe returns no test cases.
+        [TestMethod]
+        public void DiscoversNoTests()
+        {
+            // Initialize a mock sink to keep track of the discovered tests.
+            MockTestCaseDiscoverySink testSink = new MockTestCaseDiscoverySink();
+
+            TestDiscoverer discoverer = new TestDiscoverer();
+            var cd = System.IO.Directory.GetCurrentDirectory();
+            // Unfortunately it doesn't get copied with the DeployItemAttribute, no idea why.
+            System.IO.File.WriteAllText(@"nonecatchexe.cmd", @"@echo Non Catch Output line");
+            // Returns an unexpected first line.
+            discoverer.DiscoverTests(new List<String>(){ @"nonecatchexe.cmd" }, 
+                new MockDiscoveryContext(),
+                new MockMessageLogger(),
+                testSink);
+
+            // Zero test cases should be registered.
+            Assert.AreEqual(0, testSink.Tests.Count);
+        }
     }
 }

source.extension.vsixmanifest

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <PackageManifest Version="2.0.0" xmlns="http://schemas.microsoft.com/developer/vsx-schema/2011" xmlns:d="http://schemas.microsoft.com/developer/vsx-schema-design/2011">
     <Metadata>
-        <Identity Id="CatchTestAdapter.xkBeyer.f56db3ea-257e-4f65-a52c-a6160aa8e329" Version="1.3" Language="en-US" Publisher="xkbeyer" />
+        <Identity Id="CatchTestAdapter.xkBeyer.f56db3ea-257e-4f65-a52c-a6160aa8e329" Version="1.3.1" Language="en-US" Publisher="xkbeyer" />
         <DisplayName>CatchTestAdapter</DisplayName>
         <Description xml:space="preserve">Test Adapter for the Catch C++ unit test framework</Description>
         <License>LICENSE.txt</License>