Ensure only full invocations are measured (#10177) (#10178)

Author: mathewc

src/WebJobs.Script.WebHost/Metrics/FlexConsumptionMetricsPublisher.cs

@@ -268,6 +268,13 @@ internal void OnFunctionCompleted(string functionName, string invocationId, Date
                 {
                     ActiveFunctionCount--;
                 }
+                else
+                {
+                    // We got a completion event without a corresponding start.
+                    // This might happen during specialization for example.
+                    // Ignore the event.
+                    return;
+                }
 
                 if (ActiveFunctionCount == 0)
                 {

test/WebJobs.Script.Tests/Metrics/FlexConsumptionMetricsPublisherTests.cs

@@ -437,6 +437,27 @@ public void FunctionsStartStop_MinimumActivityIntervals_Scenario3()
             Assert.Equal(4800, publisher.FunctionExecutionTimeMS);
         }
 
+        [Fact]
+        public void OnFunctionCompleted_NoOutstandingInvocations_IgnoresEvent()
+        {
+            var publisher = CreatePublisher(metricsPublishInterval: TimeSpan.FromHours(1), inStandbyMode: false);
+
+            Assert.Equal(1000, _options.MinimumActivityIntervalMS);
+            Assert.Equal(0, publisher.ActiveFunctionCount);
+            Assert.Equal(0, publisher.FunctionExecutionCount);
+            Assert.Equal(0, publisher.FunctionExecutionTimeMS);
+
+            DateTime now = DateTime.UtcNow;
+
+            // send a function completion without a corresponding start event
+            now += TimeSpan.FromMilliseconds(300);
+            publisher.OnFunctionCompleted("foo", "1", now);
+
+            Assert.Equal(0, publisher.ActiveFunctionCount);
+            Assert.Equal(0, publisher.FunctionExecutionCount);
+            Assert.Equal(0, publisher.FunctionExecutionTimeMS);
+        }
+
         public void CleanupMetricsFiles()
         {
             var directory = new DirectoryInfo(_metricsFilePath);