fix: more precise checks after webdriver response deserialization

Author: itnelo

src/Client/W3CClient.php

@@ -277,6 +277,10 @@ function (ResponseInterface $response) use ($tabLookupDeferred) {
                 try {
                     $tabIdentifiers = $this->deserializeResponse($response);
 
+                    if (!is_array($tabIdentifiers)) {
+                        throw new RuntimeException('Unable to locate tab identifiers in the response.');
+                    }
+
                     $tabLookupDeferred->resolve($tabIdentifiers);
                 } catch (Throwable $exception) {
                     $reason = new RuntimeException(
@@ -323,6 +327,10 @@ public function getActiveTabIdentifier(string $sessionIdentifier): PromiseInterf
                 function (ResponseInterface $response) {
                     $tabIdentifier = $this->deserializeResponse($response);
 
+                    if (!is_string($tabIdentifier)) {
+                        throw new RuntimeException('Unable to locate a tab identifier in the response.');
+                    }
+
                     return $tabIdentifier;
                 }
             )
@@ -438,6 +446,10 @@ public function getCurrentUri(string $sessionIdentifier): PromiseInterface
                 function (ResponseInterface $response) {
                     $uriCurrent = $this->deserializeResponse($response);
 
+                    if (!is_string($uriCurrent)) {
+                        throw new RuntimeException('Unable to locate an URI in the response.');
+                    }
+
                     return $uriCurrent;
                 }
             )
@@ -475,6 +487,10 @@ public function getSource(string $sessionIdentifier): PromiseInterface
                 function (ResponseInterface $response) {
                     $sourceCode = $this->deserializeResponse($response);
 
+                    if (!is_string($sourceCode)) {
+                        throw new RuntimeException('Unable to locate source code in the response.');
+                    }
+
                     return $sourceCode;
                 }
             )
@@ -599,6 +615,10 @@ public function getElementVisibility(string $sessionIdentifier, array $elementId
                 function (ResponseInterface $response) {
                     $visibilityStatus = $this->deserializeResponse($response);
 
+                    if (!is_bool($visibilityStatus)) {
+                        throw new RuntimeException('Unable to locate a visibility status in the response.');
+                    }
+
                     return $visibilityStatus;
                 }
             )
@@ -778,7 +798,12 @@ public function getScreenshot(string $sessionIdentifier): PromiseInterface
             ->then(
                 function (ResponseInterface $response) {
                     $imageContentsEncoded = $this->deserializeResponse($response);
-                    $imageContents        = base64_decode($imageContentsEncoded);
+
+                    if (!is_string($imageContentsEncoded)) {
+                        throw new RuntimeException('Unable to locate screenshot contents in the response.');
+                    }
+
+                    $imageContents = base64_decode($imageContentsEncoded);
 
                     return $imageContents;
                 }