Convert scheduledTimeUtc to UTC in SignalEntityAsync (#1690)

convert scheduledTimeUtc to UTC in SignalEntityAsync, to make sure it is indeed UTC

Author: sebastianburckhardt

src/WebJobs.Extensions.DurableTask/ContextImplementations/DurableClient.cs

@@ -297,6 +297,11 @@ private async Task SignalEntityAsyncInternal(DurableClient durableClient, string
                 throw new ArgumentNullException(nameof(operationName));
             }
 
+            if (scheduledTimeUtc.HasValue)
+            {
+                scheduledTimeUtc = scheduledTimeUtc.Value.ToUniversalTime();
+            }
+
             if (this.ClientReferencesCurrentApp(durableClient))
             {
                 this.config.ThrowIfFunctionDoesNotExist(entityId.EntityName, FunctionType.Entity);

test/Common/DurableTaskEndToEndTests.cs

@@ -3446,9 +3446,11 @@ public async Task DurableEntity_ContinueAsNewBetweenCalls(bool extendedSessions)
         /// </summary>
         [Theory]
         [Trait("Category", PlatformSpecificHelpers.TestCategory)]
-        [InlineData(true)]
-        [InlineData(false)]
-        public async Task DurableEntity_ScheduledSignal(bool extendedSessions)
+        [InlineData(true, true)]
+        [InlineData(false, true)]
+        [InlineData(true, false)]
+        [InlineData(false, false)]
+        public async Task DurableEntity_ScheduledSignal(bool extendedSessions, bool useUtc)
         {
             using (var host = TestHelpers.GetJobHost(
                 this.loggerProvider,
@@ -3463,7 +3465,7 @@ public async Task DurableEntity_ScheduledSignal(bool extendedSessions)
                 // Wait 8 seconds to account for time to grab ownership lease.
                 await Task.Delay(8000);
 
-                var now = DateTime.UtcNow;
+                var now = useUtc ? DateTime.UtcNow : DateTime.Now;
 
                 await client.SignalEntity(this.output, now + TimeSpan.FromSeconds(4), "delayed", null);
                 await client.SignalEntity(this.output, "immediate", null);