fix: change runner scope to function for target tests (#1752)

Co-authored-by: Edgar R. M <[email protected]>

samples/sample_target_csv/csv_target_sink.py

@@ -41,6 +41,7 @@ def process_batch(self, context: dict) -> None:
                 delimiter="\t",
                 quotechar='"',
                 quoting=csv.QUOTE_NONNUMERIC,
+                escapechar="\\",
             )
             for record in records_to_drain:
                 if newfile and not records_written:

samples/sample_target_parquet/parquet_target_sink.py

@@ -36,7 +36,8 @@ def _json_schema_to_arrow_fields(schema: dict[str, t.Any]) -> pa.StructType:
     for name, property_schema in schema.get("properties", {}).items():
         field = pa.field(name, _json_type_to_arrow_field(property_schema))
         fields.append(field)
-    return fields
+
+    return fields if fields else [pa.field("dummy", pa.string())]
 
 
 def _json_type_to_arrow_field(  # noqa: PLR0911
@@ -99,22 +100,3 @@ def process_batch(self, context: dict) -> None:
         table = pa.Table.from_pylist(records_to_drain, schema=schema)
         writer.write_table(table)
         writer.close()
-
-    @staticmethod
-    def translate_data_type(singer_type: str | dict) -> t.Any:
-        """Translate from singer_type to a native type."""
-        if singer_type in ["decimal", "float", "double"]:
-            return pa.decimal128
-        if singer_type in ["date-time"]:
-            return pa.datetime
-        if singer_type in ["date"]:
-            return pa.date64
-        return pa.string
-
-    def _get_parquet_schema(self) -> list[tuple[str, t.Any]]:
-        col_list: list[tuple[str, t.Any]] = []
-        for prop in self.schema["properties"]:
-            col_list.append(
-                (prop["name"], self.translate_data_type(prop["type"])),
-            )
-        return col_list

singer_sdk/testing/__init__.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 from .config import SuiteConfig
-from .factory import get_tap_test_class, get_target_test_class, get_test_class
+from .factory import get_tap_test_class, get_target_test_class
 from .legacy import (
     _get_tap_catalog,
     _select_all,
@@ -19,7 +19,6 @@
 __all__ = [
     "get_tap_test_class",
     "get_target_test_class",
-    "get_test_class",
     "_get_tap_catalog",
     "_select_all",
     "get_standard_tap_tests",