Fix evict-on-finish change

Author: Sushisource

tests/Temporalio.Tests/Worker/WorkflowWorkerTests.cs

@@ -2997,6 +2997,10 @@ await ExecuteWorkerAsync<DynamicWorkflow>(
                 await handle.SignalAsync(wf => wf.SomeSignalAsync("signal arg"));
                 Assert.Equal("done", await handle.QueryAsync(wf => wf.SomeQuery("query arg")));
                 Assert.Equal("done", await handle.ExecuteUpdateAsync(wf => wf.SomeUpdateAsync("update arg")));
+                // Event list must be collected before the WF finishes, since when it finishes it
+                // will be evicted from the cache and the first query event will not exist upon
+                // replay.
+                var eventList = (await handle.QueryAsync(wf => wf.Events)).OrderBy(v => v).ToList();
                 await handle.SignalAsync(wf => wf.FinishAsync());
                 Assert.Equal("done", await handle.GetResultAsync());
                 Assert.Equal(
@@ -3008,7 +3012,7 @@ await ExecuteWorkerAsync<DynamicWorkflow>(
                         "update-SomeUpdate: update arg",
                         "workflow-NonDynamicWorkflow: workflow arg",
                     },
-                    (await handle.QueryAsync(wf => wf.Events)).OrderBy(v => v).ToList());
+                    eventList);
             },
             new TemporalWorkerOptions().AddActivity(DynamicWorkflow.DynamicActivity));
     }
@@ -6042,4 +6046,4 @@ private async Task<TResult> ExecuteWorkerAsync<TWorkflow, TResult>(
         using var worker = new TemporalWorker(client ?? Client, options);
         return await worker.ExecuteAsync(() => action(worker));
     }
-}
+}