Sync DeduplicationTest with Harness repo

temporalio · Jan 20, 2025 · 4c03aea · 4c03aea
1 parent fffcc39
commit 4c03aea
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/tests/Acceptance/Extra/Workflow/AllHandlersFinishedTest.php b/tests/Acceptance/Extra/Workflow/AllHandlersFinishedTest.php
@@ -29,7 +29,7 @@ public function updateHandlersWithOneCall(
         $resolver = $stub->startUpdate('resolve', "key", "resolved");
 
         // Should be completed after the previous operation
-        $result = $stub->getResult(timeout: 1);
+        $result = $stub->getResult();
 
         $this->assertSame(['key' => 'resolved'], (array) $result, 'Workflow result contains resolved value');
         $this->assertFalse($handle->hasResult());

diff --git a/tests/Acceptance/Harness/Update/DeduplicationTest.php b/tests/Acceptance/Harness/Update/DeduplicationTest.php
@@ -70,7 +70,7 @@ class FeatureWorkflow
     #[WorkflowMethod('Harness_Update_Deduplication')]
     public function run()
     {
-        yield Workflow::await(fn(): bool => $this->counter >= 2);
+        yield Workflow::await(fn(): bool => Workflow::allHandlersFinished());
         return $this->counter;
     }