Avoid passing null literal to non-nullable reference type

Author: ianfnelson

tests/ServiceLayer.Mesh.Tests/Functions/FileDiscoveryFunctionTests.cs

@@ -1,4 +1,5 @@
-using Moq;
+using Microsoft.Azure.Functions.Worker;
+using Moq;
 using NHS.MESH.Client.Contracts.Services;
 using NHS.MESH.Client.Models;
 using ServiceLayer.Data.Models;
@@ -44,7 +45,7 @@ public async Task Run_AddsNewMessageToDbAndQueue()
             });
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         var meshFile = AssertFileUpdated(testMessageId, MeshFileStatus.Discovered);
@@ -67,7 +68,7 @@ public async Task Run_DoesNotAddDuplicateMessageOrQueueIt()
             });
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         var count = DbContext.MeshFiles.Count(f => f.FileId == existingFile.FileId);
@@ -87,7 +88,7 @@ public async Task Run_NoMessagesInInbox_DoesNothing()
             });
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         Assert.Empty(DbContext.MeshFiles);
@@ -107,7 +108,7 @@ public async Task Run_MultipleMessagesInInbox_AllAreProcessed()
             });
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         foreach (var id in messageIds)

tests/ServiceLayer.Mesh.Tests/Functions/FileRetryFunctionTests.cs

@@ -1,3 +1,4 @@
+using Microsoft.Azure.Functions.Worker;
 using Moq;
 using ServiceLayer.Data.Models;
 using ServiceLayer.Mesh.Functions;
@@ -38,7 +39,7 @@ public async Task Run_EnqueuesDiscoveredOrExtractingFilesOlderThan12Hours(MeshFi
         var file = SaveMeshFile(testStatus, 13);
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         _fileExtractQueueClientMock.Verify(q => q.EnqueueFileExtractAsync(It.Is<MeshFile>(f => f.FileId == file.FileId)), Times.Once);
@@ -56,7 +57,7 @@ public async Task Run_EnqueuesExtractedOrTransformingFilesOlderThan12Hours(MeshF
         var file = SaveMeshFile(testStatus, 13);
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         _fileTransformQueueClientMock.Verify(q => q.EnqueueFileTransformAsync(It.Is<MeshFile>(f => f.FileId == file.FileId)), Times.Once);
@@ -76,7 +77,7 @@ public async Task Run_SkipsFreshFiles(MeshFileStatus testStatus)
         var file = SaveMeshFile(testStatus);
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         _fileExtractQueueClientMock.Verify(q => q.EnqueueFileExtractAsync(It.IsAny<MeshFile>()), Times.Never);
@@ -95,7 +96,7 @@ public async Task Run_IgnoresFilesInOtherStatuses(MeshFileStatus ignoredStatus)
         SaveMeshFile(ignoredStatus, 20);
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         _fileTransformQueueClientMock.Verify(q => q.EnqueueFileTransformAsync(It.IsAny<MeshFile>()), Times.Never);
@@ -106,7 +107,7 @@ public async Task Run_IgnoresFilesInOtherStatuses(MeshFileStatus ignoredStatus)
     public async Task Run_IfNoFilesFoundDoNothing()
     {
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         _fileExtractQueueClientMock.Verify(q => q.EnqueueFileExtractAsync(It.IsAny<MeshFile>()), Times.Never);
@@ -122,7 +123,7 @@ public async Task Run_ProcessesMultipleEligibleFiles()
         var file3 = SaveMeshFile(MeshFileStatus.Transforming, 13);
 
         // Act
-        await _function.Run(null);
+        await _function.Run(new TimerInfo());
 
         // Assert
         _fileExtractQueueClientMock.Verify(q => q.EnqueueFileExtractAsync(It.Is<MeshFile>(f => f.FileId == file1.FileId)), Times.Once);