Skip eager_workflow test if the server doesn't support it on the Se…

…rverCapabilities level
temporalio · Oct 23, 2024 · de97b45 · de97b45
1 parent 0628cad
commit de97b45
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/features/eager_workflow/successful_start/feature.php b/features/eager_workflow/successful_start/feature.php
@@ -57,7 +57,7 @@ public function pipelineProvider(): PipelineProvider
 
     #[Check]
     public function check(
-        #[Stub('Workflow', eagerStart: true, )]
+        #[Stub('Workflow', eagerStart: true)]
         #[Client(timeout:30, pipelineProvider: [FeatureChecker::class, 'pipelineProvider'])]
         WorkflowStubInterface $stub,
     ): void {

diff --git a/harness/php/src/Feature/WorkflowStubInjector.php b/harness/php/src/Feature/WorkflowStubInjector.php
@@ -5,6 +5,7 @@
 namespace Harness\Feature;
 
 use Harness\Attribute\Stub;
+use Harness\Exception\SkipTest;
 use Harness\Runtime\Feature;
 use Psr\Container\ContainerInterface;
 use Spiral\Core\Attribute\Proxy;
@@ -39,6 +40,13 @@ public function createInjection(
 
         $client = $this->clientFactory->workflowClient($context);
 
+        if ($attribute->eagerStart) {
+            // If the server does not support eager start, skip the test
+            $client->getServiceClient()->getServerCapabilities()->eagerWorkflowStart or throw new SkipTest(
+                'Eager workflow start is not supported by the server.'
+            );
+        }
+
         /** @var Feature $feature */
         $feature = $this->container->get(Feature::class);
         $options = WorkflowOptions::new()