Make Message required to get rid of null bang

sufyannisar · Oct 18, 2023 · 4c9eb43 · 4c9eb43
1 parent f9d4ebd
commit 4c9eb43
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Hangfire.Console/Serialization/ConsoleLine.cs b/src/Hangfire.Console/Serialization/ConsoleLine.cs
@@ -20,7 +20,7 @@ internal class ConsoleLine
     ///     Message text, or message reference, or progress bar id
     /// </summary>
     [JsonProperty("s", Required = Required.Always)]
-    public string Message { get; set; } = null!;
+    public required string Message { get; set; }
 
     /// <summary>
     ///     Text color for this message

diff --git a/tests/Hangfire.Console.Tests/Storage/ConsoleStorageFacts.cs b/tests/Hangfire.Console.Tests/Storage/ConsoleStorageFacts.cs
@@ -93,7 +93,7 @@ public void AddLine_ThrowsException_IfConsoleIdIsNull()
     {
         var storage = new ConsoleStorage(_connection.Object);
 
-        Assert.Throws<ArgumentNullException>("consoleId", () => storage.AddLine(null!, new ConsoleLine()));
+        Assert.Throws<ArgumentNullException>("consoleId", () => storage.AddLine(null!, new ConsoleLine{ Message = "" }));
     }
 
     [Fact]
@@ -109,7 +109,7 @@ public void AddLine_ThrowsException_IfLineIsReference()
     {
         var storage = new ConsoleStorage(_connection.Object);
 
-        Assert.Throws<ArgumentException>("line", () => storage.AddLine(_consoleId, new ConsoleLine { IsReference = true }));
+        Assert.Throws<ArgumentException>("line", () => storage.AddLine(_consoleId, new ConsoleLine { Message = "", IsReference = true }));
     }
 
     [Fact]