Remove using environment variables for specifying tests. (#1410)

Author: jkotalik

test/Common.FunctionalTests/Inprocess/LogPipeTests.cs

@@ -1,7 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System.IO;
 using System.Net;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Server.IIS.FunctionalTests.Utilities;
@@ -41,27 +40,26 @@ public async Task CheckStdoutLoggingToPipe_DoesNotCrashProcess(string path)
         }
 
         [ConditionalTheory]
-        [InlineData("ConsoleErrorWrite")]
-        [InlineData("ConsoleWrite")]
+        [InlineData("ConsoleErrorWriteStartServer")]
+        [InlineData("ConsoleWriteStartServer")]
         public async Task CheckStdoutLoggingToPipeWithFirstWrite(string path)
         {
             var deploymentParameters = _fixture.GetBaseDeploymentParameters(publish: true);
 
-            var firstWriteString = path + path;
+            var firstWriteString = "TEST MESSAGE";
 
-            deploymentParameters.WebConfigBasedEnvironmentVariables["ASPNETCORE_INPROCESS_INITIAL_WRITE"] = firstWriteString;
+            deploymentParameters.TransformArguments((a, _) => $"{a} {path}");
 
             var deploymentResult = await DeployAsync(deploymentParameters);
 
-            await Helpers.AssertStarts(deploymentResult, path);
+            await Helpers.AssertStarts(deploymentResult);
 
             StopServer();
 
             if (deploymentParameters.ServerType == ServerType.IISExpress)
             {
                 // We can't read stdout logs from IIS as they aren't redirected.
                 Assert.Contains(TestSink.Writes, context => context.Message.Contains(firstWriteString));
-                Assert.Contains(TestSink.Writes, context => context.Message.Contains("TEST MESSAGE"));
             }
         }
 

test/Common.FunctionalTests/LogFileTests.cs

@@ -3,7 +3,6 @@
 
 using System;
 using System.IO;
-using System.Net;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Server.IIS.FunctionalTests.Utilities;
 using Microsoft.AspNetCore.Server.IntegrationTesting;

test/WebSites/InProcessWebSite/Program.cs

@@ -79,31 +79,33 @@ public static int Main(string[] args)
                         host.Run();
                     }
                     break;
+                case "ConsoleErrorWriteStartServer":
+                    Console.Error.WriteLine("TEST MESSAGE");
+                    return StartServer();
+                case "ConsoleWriteStartServer":
+                    Console.WriteLine("TEST MESSAGE");
+                    return StartServer();
                 default:
-                    {
-
-                        var envVariable = Environment.GetEnvironmentVariable("ASPNETCORE_INPROCESS_INITIAL_WRITE");
-                        if (!string.IsNullOrEmpty(envVariable))
-                        {
-                            Console.WriteLine(envVariable);
-                            Console.Error.WriteLine(envVariable);
-                        }
-
-                        var host = new WebHostBuilder()
-                            .ConfigureLogging((_, factory) =>
-                            {
-                                factory.AddConsole();
-                                factory.AddFilter("Console", level => level >= LogLevel.Information);
-                            })
-                            .UseIIS()
-                            .UseStartup<Startup>()
-                            .Build();
+                    return StartServer();
 
-                        host.Run();
-                        return 0;
-                    }
             }
             return 12;
         }
+
+        private static int StartServer()
+        {
+            var host = new WebHostBuilder()
+                .ConfigureLogging((_, factory) =>
+                {
+                    factory.AddConsole();
+                    factory.AddFilter("Console", level => level >= LogLevel.Information);
+                })
+                .UseIIS()
+                .UseStartup<Startup>()
+                .Build();
+
+            host.Run();
+            return 0;
+        }
     }
 }