Harden docker detection (#2552)
This commit is contained in:
parent
bc8aae5d50
commit
bafde7738f
|
|
@ -37,15 +37,12 @@ namespace Microsoft.AspNetCore.SignalR.Redis.Tests
|
||||||
|
|
||||||
var docker = new Docker(location);
|
var docker = new Docker(location);
|
||||||
|
|
||||||
// Windows docker must have Linux containers turned on, if they don't skip the docker tests
|
docker.RunCommand("info --format '{{.OSType}}'", out var output);
|
||||||
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
|
|
||||||
{
|
|
||||||
docker.RunCommand("info --format '{{.OSType}}'", out var output);
|
|
||||||
|
|
||||||
if (!string.Equals(output, "linux"))
|
if (!string.Equals(output.Trim('\'', '"', '\r', '\n', ' '), "linux"))
|
||||||
{
|
{
|
||||||
return null;
|
Console.WriteLine($"'docker info' output: {output}");
|
||||||
}
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
return docker;
|
return docker;
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue