fix: use 127.0.0.1 instead of localhost for temporal address

.github/workflows/run-test-suite.yml

@@ -95,6 +95,8 @@ jobs:
 
       - name: Run tests
         run: ${{ inputs.test-command }}
+        env:
+          XDEBUG_MODE: off
 
       - name: Check for failures
         if: steps.validate.outcome == 'failure'

src/Workflow/WorkflowExecutionInfo.php

@@ -96,4 +96,29 @@ public function __construct(
          */
         public readonly string $firstRunId,
     ) {}
+
+    public function __debugInfo(): array
+    {
+        return [
+            'execution' => $this->execution,
+            'type' => $this->type,
+            'startTime' => $this->startTime,
+            'closeTime' => $this->closeTime,
+            'status' => $this->status,
+            'historyLength' => $this->historyLength,
+            'parentNamespaceId' => $this->parentNamespaceId,
+            'parentExecution' => $this->parentExecution,
+            'executionTime' => $this->executionTime,
+            //            'memo' => $this->memo,
+            //            'searchAttributes' => $this->searchAttributes,
+            'autoResetPoints' => $this->autoResetPoints,
+            'taskQueue' => $this->taskQueue,
+            'stateTransitionCount' => $this->stateTransitionCount,
+            'historySizeBytes' => $this->historySizeBytes,
+            'mostRecentWorkerVersionStamp' => $this->mostRecentWorkerVersionStamp,
+            'executionDuration' => $this->executionDuration,
+            'rootExecution' => $this->rootExecution,
+            'firstRunId' => $this->firstRunId,
+        ];
+    }
 }

testing/README.md

@@ -139,7 +139,7 @@ workflow has a timer, the server doesn't wait for it and continues immediately.
 this behaviour you can use `TestService` class:
 
 ```php
-$testService = TestService::create('localhost:7233');
+$testService = TestService::create('127.0.0.1:7233');
 $testService->lockTimeSkipping();
 
 // ...
@@ -217,7 +217,7 @@ final class SimpleWorkflowTestCase extends TestCase
 
     protected function setUp(): void
     {
-        $this->workflowClient = new WorkflowClient(ServiceClient::create('localhost:7233'));
+        $this->workflowClient = new WorkflowClient(ServiceClient::create('127.0.0.1:7233'));
         $this->activityMocks = new ActivityMocker();
 
         parent::setUp();

testing/src/Command.php

@@ -9,8 +9,10 @@ final class Command
     /** @var non-empty-string|null Temporal Namespace */
     public ?string $namespace = null;
 
-    /** @var non-empty-string|null Temporal Address */
-    public ?string $address = null;
+    /**
+     * @var non-empty-string Temporal Address
+     */
+    public string $address;
 
     /** @var non-empty-string|null */
     public ?string $tlsKey = null;
@@ -20,12 +22,17 @@ final class Command
 
     private array $xdebug;
 
+    public function __construct(
+        string $address,
+    ) {
+        $this->address = $address;
+    }
+
     public static function fromEnv(): self
     {
-        $self = new self();
+        $self = new self(\getenv('TEMPORAL_ADDRESS') ?: '127.0.0.1:7233');
 
         $self->namespace = \getenv('TEMPORAL_NAMESPACE') ?: 'default';
-        $self->address = \getenv('TEMPORAL_ADDRESS') ?: 'localhost:7233';
         $self->xdebug = [
             'xdebug.mode' => \ini_get('xdebug.mode'),
             'xdebug.start_with_request' => \ini_get('xdebug.start_with_request'),
@@ -42,46 +49,37 @@ public static function fromEnv(): self
      */
     public static function fromCommandLine(array $argv): self
     {
-        $self = new self();
-
-        \array_shift($argv); // remove the script name (worker.php or runner.php)
-        foreach ($argv as $chunk) {
-            if (\str_starts_with($chunk, 'namespace=')) {
-                $self->namespace = \substr($chunk, 10);
-                continue;
-            }
-
-            if (\str_starts_with($chunk, 'address=')) {
-                $self->address = \substr($chunk, 8);
-                continue;
-            }
-
-            if (\str_starts_with($chunk, 'tls.cert=')) {
-                $self->tlsCert = \substr($chunk, 9);
-                continue;
-            }
-
-            if (\str_starts_with($chunk, 'tls.key=')) {
-                $self->tlsKey = \substr($chunk, 8);
-                continue;
+        $address = '';
+        $namespace = '';
+        $tlsCert = '';
+        $tlsKey = '';
+
+        // remove the script name (worker.php or runner.php)
+        $chunks = \array_slice($argv, 1);
+        foreach ($chunks as $chunk) {
+            switch (true) {
+                case \str_starts_with($chunk, 'namespace='):
+                    $namespace = \substr($chunk, 10);
+                    break;
+                case \str_starts_with($chunk, 'address='):
+                    $address = \substr($chunk, 8);
+                    break;
+                case \str_starts_with($chunk, 'tls.cert='):
+                    $tlsCert = \substr($chunk, 9);
+                    break;
+                case \str_starts_with($chunk, 'tls.key='):
+                    $tlsKey = \substr($chunk, 8);
+                    break;
             }
         }
 
-        return $self;
-    }
+        $self = new self($address);
 
-    /**
-     * @return list<non-empty-string> CLI arguments that can be parsed by `fromCommandLine`
-     */
-    public function getCommandLineArguments(): array
-    {
-        $result = [];
-        $this->namespace === null or $result[] = "namespace=$this->namespace";
-        $this->address === null or $result[] = "address=$this->address";
-        $this->tlsCert === null or $result[] = "tls.cert=$this->tlsCert";
-        $this->tlsKey === null or $result[] = "tls.key=$this->tlsKey";
+        $self->namespace = $namespace;
+        $self->tlsCert = $tlsCert;
+        $self->tlsKey = $tlsKey;
 
-        return $result;
+        return $self;
     }
 
     public function getPhpBinaryArguments(): array