adjust property chunking splitting count and tests

Author: brianjlai

airbyte_cdk/sources/declarative/requesters/query_properties/property_chunking.py

@@ -52,9 +52,10 @@ def get_request_property_chunks(
         chunk_size = 0
         for property_field in property_fields:
             # If property_limit_type is not defined, we default to property_count which is just an incrementing count
-            # +2 for the comma ToDo: Add possibility to specify parameter representation and take it into account in property_field_size
+            # todo: Add ability to specify parameter delimiter representation and take into account in property_field_size
             property_field_size = (
-                len(property_field) + 2
+                len(property_field)
+                + 1  # The +1 represents the extra character for the delimiter in between properties
                 if self.property_limit_type == PropertyLimitType.characters
                 else 1
             )

unit_tests/sources/declarative/requesters/query_properties/test_property_chunking.py

@@ -43,10 +43,18 @@
             ["kate", "laurie", "jaclyn"],
             None,
             PropertyLimitType.characters,
-            10,
-            [["kate"], ["laurie"], ["jaclyn"]],
+            15,
+            [["kate", "laurie"], ["jaclyn"]],
             id="test_property_chunking_limit_characters",
         ),
+        pytest.param(
+            ["laurie", "jaclyn", "kaitlin"],
+            None,
+            PropertyLimitType.characters,
+            12,
+            [["laurie"], ["jaclyn"], ["kaitlin"]],
+            id="test_property_chunking_includes_extra_delimiter",
+        ),
         pytest.param(
             [],
             None,