Do not fail getCookies in Microsoft Edge if there are no cookies

Author: OndraM

CHANGELOG.md

@@ -18,6 +18,7 @@ This project versioning adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 - `WebDriverExpectedCondition::presenceOfElementLocated()` works correctly when used within `WebDriverExpectedCondition::not()`.
+- Improper behavior of Microsoft Edge when retrieving all cookies via `getCookies()` (it was causing fatal error  when there were no cookies).
 
 ## 1.7.1 - 2019-06-13
 ### Fixed

lib/WebDriverOptions.php

@@ -107,6 +107,10 @@ public function getCookieNamed($name)
                 [':name' => $name]
             );
 
+            if (!is_array($cookieArray)) { // Microsoft Edge returns null even in W3C mode => emulate proper behavior
+                throw new NoSuchCookieException('no such cookie');
+            }
+
             return Cookie::createFromArray($cookieArray);
         }
 
@@ -128,8 +132,11 @@ public function getCookieNamed($name)
     public function getCookies()
     {
         $cookieArrays = $this->executor->execute(DriverCommand::GET_ALL_COOKIES);
-        $cookies = [];
+        if (!is_array($cookieArrays)) { // Microsoft Edge returns null if there are no cookies...
+            return [];
+        }
 
+        $cookies = [];
         foreach ($cookieArrays as $cookieArray) {
             $cookies[] = Cookie::createFromArray($cookieArray);
         }