From 67417b6ec8e7b7da2ba28d7037a6f3958ebaecde Mon Sep 17 00:00:00 2001
From: Jordan Dominion <Cyberboss@users.noreply.github.com>
Date: Mon, 18 Dec 2023 22:45:12 -0500
Subject: [PATCH] Nullify `TestMerge`

---
 src/Tgstation.Server.Host/Models/TestMerge.cs | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/Tgstation.Server.Host/Models/TestMerge.cs b/src/Tgstation.Server.Host/Models/TestMerge.cs
index 3733ca62fc..3641640378 100644
--- a/src/Tgstation.Server.Host/Models/TestMerge.cs
+++ b/src/Tgstation.Server.Host/Models/TestMerge.cs
@@ -1,8 +1,7 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using System.ComponentModel.DataAnnotations;
 
-#nullable disable
-
 namespace Tgstation.Server.Host.Models
 {
 	/// <inheritdoc cref="Api.Models.Internal.TestMergeApiBase" />
@@ -12,13 +11,13 @@ namespace Tgstation.Server.Host.Models
 		/// See <see cref="Api.Models.TestMerge.MergedBy"/>.
 		/// </summary>
 		[Required]
-		public User MergedBy { get; set; }
+		public User? MergedBy { get; set; }
 
 		/// <summary>
 		/// The initial <see cref="RevisionInformation"/> the <see cref="TestMerge"/> was merged with.
 		/// </summary>
 		[Required]
-		public RevisionInformation PrimaryRevisionInformation { get; set; }
+		public RevisionInformation? PrimaryRevisionInformation { get; set; }
 
 		/// <summary>
 		/// Foreign key for <see cref="PrimaryRevisionInformation"/>.
@@ -28,10 +27,10 @@ namespace Tgstation.Server.Host.Models
 		/// <summary>
 		/// All the <see cref="RevInfoTestMerge"/> for the <see cref="TestMerge"/>.
 		/// </summary>
-		public ICollection<RevInfoTestMerge> RevisonInformations { get; set; }
+		public ICollection<RevInfoTestMerge>? RevisonInformations { get; set; }
 
 		/// <inheritdoc />
-		public Api.Models.TestMerge ToApi() => new Api.Models.TestMerge
+		public Api.Models.TestMerge ToApi() => new()
 		{
 			Author = Author,
 			BodyAtMerge = BodyAtMerge,
@@ -39,7 +38,7 @@ namespace Tgstation.Server.Host.Models
 			TitleAtMerge = TitleAtMerge,
 			Comment = Comment,
 			Id = Id,
-			MergedBy = MergedBy.CreateUserName(),
+			MergedBy = (MergedBy ?? throw new InvalidOperationException("MergedBy must be set!")).CreateUserName(),
 			Number = Number,
 			TargetCommitSha = TargetCommitSha,
 			Url = Url,