From ae14c2239b6d1e5dfac29bc64861af43b5321bb7 Mon Sep 17 00:00:00 2001 From: Durable Workflow Date: Mon, 31 Aug 2026 15:23:04 +0000 Subject: [PATCH] Update activity tests for current workflow API --- tests/Feature/DeleteSnapshotActivityTest.php | 6 ++---- tests/Feature/InjectSandboxLossActivityTest.php | 6 ++---- tests/Feature/ProvisionSandboxActivityTest.php | 4 +--- tests/Feature/RestoreSandboxActivityTest.php | 6 ++---- tests/Feature/SnapshotSandboxActivityTest.php | 4 +--- 5 files changed, 8 insertions(+), 18 deletions(-) diff --git a/tests/Feature/DeleteSnapshotActivityTest.php b/tests/Feature/DeleteSnapshotActivityTest.php index e81de16..dc63179 100644 --- a/tests/Feature/DeleteSnapshotActivityTest.php +++ b/tests/Feature/DeleteSnapshotActivityTest.php @@ -11,8 +11,6 @@ use DurableWorkflow\AI\Tests\Fakes\Rc1SandboxProvider; use DurableWorkflow\AI\Tests\Fakes\StubSandboxProvider; use DurableWorkflow\AI\Tests\TestCase; -use Workflow\V2\Models\ActivityExecution; -use Workflow\V2\Models\WorkflowRun; use Workflow\V2\Support\ActivityRetryPolicy; final class DeleteSnapshotActivityTest extends TestCase @@ -24,7 +22,7 @@ public function test_deletion_is_idempotent_and_retries_without_a_finite_cleanup 'stub', static fn ($container, array $config): StubSandboxProvider => $provider, ); - $activity = new DeleteSnapshotActivity(new ActivityExecution, new WorkflowRun); + $activity = new DeleteSnapshotActivity; $activity->handle('snapshot-1', 'stub'); $activity->handle('snapshot-1', 'stub'); @@ -44,7 +42,7 @@ public function test_an_rc1_shaped_provider_loads_and_fails_at_capability_negoti static fn ($container, array $config): Rc1SandboxProvider => $provider, ); - $activity = new DeleteSnapshotActivity(new ActivityExecution, new WorkflowRun); + $activity = new DeleteSnapshotActivity; $this->expectException(UnsupportedSandboxCapabilityException::class); $this->expectExceptionMessage('does not support [snapshot_deletion]; the operation was not attempted'); diff --git a/tests/Feature/InjectSandboxLossActivityTest.php b/tests/Feature/InjectSandboxLossActivityTest.php index 3261772..25a86f4 100644 --- a/tests/Feature/InjectSandboxLossActivityTest.php +++ b/tests/Feature/InjectSandboxLossActivityTest.php @@ -10,8 +10,6 @@ use DurableWorkflow\AI\Tests\Fakes\StubSandboxProvider; use DurableWorkflow\AI\Tests\TestCase; use Workflow\Exceptions\NonRetryableException; -use Workflow\V2\Models\ActivityExecution; -use Workflow\V2\Models\WorkflowRun; use Workflow\V2\Support\ActivityRetryPolicy; final class InjectSandboxLossActivityTest extends TestCase @@ -23,7 +21,7 @@ public function test_retried_injection_reuses_the_identity_and_idempotent_local_ 'local', static fn ($container, array $config): StubSandboxProvider => $provider, ); - $activity = new InjectSandboxLossActivity(new ActivityExecution, new WorkflowRun); + $activity = new InjectSandboxLossActivity; $handle = (new SandboxHandle('sandbox-1', 'local'))->toArray(); $first = $activity->handle($handle, 'loss-injection-operation-1'); @@ -37,7 +35,7 @@ public function test_retried_injection_reuses_the_identity_and_idempotent_local_ public function test_injection_rejects_non_local_providers_before_destroy(): void { - $activity = new InjectSandboxLossActivity(new ActivityExecution, new WorkflowRun); + $activity = new InjectSandboxLossActivity; $this->expectException(NonRetryableException::class); $this->expectExceptionMessage('requires the [local] provider; [e2b] was selected'); diff --git a/tests/Feature/ProvisionSandboxActivityTest.php b/tests/Feature/ProvisionSandboxActivityTest.php index f944b24..727c95f 100644 --- a/tests/Feature/ProvisionSandboxActivityTest.php +++ b/tests/Feature/ProvisionSandboxActivityTest.php @@ -16,8 +16,6 @@ use InvalidArgumentException; use RuntimeException; use Workflow\Exceptions\NonRetryableException; -use Workflow\V2\Models\ActivityExecution; -use Workflow\V2\Models\WorkflowRun; final class ProvisionSandboxActivityTest extends TestCase { @@ -167,7 +165,7 @@ private function configureE2b(string $apiKey = 'e2b-test-key'): void private function activity(): ProvisionSandboxActivity { - return new ProvisionSandboxActivity(new ActivityExecution, new WorkflowRun); + return new ProvisionSandboxActivity; } private function extendManager(string $name, StubSandboxProvider $provider): void diff --git a/tests/Feature/RestoreSandboxActivityTest.php b/tests/Feature/RestoreSandboxActivityTest.php index 16ee6d3..521d28e 100644 --- a/tests/Feature/RestoreSandboxActivityTest.php +++ b/tests/Feature/RestoreSandboxActivityTest.php @@ -10,8 +10,6 @@ use DurableWorkflow\AI\Tests\Fakes\StubSandboxProvider; use DurableWorkflow\AI\Tests\TestCase; use RuntimeException; -use Workflow\V2\Models\ActivityExecution; -use Workflow\V2\Models\WorkflowRun; final class RestoreSandboxActivityTest extends TestCase { @@ -25,7 +23,7 @@ public function test_failed_initial_lease_renewal_destroys_the_restored_handle_w 'stub', static fn ($container, array $config): StubSandboxProvider => $provider, ); - $activity = new RestoreSandboxActivity(new ActivityExecution, new WorkflowRun); + $activity = new RestoreSandboxActivity; try { $activity->handle('snapshot-1', 'stub'); @@ -48,7 +46,7 @@ public function test_provider_handle_name_mismatch_is_cleaned_up_and_nonretryabl 'selected', static fn ($container, array $config): StubSandboxProvider => $provider, ); - $activity = new RestoreSandboxActivity(new ActivityExecution, new WorkflowRun); + $activity = new RestoreSandboxActivity; try { $activity->handle('snapshot-1', 'selected'); diff --git a/tests/Feature/SnapshotSandboxActivityTest.php b/tests/Feature/SnapshotSandboxActivityTest.php index b7eb119..2f96c61 100644 --- a/tests/Feature/SnapshotSandboxActivityTest.php +++ b/tests/Feature/SnapshotSandboxActivityTest.php @@ -9,8 +9,6 @@ use DurableWorkflow\AI\SandboxHandle; use DurableWorkflow\AI\Tests\Fakes\StubSandboxProvider; use DurableWorkflow\AI\Tests\TestCase; -use Workflow\V2\Models\ActivityExecution; -use Workflow\V2\Models\WorkflowRun; final class SnapshotSandboxActivityTest extends TestCase { @@ -21,7 +19,7 @@ public function test_snapshot_operation_identity_reaches_a_reconciling_provider( 'stub', static fn ($container, array $config): StubSandboxProvider => $provider, ); - $activity = new SnapshotSandboxActivity(new ActivityExecution, new WorkflowRun); + $activity = new SnapshotSandboxActivity; $snapshot = $activity->handle( (new SandboxHandle('sandbox-1', 'stub'))->toArray(),