file-mode-api: remove_unnecesary absolut_path field from _get_file_transfer_paths response

Author: aldogonzalez8

airbyte_cdk/sources/file_based/file_based_stream_reader.py

@@ -33,7 +33,6 @@ class AbstractFileBasedStreamReader(ABC):
     FILE_RELATIVE_PATH = "file_relative_path"
     FILE_NAME = "file_name"
     LOCAL_FILE_PATH = "local_file_path"
-    ABSOLUTE_FILE_PATH = "absolute_file_path"
     SOURCE_FILE_URI = "source_file_relative_path"
     FILE_FOLDER = "file_folder"
 
@@ -190,12 +189,10 @@ def _get_file_transfer_paths(self, file: RemoteFile, local_directory: str) -> Mu
         local_file_path = path.join(local_directory, file_relative_path)
         # Ensure the local directory exists
         makedirs(path.dirname(local_file_path), exist_ok=True)
-        absolute_file_path = path.abspath(local_file_path)
 
         file_paths = {
             self.FILE_RELATIVE_PATH: file_relative_path,
             self.LOCAL_FILE_PATH: local_file_path,
-            self.ABSOLUTE_FILE_PATH: absolute_file_path,
             self.FILE_NAME: file_name,
             self.FILE_FOLDER: file_folder,
             self.SOURCE_FILE_URI: file_uri,

unit_tests/sources/file_based/test_file_based_stream_reader.py

@@ -384,7 +384,7 @@ def test_globs_and_prefixes_from_globs(
 
 
 @pytest.mark.parametrize(
-    "config, source_file, expected_file_relative_path, expected_local_file_path, expected_absolute_file_path",
+    "config, source_file, expected_file_relative_path, expected_local_file_path",
     [
         pytest.param(
             {
@@ -397,7 +397,6 @@ def test_globs_and_prefixes_from_globs(
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "/tmp/transfer-files/mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
-            "/tmp/transfer-files/mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             id="preserve_directories_present_and_true",
         ),
         pytest.param(
@@ -411,23 +410,20 @@ def test_globs_and_prefixes_from_globs(
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "monthly-kickoff-202402.mpeg",
             "/tmp/transfer-files/monthly-kickoff-202402.mpeg",
-            "/tmp/transfer-files/monthly-kickoff-202402.mpeg",
             id="preserve_directories_present_and_false",
         ),
         pytest.param(
             {"streams": [], "delivery_method": {"delivery_type": "use_file_transfer"}},
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "/tmp/transfer-files/mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
-            "/tmp/transfer-files/mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             id="preserve_directories_not_present_defaults_true",
         ),
         pytest.param(
             {"streams": []},
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             "/tmp/transfer-files/mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
-            "/tmp/transfer-files/mirror_paths_testing/not_duplicates/data/jan/monthly-kickoff-202402.mpeg",
             id="file_transfer_flag_not_present_defaults_true",
         ),
     ],
@@ -437,7 +433,6 @@ def test_preserve_sub_directories_scenarios(
     source_file: str,
     expected_file_relative_path: str,
     expected_local_file_path: str,
-    expected_absolute_file_path: str,
 ) -> None:
     remote_file = RemoteFile(
         uri=source_file,
@@ -452,7 +447,6 @@ def test_preserve_sub_directories_scenarios(
 
     assert file_paths[AbstractFileBasedStreamReader.FILE_RELATIVE_PATH] == expected_file_relative_path
     assert file_paths[AbstractFileBasedStreamReader.LOCAL_FILE_PATH] == expected_local_file_path
-    assert file_paths[AbstractFileBasedStreamReader.ABSOLUTE_FILE_PATH] == expected_absolute_file_path
     assert file_paths[AbstractFileBasedStreamReader.SOURCE_FILE_URI] == source_file
     assert file_paths[AbstractFileBasedStreamReader.FILE_NAME] == path.basename(source_file)
     assert file_paths[AbstractFileBasedStreamReader.FILE_FOLDER] == path.dirname(source_file)