[DOP-21445] Add XML integration tests

Author: maxim-lixakov

syncmaster/dto/transfers.py

@@ -4,7 +4,7 @@
 from dataclasses import dataclass
 from typing import ClassVar
 
-from onetl.file.format import CSV, JSON, ORC, Excel, JSONLine, Parquet
+from onetl.file.format import CSV, JSON, ORC, XML, Excel, JSONLine, Parquet
 
 
 @dataclass
@@ -20,7 +20,7 @@ class DBTransferDTO(TransferDTO):
 @dataclass
 class FileTransferDTO(TransferDTO):
     directory_path: str
-    file_format: CSV | JSONLine | JSON | Excel | ORC | Parquet
+    file_format: CSV | JSONLine | JSON | Excel | XML | ORC | Parquet
     options: dict
     df_schema: dict | None = None
 
@@ -31,6 +31,7 @@ class FileTransferDTO(TransferDTO):
         "excel": Excel,
         "orc": ORC,
         "parquet": Parquet,
+        "xml": XML,
     }
 
     def __post_init__(self):

syncmaster/worker/spark.py

@@ -36,8 +36,14 @@ def get_worker_spark_session(
 
 
 def get_packages(db_type: str) -> list[str]:
+    import pyspark
     from onetl.connection import MSSQL, Clickhouse, MySQL, Oracle, Postgres, SparkS3
-    from onetl.file.format import Excel
+    from onetl.file.format import XML, Excel
+
+    # see supported versions from https://mvnrepository.com/artifact/com.crealytics/spark-excel
+    file_formats_spark_packages: list[str] = XML.get_packages(spark_version=pyspark.__version__) + Excel.get_packages(
+        spark_version="3.5.1",
+    )
 
     if db_type == "postgres":
         return Postgres.get_packages()
@@ -55,10 +61,11 @@ def get_packages(db_type: str) -> list[str]:
 
         spark_version = pyspark.__version__
         # see supported versions from https://mvnrepository.com/artifact/com.crealytics/spark-excel
-        return SparkS3.get_packages(spark_version=spark_version) + Excel.get_packages(spark_version="3.5.1")
+
+        return SparkS3.get_packages(spark_version=spark_version) + file_formats_spark_packages
     if db_type == "hdfs":
         # see supported versions from https://mvnrepository.com/artifact/com.crealytics/spark-excel
-        return Excel.get_packages(spark_version="3.5.1")
+        return file_formats_spark_packages
 
     # If the database type does not require downloading .jar packages
     return []

tests/test_integration/test_run_transfer/conftest.py

@@ -11,7 +11,7 @@
 from onetl.connection import MSSQL, Clickhouse, Hive, MySQL, Oracle, Postgres, SparkS3
 from onetl.connection.file_connection.s3 import S3
 from onetl.db import DBWriter
-from onetl.file.format import CSV, JSON, ORC, Excel, JSONLine, Parquet
+from onetl.file.format import CSV, JSON, ORC, XML, Excel, JSONLine, Parquet
 from pyspark.sql import DataFrame, SparkSession
 from pyspark.sql.types import (
     DateType,
@@ -115,7 +115,10 @@ def spark(settings: Settings, request: FixtureRequest) -> SparkSession:
 
     if "hdfs" in markers or "s3" in markers:
         # see supported versions from https://mvnrepository.com/artifact/com.crealytics/spark-excel
-        maven_packages.extend(Excel.get_packages(spark_version="3.5.1"))
+        file_formats_spark_packages: list[str] = XML.get_packages(
+            spark_version=pyspark.__version__,
+        ) + Excel.get_packages(spark_version="3.5.1")
+        maven_packages.extend(file_formats_spark_packages)
 
     if maven_packages:
         spark = spark.config("spark.jars.packages", ",".join(maven_packages))
@@ -804,7 +807,9 @@ def fill_with_data(df: DataFrame):
         pass
 
 
-@pytest.fixture()
+@pytest.fixture(
+    params=[("csv", {}), ("jsonline", {}), ("json", {}), ("excel", {}), ("xml", {}), ("orc", {}), ("parquet", {})],
+)
 def source_file_format(request: FixtureRequest):
     name, params = request.param
     if name == "csv":
@@ -845,6 +850,12 @@ def source_file_format(request: FixtureRequest):
             **params,
         )
 
+    if name == "xml":
+        return "xml", XML(
+            row_tag="item",
+            **params,
+        )
+
     raise ValueError(f"Unsupported file format: {name}")
 
 
@@ -883,6 +894,12 @@ def target_file_format(request: FixtureRequest):
             **params,
         )
 
+    if name == "xml":
+        return "xml", XML(
+            row_tag="item",
+            **params,
+        )
+
     raise ValueError(f"Unsupported file format: {name}")
 
 

tests/test_integration/test_run_transfer/test_hdfs.py

@@ -139,6 +139,11 @@ async def postgres_to_hdfs(
             "without_compression",
             id="parquet",
         ),
+        pytest.param(
+            ("xml", {}),
+            "without_compression",
+            id="xml",
+        ),
     ],
     indirect=["source_file_format", "file_format_flavor"],
 )
@@ -223,6 +228,11 @@ async def test_run_transfer_hdfs_to_postgres(
             "with_compression",
             id="parquet",
         ),
+        pytest.param(
+            ("xml", {}),
+            "without_compression",
+            id="xml",
+        ),
     ],
     indirect=["target_file_format", "file_format_flavor"],
 )

tests/test_integration/test_run_transfer/test_s3.py

@@ -130,15 +130,20 @@ async def postgres_to_s3(
             id="excel",
         ),
         pytest.param(
-            ("orc", {}),
-            "without_compression",
+            ("orc", {"compression": "none"}),
+            "with_compression",
             id="orc",
         ),
         pytest.param(
-            ("parquet", {}),
-            "without_compression",
+            ("parquet", {"compression": "gzip"}),
+            "with_compression",
             id="parquet",
         ),
+        pytest.param(
+            ("xml", {}),
+            "without_compression",
+            id="xml",
+        ),
     ],
     indirect=["source_file_format", "file_format_flavor"],
 )
@@ -224,6 +229,11 @@ async def test_run_transfer_s3_to_postgres(
             "with_compression",
             id="parquet",
         ),
+        pytest.param(
+            ("xml", {}),
+            "without_compression",
+            id="xml",
+        ),
     ],
     indirect=["target_file_format", "file_format_flavor"],
 )