[player] ensure last jsonview is sent when last step of the last scen…

…ario errored

Player/Extension/BlackfireExtension.php

@@ -212,6 +212,9 @@ public function afterStep(AbstractStep $step, StepContext $stepContext, Scenario
 
             $parentStep = $step->getInitiator();
             $profile = $this->blackfire->getProfile($uuid);
+            // Initialize profile data by calling getUrl()
+            $profile->getUrl();
+
             $parentStep->setBlackfireProfileUuid($uuid);
             $this->assertProfile($parentStep, $profile);
         }

Player/Extension/ProcessedException.php

@@ -0,0 +1,16 @@
+<?php
+
+/*
+ * This file is part of the Blackfire Player package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Blackfire\Player\Extension;
+
+class ProcessedException extends \RuntimeException
+{
+}

Player/PlayerNext.php

@@ -14,6 +14,7 @@
 use Blackfire\Player\Enum\BuildStatus;
 use Blackfire\Player\Extension\ExceptionExtensionInterface;
 use Blackfire\Player\Extension\NextStepExtensionInterface;
+use Blackfire\Player\Extension\ProcessedException;
 use Blackfire\Player\Extension\ScenarioExtensionInterface;
 use Blackfire\Player\Extension\ScenarioSetExtensionInterface;
 use Blackfire\Player\Extension\StepExtensionInterface;
@@ -122,10 +123,8 @@ private function handleScenario(int|string $index, Scenario $scenario, ScenarioS
             );
         } catch (\Throwable $e) {
             $exception = $e;
-            foreach ($this->getSortedExtensions() as $extension) {
-                if ($extension instanceof ExceptionExtensionInterface) {
-                    $extension->failStep($this->currentStep, $e);
-                }
+            if ($e instanceof ProcessedException) {
+                $exception = $e->getPrevious();
             }
         }
 
@@ -211,6 +210,16 @@ private function handleStep(ConfigurableStep $step, StepContext $stepContext, Sc
                     }
                 }
             }
+        } catch (ProcessedException $exception) {
+            throw $exception;
+        } catch (\Throwable $exception) {
+            foreach ($this->getSortedExtensions() as $extension) {
+                if ($extension instanceof ExceptionExtensionInterface) {
+                    $extension->failStep($this->currentStep, $exception);
+                }
+            }
+
+            throw new ProcessedException($exception->getMessage(), $exception->getCode(), $exception);
         } finally {
             $step->setStatus(BuildStatus::DONE);
 

Player/Tests/Adapter/StubbedSdkAdapter.php

@@ -60,7 +60,16 @@ public function getProfile(string $uuid): Profile
             return ($this->profileFactory)($uuid);
         }
 
-        return new Profile(fn () => ['report' => ['state' => 'successful']], $uuid);
+        return new Profile(fn () => [
+            'report' => [
+                'state' => 'successful',
+            ],
+            '_links' => [
+                'graph_url' => [
+                    'href' => \sprintf('https://app.blackfire.io/profiles/%s/graph', $uuid),
+                ],
+            ],
+        ], $uuid);
     }
 
     public function startBuild(string|null $env = null, array $options = []): Build

Player/Tests/fixtures-run-with-env/profile-assertion-failure/stubbedSdkAdapter.php

@@ -26,6 +26,11 @@
                 ],
             ],
         ],
+        '_links' => [
+            'graph_url' => [
+                'href' => sprintf('https://app.blackfire.io/profiles/%s/graph', $uuid),
+            ],
+        ],
     ], $uuid);
 }
 );

composer.json

@@ -19,7 +19,7 @@
         "ext-filter": "*",
         "ext-intl": "*",
         "ext-mbstring": "*",
-        "blackfire/php-sdk": "^2.1 ",
+        "blackfire/php-sdk": "^2.5",
         "fakerphp/faker": "^1.10",
         "maltyxx/images-generator": "^1.0",
         "mtdowling/jmespath.php": "^2.2",