Skip to content

Commit c4f2aca

Browse files
committed
fix dodgy container spawning
1 parent a76339a commit c4f2aca

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

test/SeqCli.EndToEnd/Support/TestConfiguration.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public CaptiveProcess SpawnCliProcess(string command, string additionalArgs = nu
3232
return new CaptiveProcess("dotnet", $"{TestedBinary} {commandWithArgs}", environment, supplyInput: supplyInput);
3333
}
3434

35-
public CaptiveProcess SpawnServerProcess(string storagePath, out Func<string> getOutput)
35+
public CaptiveProcess SpawnServerProcess(string storagePath)
3636
{
3737
if (storagePath == null) throw new ArgumentNullException(nameof(storagePath));
3838

@@ -41,9 +41,9 @@ public CaptiveProcess SpawnServerProcess(string storagePath, out Func<string> ge
4141
{
4242
var containerName = Guid.NewGuid().ToString("n");
4343
const string containerRuntime = "docker";
44-
return new CaptiveProcess(containerRuntime, $"run --name {containerName} -e ACCEPT_EULA=Y -p {_serverListenPort}:80 datalust/seq:{imageTag}", stopCommandFullExePath: containerRuntime, stopCommandArgs: $"rm -f {containerName}");
44+
return new CaptiveProcess(containerRuntime, $"run --name {containerName} -d -e ACCEPT_EULA=Y -p {_serverListenPort}:80 datalust/seq:{imageTag}", stopCommandFullExePath: containerRuntime, stopCommandArgs: $"rm -f {containerName}");
4545
}
4646

47-
var proc = new CaptiveProcess("seq", commandWithArgs);
47+
return new CaptiveProcess("seq", commandWithArgs);
4848
}
4949
}

0 commit comments

Comments
 (0)