changes based on review

Author: patelchandni

src/WebJobs.Script.WebHost/Management/FunctionsSyncManager.cs

@@ -807,15 +807,9 @@ public async Task<GetTriggersResult> GetTriggersPayloadAsync()
 
                 PrepareSyncTriggers();
 
-                var hashBlobClient = await GetHashBlobAsync();
-
                 var payload = await GetSyncTriggersPayload();
                 if (payload.Count == 0)
                 {
-                    // We don't do background sync for empty triggers.
-                    // We've seen error cases where a site temporarily gets into a situation
-                    // where it's site content is empty. Doing the empty sync can cause the app
-                    // to go idle when it shouldn't.
                     _logger.LogDebug("No functions found in GetTriggers operation.");
                 }
 

src/WebJobs.Script.WebHost/Management/GetTriggersResult.cs

@@ -8,17 +8,17 @@ namespace Microsoft.Azure.WebJobs.Script.WebHost.Management
     public class GetTriggersResult
     {
         /// <summary>
-        /// Gets or sets a value indicating whether the sync operation was successful.
+        /// Gets or sets a value indicating whether the operation was successful.
         /// </summary>
         public bool Success { get; set; }
 
         /// <summary>
-        /// Gets or sets the error string in the case of a failed sync operation.
+        /// Gets or sets the error string in the case of a failed operation.
         /// </summary>
         public string Error { get; set; }
 
         /// <summary>
-        /// Gets or sets the payload of the triggers.
+        /// Gets or sets the triggers payload.
         /// </summary>
         public SyncTriggersPayload Payload { get; set; }
     }

test/WebJobs.Script.Tests.Integration/Management/FunctionsSyncManagerTests.cs

@@ -257,18 +257,18 @@ private void ResetMockFileSystem(string hostJsonContent = null, string extension
         }
 
         [Fact]
-        public async Task GetSyncTriggersPayload_ExpectedContent()
+        public async Task GetTriggersPayload_ExpectedContent()
         {
             // Act
-            var syncResult = await _functionsSyncManager.GetTriggersPayloadAsync();
+            var result = await _functionsSyncManager.GetTriggersPayloadAsync();
 
             // Assert
-            Assert.True(syncResult.Success);
-            Assert.Null(syncResult.Error);
-            Assert.NotNull(syncResult.Payload);
+            Assert.True(result.Success);
+            Assert.Null(result.Error);
+            Assert.NotNull(result.Payload);
 
-            var result = JObject.Parse(syncResult.Payload.Content);
-            var triggers = result["triggers"];
+            var content = JObject.Parse(result.Payload.Content);
+            var triggers = content["triggers"];
             Assert.Equal(GetExpectedTriggersPayload(durableVersion: "V1"), triggers.ToString(Formatting.None));
         }