refactor: Removed defensive check

Author: linkdotnet

CHANGELOG.md

@@ -6,6 +6,9 @@ All notable changes to **ValueStringBuilder** will be documented in this file. T
 
 ## [Unreleased]
 
+### Changed
+- Remove unused defensive check in `EnsureCapacity`
+
 ## [1.13.0] - 2023-03-04
 
 ### Added

src/LinkDotNet.StringBuilder/ValueStringBuilder.cs

@@ -159,11 +159,6 @@ public ValueStringBuilder(ReadOnlySpan<char> initialText)
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public void EnsureCapacity(int newCapacity)
     {
-        if (newCapacity < 0)
-        {
-            throw new ArgumentOutOfRangeException(nameof(newCapacity), "Capacity can't be negative.");
-        }
-
         if (newCapacity > Length)
         {
             Grow(newCapacity);

tests/LinkDotNet.StringBuilder.UnitTests/ValueStringBuilderTests.cs

@@ -259,23 +259,6 @@ public void ShouldNotSetCapacityWhenSmallerThanCurrentString()
         builder.Length.Should().BeGreaterOrEqualTo(128);
     }
 
-    [Fact]
-    public void ShouldThrowWhenNegativeValueInEnsureCapacity()
-    {
-        using var builder = new ValueStringBuilder();
-        try
-        {
-            builder.EnsureCapacity(-1);
-        }
-        catch (ArgumentOutOfRangeException)
-        {
-            Assert.True(true);
-            return;
-        }
-
-        Assert.False(true);
-    }
-
     [Fact]
     public void ShouldFindLastOccurence()
     {