BuildServerDetector.Detected also set for tests running in Dockerfile

SimonCropp · SimonCropp · commit e4afc9acdec0 · 2021-05-19T10:41:22.000+10:00
fixes #226
diff --git a/src/DiffEngine/BuildServerDetector.cs b/src/DiffEngine/BuildServerDetector.cs
@@ -76,6 +76,14 @@ static BuildServerDetector()
                 Detected = true;
                 return;
             }
+
+            // Docker
+            // https://www.hanselman.com/blog/detecting-that-a-net-core-app-is-running-in-a-docker-container-and-skippablefacts-in-xunit
+            if (Environment.GetEnvironmentVariable("DOTNET_RUNNING_IN_CONTAINER") == "true")
+            {
+                Detected = true;
+                return;
+            }
         }
 
         public static bool Detected { get; set; }
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -2,7 +2,7 @@
 <Project>
   <PropertyGroup>
     <NoWarn>CS1591;CS0649</NoWarn>
-    <Version>6.8.0</Version>
+    <Version>6.8.1</Version>
     <AssemblyVersion>1.0.0</AssemblyVersion>
     <PackageTags>Testing, Snapshot, Diff, Compare</PackageTags>
     <Description>Launches diff tools based on file extensions. Designed to be consumed by snapshot testing libraries.</Description>

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,14 @@ static BuildServerDetector()`
`76`	`76`	`Detected = true;`
`77`	`77`	`return;`
`78`	`78`	`}`
	`79`	`+`
	`80`	`+ // Docker`
	`81`	`+ // https://www.hanselman.com/blog/detecting-that-a-net-core-app-is-running-in-a-docker-container-and-skippablefacts-in-xunit`
	`82`	`+ if (Environment.GetEnvironmentVariable("DOTNET_RUNNING_IN_CONTAINER") == "true")`
	`83`	`+ {`
	`84`	`+ Detected = true;`
	`85`	`+ return;`
	`86`	`+ }`
`79`	`87`	`}`
`80`	`88`
`81`	`89`	`public static bool Detected { get; set; }`