Merge pull request #110 from benpollarduk/benpollarduk-patch-1

Benpollarduk patch 1
benpollarduk · Dec 4, 2024 · fbcd84e · fbcd84e
2 parents 805a782 + cfb4934
commit fbcd84e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/NetAF.Tests/Rendering/Console/Ansi_Tests.cs b/NetAF.Tests/Rendering/Console/Ansi_Tests.cs
@@ -18,23 +18,23 @@ public void GivenNoColorEnvironmentVariableSetToEmptyString_WhenIsColorSuppresse
         }
 
         [TestMethod]
-        public void GivenNoColorEnvironmentVariableSetTo0_WhenIsColorSuppressed_ThenReturnFalse()
+        public void GivenNoColorEnvironmentVariableSetTo0_WhenIsColorSuppressed_ThenReturnTrue()
         {
             Environment.SetEnvironmentVariable(Ansi.NO_COLOR, "0");
 
             var result = Ansi.IsColorSuppressed();
 
-            Assert.IsFalse(result);
+            Assert.IsTrue(result);
         }
 
         [TestMethod]
-        public void GivenNoColorEnvironmentVariableSetToFalse_WhenIsColorSuppressed_ThenReturnFalse()
+        public void GivenNoColorEnvironmentVariableSetToFalse_WhenIsColorSuppressed_ThenReturnTrue()
         {
             Environment.SetEnvironmentVariable(Ansi.NO_COLOR, "False");
 
             var result = Ansi.IsColorSuppressed();
 
-            Assert.IsFalse(result);
+            Assert.IsTrue(result);
         }
 
         [TestMethod]

diff --git a/NetAF/Rendering/Console/Ansi.cs b/NetAF/Rendering/Console/Ansi.cs
@@ -31,14 +31,14 @@ public static class Ansi
         /// <summary>
         /// Determine if color is suppressed. If the NO_COLOR environment variable is present and set to anything other than '0' or 'false' this will return true.
         /// </summary>
-        /// <returns>True if the NO_COLOR environment variable is present and set to anything other than '0' or 'false', else false.</returns>
+        /// <returns>True if the NO_COLOR environment variable is present and set to anything other than an empty string, else false.</returns>
         public static bool IsColorSuppressed()
         {
             var value = Environment.GetEnvironmentVariable(NO_COLOR)?.ToLower() ?? string.Empty;
 
             return value switch
             {
-                "" or "0" or "false" => false,
+                "" => false,
                 _ => true,
             };
         }