Merge branch 'fix-player-profile-binding-issue' into 'main'

[player] stop attaching the profile uuid to step before ensuring the profile is ready

See merge request platformsh/observability/blackfire/blackfire.io!36926

Player/Extension/BlackfireExtension.php

@@ -32,6 +32,7 @@
 use Blackfire\Player\Step\Step;
 use Blackfire\Player\Step\StepContext;
 use Blackfire\Player\Step\StepInitiatorInterface;
+use Blackfire\Profile;
 use Blackfire\Profile\Configuration as ProfileConfiguration;
 
 /**
@@ -210,8 +211,9 @@ public function afterStep(AbstractStep $step, StepContext $stepContext, Scenario
             }
 
             $parentStep = $step->getInitiator();
+            $profile = $this->blackfire->getProfile($uuid);
             $parentStep->setBlackfireProfileUuid($uuid);
-            $this->assertProfile($parentStep, $uuid);
+            $this->assertProfile($parentStep, $profile);
         }
     }
 
@@ -249,10 +251,8 @@ private function createBuildProfileConfig(Step $step, StepContext $stepContext,
         return $config;
     }
 
-    private function assertProfile(Step $step, string $blackfireProfileUuid): void
+    private function assertProfile(Step $step, Profile $profile): void
     {
-        $profile = $this->blackfire->getProfile($blackfireProfileUuid);
-
         if ($profile->isErrored()) {
             if ($profile->getTests()) {
                 throw new ExpectationErrorException('At least one assertion is invalid.');