[cherry-pick-me][chore]: resolve pytest warnings undeclared marks, and for classes starting with "Test"

Author: aaronsteers

airbyte_cdk/connector_builder/connector_builder_handler.py

@@ -37,6 +37,8 @@
 
 @dataclass
 class TestLimits:
+    __test__: bool = False  # Prevent pytest from treating this as a test case, despite its name
+
     max_records: int = field(default=DEFAULT_MAXIMUM_RECORDS)
     max_pages_per_slice: int = field(default=DEFAULT_MAXIMUM_NUMBER_OF_PAGES_PER_SLICE)
     max_slices: int = field(default=DEFAULT_MAXIMUM_NUMBER_OF_SLICES)

airbyte_cdk/connector_builder/test_reader/reader.py

@@ -66,6 +66,8 @@ class TestReader:
 
     """
 
+    __test__: bool = False  # Prevent pytest from treating this as a test case, despite its name
+
     logger = logging.getLogger("airbyte.connector-builder")
 
     def __init__(

pytest.ini

@@ -5,3 +5,6 @@ log_cli_format = %(asctime)s [%(levelname)8s] %(message)s (%(filename)s:%(lineno
 log_cli_date_format=%Y-%m-%d %H:%M:%S
 filterwarnings =
     ignore::airbyte_cdk.sources.source.ExperimentalClassWarning
+markers =
+    slow: mark tests as slow
+    asyncio: mark test as asyncio

unit_tests/sources/declarative/decoders/test_composite_decoder.py

@@ -259,6 +259,8 @@ def test_composite_raw_decoder_csv_parser_values(requests_mock, encoding: str, d
 
 
 class TestServer(BaseHTTPRequestHandler):
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     def do_GET(self) -> None:
         self.send_response(200)
         self.end_headers()

unit_tests/sources/declarative/parsers/testing_components.py

@@ -21,6 +21,8 @@ class TestingSomeComponent(DefaultErrorHandler):
     A basic test class with various field permutations used to test manifests with custom components
     """
 
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     subcomponent_field_with_hint: DpathExtractor = field(
         default_factory=lambda: DpathExtractor(field_path=[], config={}, parameters={})
     )
@@ -37,5 +39,7 @@ class TestingCustomSubstreamPartitionRouter(SubstreamPartitionRouter):
     A test class based on a SubstreamPartitionRouter used for testing manifests that use custom components.
     """
 
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     custom_field: str
     custom_pagination_strategy: PaginationStrategy

unit_tests/sources/declarative/test_yaml_declarative_source.py

@@ -133,6 +133,8 @@ def test_source_with_missing_reference_fails(self):
 
 
 class TestFileContent:
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     def __init__(self, content):
         self.file = tempfile.NamedTemporaryFile(mode="w", delete=False)
 

unit_tests/sources/file_based/scenarios/scenario_builder.py

@@ -42,6 +42,8 @@ def build(
 
 
 class TestScenario(Generic[SourceType]):
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     def __init__(
         self,
         name: str,

unit_tests/sources/file_based/test_file_based_stream_reader.py

@@ -62,6 +62,8 @@
 
 
 class TestStreamReader(AbstractFileBasedStreamReader):
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     @property
     def config(self) -> Optional[AbstractFileBasedSpec]:
         return self._config
@@ -100,6 +102,8 @@ def identities_schema(self) -> Dict[str, Any]:
 
 
 class TestSpec(AbstractFileBasedSpec):
+    __test__ = False  # Prevent pytest from thinking that this is a test class, despite the name
+
     @classmethod
     def documentation_url(cls) -> AnyUrl:
         return AnyUrl(scheme="https", url="https://docs.airbyte.com/integrations/sources/test")  # type: ignore