feat: Removed unnecessary log warning

Author: alex-clayton-1

src/ServiceLayer.Mesh/Functions/FileExtractFunction.cs

@@ -30,13 +30,11 @@ public async Task Run([QueueTrigger("%FileExtractQueueName%")] FileExtractQueueM
         var file = await GetFileAsync(message.FileId);
         if (file == null)
         {
-            logger.LogWarning("Exiting function.");
             return;
         }
 
         if (!IsFileSuitableForExtraction(file))
         {
-            logger.LogWarning("Exiting function.");
             return;
         }
 

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

@@ -74,14 +74,6 @@ public async Task Run_FileNotFound_ExitsSilently()
                 null,
                 It.IsAny<Func<It.IsAnyType, Exception?, string>>()
             ), Times.Once);
-        _loggerMock.Verify(
-            x => x.Log(
-                LogLevel.Warning,
-                It.IsAny<EventId>(),
-                It.Is<It.IsAnyType>((v, t) => v.ToString() == "Exiting function."),
-                null,
-                It.IsAny<Func<It.IsAnyType, Exception?, string>>()
-            ), Times.Once);
 
         Assert.Equal(0, _dbContext.MeshFiles.Count());
         _meshInboxServiceMock.Verify(x => x.GetHeadMessageByIdAsync(It.IsAny<string>(), It.IsAny<string>()), Times.Never);
@@ -119,14 +111,6 @@ public async Task Run_FileStatusInvalid_ExitsSilently()
                 null,
                 It.IsAny<Func<It.IsAnyType, Exception?, string>>()
             ), Times.Once);
-        _loggerMock.Verify(
-            x => x.Log(
-                LogLevel.Warning,
-                It.IsAny<EventId>(),
-                It.Is<It.IsAnyType>((v, t) => v.ToString() == "Exiting function."),
-                null,
-                It.IsAny<Func<It.IsAnyType, Exception?, string>>()
-            ), Times.Once);
 
         _meshInboxServiceMock.Verify(x => x.GetHeadMessageByIdAsync(It.IsAny<string>(), It.IsAny<string>()), Times.Never);
         _blobStoreMock.Verify(x => x.UploadAsync(It.IsAny<MeshFile>(), It.IsAny<byte[]>()), Times.Never);
@@ -163,14 +147,6 @@ public async Task Run_FileStatusExtractingButNotTimedOut_ExitsSilently()
                 null,
                 It.IsAny<Func<It.IsAnyType, Exception?, string>>()
             ), Times.Once);
-        _loggerMock.Verify(
-            x => x.Log(
-                LogLevel.Warning,
-                It.IsAny<EventId>(),
-                It.Is<It.IsAnyType>((v, t) => v.ToString() == "Exiting function."),
-                null,
-                It.IsAny<Func<It.IsAnyType, Exception?, string>>()
-            ), Times.Once);
 
         _meshInboxServiceMock.Verify(x => x.GetHeadMessageByIdAsync(It.IsAny<string>(), It.IsAny<string>()), Times.Never);
         _blobStoreMock.Verify(x => x.UploadAsync(It.IsAny<MeshFile>(), It.IsAny<byte[]>()), Times.Never);