Add overload to ContainerSpan.Add() to accept a string which becomes …

…a ContentSpan (dotnet#908)
QuentinPom · May 20, 2020 · 425cf56 · 425cf56
1 parent d252893
commit 425cf56
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/src/System.CommandLine.Rendering.Tests/ContainerSpanTests.cs b/src/System.CommandLine.Rendering.Tests/ContainerSpanTests.cs
@@ -17,5 +17,14 @@ public void Container_span_supports_add_method()
 
             span.ContentLength.Should().Be("content with child".Length);
         }
+
+        [Fact]
+        public void Container_span_supports_add_string_method()
+        {
+            var span = new ContainerSpan(new ContentSpan("content"));
+            span.Add(" with string");
+
+            span.ContentLength.Should().Be("content with string".Length);
+        }
     }
 }
diff --git a/src/System.CommandLine.Rendering/ContainerSpan.cs b/src/System.CommandLine.Rendering/ContainerSpan.cs
@@ -72,5 +72,17 @@ public void Add(TextSpan child)
 
             RecalculateChildPositions();
         }
+
+        public void Add(string text)
+        {
+            if (string.IsNullOrEmpty(text))
+            {
+                return;
+            }
+
+            _children.Add(new ContentSpan(text));
+
+            RecalculateChildPositions();
+        }
     }
 }