Check for empty strings for brandurl and background

Author: linkdotnet

src/LinkDotNet.Blog.Web/Shared/IntroductionCard.razor

@@ -14,5 +14,7 @@
     [Parameter]
     public Introduction Introduction { get; set; }
 
-    private string IntroductionClass => Introduction.BackgroundUrl != null ? "introduction-background" : string.Empty;
+    private string IntroductionClass => !string.IsNullOrEmpty(Introduction.BackgroundUrl)
+        ? "introduction-background"
+        : string.Empty;
 }

src/LinkDotNet.Blog.Web/Shared/NavMenu.razor

@@ -4,7 +4,7 @@
 
 <nav class="navbar navbar-expand-lg navbar-light w-100">
     <div class="container-fluid">
-        @if (configuration.BlogBrandUrl != null)
+        @if (!string.IsNullOrEmpty(configuration.BlogBrandUrl))
         {
             <a class="nav-brand ms-5" href="/">
                 <img style="max-height: 70px;"

tests/LinkDotNet.Blog.IntegrationTests/Web/Shared/NavMenuTests.cs

@@ -99,9 +99,30 @@ public void ShouldShowBrandImageIfAvailable()
         var cut = RenderComponent<NavMenu>();
 
         var brandImage = cut.Find(".nav-brand img");
-
         var image = brandImage.Unwrap() as IHtmlImageElement;
         image.Should().NotBeNull();
         image.Source.Should().Be("http://localhost/img.png");
     }
+
+    [Theory]
+    [InlineData(null)]
+    [InlineData("")]
+    public void ShouldShowBlogNameWhenNotBrand(string brandUrl)
+    {
+        var config = new AppConfiguration
+        {
+            ProfileInformation = new ProfileInformation(),
+            BlogBrandUrl = brandUrl,
+            BlogName = "Steven",
+        };
+        Services.AddScoped(_ => config);
+        this.AddTestAuthorization();
+
+        var cut = RenderComponent<NavMenu>();
+
+        var brandImage = cut.Find(".nav-brand");
+        var image = brandImage.Unwrap() as IHtmlAnchorElement;
+        image.Should().NotBeNull();
+        image.TextContent.Should().Be("Steven");
+    }
 }

tests/LinkDotNet.Blog.UnitTests/Web/Shared/IntroductionCardTests.cs

@@ -11,7 +11,7 @@ public class IntroductionCardTests : TestContext
     [Fact]
     public void ShouldSetBackgroundWhenSet()
     {
-        var introduction = new Introduction()
+        var introduction = new Introduction
         {
             BackgroundUrl = "something_but_null",
         };
@@ -24,4 +24,21 @@ public void ShouldSetBackgroundWhenSet()
         background.Should().HaveCount(1);
         background.Single().GetStyle().CssText.Should().Contain(introduction.BackgroundUrl);
     }
+
+    [Theory]
+    [InlineData(null)]
+    [InlineData("")]
+    public void ShouldNotSetBackgroundWhenNotSet(string url)
+    {
+        var introduction = new Introduction
+        {
+            BackgroundUrl = url,
+        };
+
+        var cut = RenderComponent<IntroductionCard>(
+            p => p.Add(s => s.Introduction, introduction));
+
+        var background = cut.FindAll(".introduction-background");
+        background.Should().BeNullOrEmpty();
+    }
 }