Merge pull request #130 from dniccum/main

Undefined index within customer getByEmail method

Author: jswift

src/BigCommerce/Api/Customers/CustomersApi.php

@@ -33,7 +33,7 @@ public function getByEmail(string $email): ?Customer
     {
         $customers = $this->getAll([self::FILTER__EMAIL_IN => $email])->getCustomers();
 
-        if (!$customers[0]) {
+        if (count($customers) === 0) {
             return null;
         } elseif (count($customers) > 1) {
             throw new UnexpectedValueException("There are more than one customer with the email address $email");

tests/BigCommerce/Api/Customers/CustomersApiTest.php

@@ -38,6 +38,14 @@ public function testCanGetCustomerByEmail()
         $this->assertEquals('John', $customer->first_name);
     }
 
+    public function testUnknownCustomerReturnsEmptyResult()
+    {
+        $this->setReturnData('customers__get_all_no_results.json');
+        $customer = $this->getApi()->customers()->getByEmail('[email protected]');
+
+        $this->assertNull($customer);
+    }
+
     public function testCanGetCustomerById()
     {
         $this->setReturnData('customers__get_all.json');