Improve code coverage

Author: arbitrarybytes

src/ArbitraryExtensions.Tests/StringExtensionsTests.cs

@@ -8,10 +8,6 @@ namespace ArbitraryExtensions.Tests
     [TestFixture]
     public class StringExtensionsTests
     {
-        [SetUp]
-        public void Setup()
-        {
-        }
 
         [Test]
         [TestCase(null, default(string))]
@@ -20,7 +16,7 @@ public void Setup()
         [TestCase(" ", " ")]
         public void TestDefaultIfNullOrEmpty(string value, string expect)
         {
-            Assert.AreEqual(expect, StringExtensions.DefaultIfNullOrEmpty(value));
+            Assert.AreEqual(expect, value.DefaultIfNullOrEmpty());
         }
 
         [Test]
@@ -31,7 +27,7 @@ public void TestDefaultIfNullOrEmpty(string value, string expect)
         [TestCase(" ", "Default", " ")]
         public void TestDefaultIfNullOrEmptyWithDefaultValue(string value, string defaultValue, string expect)
         {
-            Assert.AreEqual(expect, StringExtensions.DefaultIfNullOrEmpty(value, defaultValue));
+            Assert.AreEqual(expect, value.DefaultIfNullOrEmpty(defaultValue));
         }
 
         [Test]
@@ -44,7 +40,7 @@ public void TestDefaultIfNullOrEmptyWithDefaultValue(string value, string defaul
         [TestCase("\t", default(string))]
         public void TestDefaultIfNullOrWhitespace(string value, string expect)
         {
-            Assert.AreEqual(expect, StringExtensions.DefaultIfNullOrWhitespace(value));
+            Assert.AreEqual(expect, value.DefaultIfNullOrWhitespace());
         }
 
         [Test]
@@ -58,13 +54,14 @@ public void TestDefaultIfNullOrWhitespace(string value, string expect)
         [TestCase("\t", "Default", "Default")]
         public void TestDefaultIfNullOrWhitespaceWithDefaultValue(string value, string defaultValue, string expect)
         {
-            Assert.AreEqual(expect, StringExtensions.DefaultIfNullOrWhitespace(value, defaultValue));
+            Assert.AreEqual(expect, value.DefaultIfNullOrWhitespace(defaultValue));
         }
 
         [Test]
         [TestCase("this is a lowercase statement", false, "This Is A Lowercase Statement")]
         [TestCase("MS VS", false, "MS VS")]
-        [TestCase("MSVS", true, "Msvs")]
+        [TestCase("MASKS", true, "Masks")]
+        [TestCase("", false, "")]
         public void TestTitleCase(string value, bool includeAllCaps, string expect)
         {
             Assert.AreEqual(expect, value.ToTitleCase(includeAllCaps));

src/ArbitraryExtensions/Core/StringExtensions.cs

@@ -10,7 +10,7 @@ public static class StringExtensions
 
         /// <summary>Checks if the provided input is null or whitespace and returns the provided default value instead.</summary>
         /// <param name="input">the value to check</param>
-        /// <param name="defaultValue">optional, the default value to return if input is null or whitepace.</param>
+        /// <param name="defaultValue">optional, the default value to return if input is null or whitespace.</param>
         /// <returns>input if it's not null or whitespace, otherwise the provided default value.</returns>
         public static string DefaultIfNullOrWhitespace(this string input, string defaultValue = default) => string.IsNullOrWhiteSpace(input) ? defaultValue : input;
 
@@ -21,10 +21,8 @@ public static class StringExtensions
         public static string ToTitleCase(this string text, bool includeAllCaps = false)
         {
             if (string.IsNullOrWhiteSpace(text)) return text;
-            System.Globalization.TextInfo textInfo = System.Threading.Thread.CurrentThread.CurrentCulture.TextInfo;
+            var textInfo = System.Threading.Thread.CurrentThread.CurrentCulture.TextInfo;
             return textInfo.ToTitleCase(includeAllCaps ? text.ToLower() : text);
         }
-
-
     }
 }