Merge pull request #25 from cloudblue/LITE-27338-fix-report-filename

LITE-27338 fix report filename

Author: ffaraone

executor/utils.py

@@ -108,12 +108,13 @@ def get_user_agent():
 
 def upload_file(client, report_name, report_id, owner_id):
     report_filename = os.path.basename(report_name)
+    _, report_extension = report_filename.rsplit('.', 1)
     reports_media_api = client.ns('media').ns('folders').collection('reports_report_file')
     media_file = reports_media_api[owner_id].action('files').post(
         data=open(report_name, 'rb'),
         headers={
             'Content-Type': 'application/octet-stream',
-            'Content-Disposition': f'attachment; filename="{report_filename}"',
+            'Content-Disposition': f'attachment; filename="{report_id}.{report_extension}"',
         },
     )
 

tests/test_utils.py

@@ -5,6 +5,7 @@
 import pytest
 from connect.client import ConnectClient
 from pkg_resources import DistributionNotFound
+from responses import matchers
 
 from executor.exceptions import RunnerException
 from executor.utils import (
@@ -102,6 +103,10 @@ def test_upload_file(mocked_responses, fs):
         url='https://localhost/public/v1/media/folders/reports_report_file/VA-001/files',
         status=201,
         body=b'{"id": "MFL-001"}',
+        match=[matchers.header_matcher({
+            'Content-Type': 'application/octet-stream',
+            'Content-Disposition': 'attachment; filename="REC-000-000-0000-000000.zip"',
+        })],
     )
 
     mocked_responses.add(