fix: Ineligible customers are now skipped properly (#64)

* Missing `return` statements

* Respect both `customer_ids`/`customer_id`from config when generating child contexts

tap_googleads/client.py

@@ -119,7 +119,9 @@ def end_date(self):
 
     @cached_property
     def customer_ids(self):
-        if self.config.get("customer_ids"):
-            return self.config["customer_ids"]
+        customer_ids = self.config.get("customer_ids")
+        if customer_ids is not None:
+            return customer_ids
 
-        return [self.config["customer_id"]] if self.config.get("customer_id") else []
+        customer_id = self.config.get("customer_id")
+        return customer_id and [customer_id]

tap_googleads/streams.py

@@ -108,7 +108,7 @@ def validate_response(self, response):
         super().validate_response(response)
 
     def generate_child_contexts(self, record, context):
-        customer_ids = self.config.get("customer_ids")
+        customer_ids = self.customer_ids
 
         if customer_ids is None:
             customer = record["customerClient"]
@@ -118,12 +118,14 @@ def generate_child_contexts(self, record, context):
                     "%s is a manager, skipping",
                     customer["clientCustomer"],
                 )
+                return
 
             if customer["status"] != "ENABLED":
                 self.logger.warning(
                     "%s is not enabled, skipping",
                     customer["clientCustomer"],
                 )
+                return
 
             customer_ids = {customer["id"]}