Fixes two sources of intermittent errors: (#430)

* the XUnit demo's random port selection now avoids ports that are in use
* we have used all three ways of declaring that non-test projects really aren't test projects to prevent the 'testhost not found' errors the VS Test Explorer was sometimes reporting

Author: idg10

Solutions/Corvus.Testing.AzureFunctions.DemoFunction.InProcess/Corvus.Testing.AzureFunctions.DemoFunctions.InProcess.csproj

@@ -11,6 +11,7 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
 
   <PropertyGroup>

Solutions/Corvus.Testing.AzureFunctions.DemoFunctions.Isolated/Corvus.Testing.AzureFunctions.DemoFunctions.Isolated.csproj

@@ -12,6 +12,7 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
 
   <ItemGroup>

Solutions/Corvus.Testing.AzureFunctions.SpecFlow.NUnit/Corvus.Testing.AzureFunctions.SpecFlow.NUnit.csproj

@@ -7,7 +7,12 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
+  <ItemGroup>
+    <!-- This project takes depedency on nUnit that adds the TestContainer capability importing NUnit.props, but this is not a test project -->
+    <ProjectCapability Remove="TestContainer" />
+  </ItemGroup>
 
   <PropertyGroup>
     <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>

Solutions/Corvus.Testing.AzureFunctions.SpecFlow/Corvus.Testing.AzureFunctions.SpecFlow.csproj

@@ -10,7 +10,12 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
+  <ItemGroup>
+    <!-- This project takes depedency on nUnit that adds the TestContainer capability importing NUnit.props, but this is not a test project -->
+    <ProjectCapability Remove="TestContainer" />
+  </ItemGroup>
 
   <PropertyGroup>
     <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>

Solutions/Corvus.Testing.AzureFunctions.Xunit.Demo/FunctionPerTestFacts.cs

@@ -35,7 +35,7 @@ public FunctionPerTestFacts(ITestOutputHelper output)
                 .CreateLogger("Xunit Demo tests");
 
             this.function = new FunctionsController(logger);
-            this.Port = new Random().Next(50000, 60000);
+            this.Port = this.function.FindAvailableTcpPort(50000, 60000);
         }
 
         public int Port { get; }

Solutions/Corvus.Testing.AzureFunctions/Corvus.Testing.AzureFunctions.csproj

@@ -11,6 +11,7 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
 
   <PropertyGroup>

Solutions/Corvus.Testing.AzureFunctions/Corvus/Testing/AzureFunctions/FunctionsController.cs

@@ -133,6 +133,34 @@ await Task.WhenAny(
             this.logger.LogInformation("Function {Path} now running on port {Port}", path, port);
         }
 
+        /// <summary>
+        /// Randomly selects a port that appears to be available for use.
+        /// </summary>
+        /// <param name="lowerBoundInclusive">
+        /// The lowest port number acceptable. Defaults to 50000.
+        /// </param>
+        /// <param name="upperBoundExclusive">
+        /// The port number above which no port will be selected. Defaults to 60000.
+        /// </param>
+        /// <returns>A port number that seems to be available.</returns>
+        public int FindAvailableTcpPort(int? lowerBoundInclusive, int? upperBoundExclusive)
+        {
+            int lb = lowerBoundInclusive ?? 50000;
+            int ub = upperBoundExclusive ?? 60000;
+
+            var portsInRangeInUse = IPGlobalProperties
+                .GetIPGlobalProperties()
+                .GetActiveTcpListeners()
+                .Select(e => e.Port)
+                .Where(p => p >= lb && p < ub)
+                .ToHashSet();
+
+            int availablePorts = ub - lb - portsInRangeInUse.Count;
+            int availablePortOffset = Random.Shared.Next(availablePorts);
+            int port = Enumerable.Range(lb, ub - lb).Where(p => !portsInRangeInUse.Contains(p)).ElementAt(availablePortOffset);
+            return port;
+        }
+
         /// <summary>
         /// Provides access to the output.
         /// </summary>

Solutions/Corvus.Testing.SpecFlow.NUnit/Corvus.Testing.SpecFlow.NUnit.csproj

@@ -7,7 +7,12 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
+  <ItemGroup>
+    <!-- This project takes depedency on nUnit that adds the TestContainer capability importing NUnit.props, but this is not a test project -->
+    <ProjectCapability Remove="TestContainer" />
+  </ItemGroup>
 
   <PropertyGroup>
     <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>

Solutions/Corvus.Testing.SpecFlow/Corvus.Testing.SpecFlow.csproj

@@ -10,7 +10,12 @@
 
   <PropertyGroup>
     <IsTestProject>false</IsTestProject>
+    <TestProject>false</TestProject>
   </PropertyGroup>
+  <ItemGroup>
+    <!-- This project takes depedency on nUnit that adds the TestContainer capability importing NUnit.props, but this is not a test project -->
+    <ProjectCapability Remove="TestContainer" />
+  </ItemGroup>
 
   <PropertyGroup>
     <PackageLicenseExpression>Apache-2.0</PackageLicenseExpression>