fix test

Author: Konboi

smart_tests/commands/record/attachment.py

@@ -53,7 +53,7 @@ def attachment(
                         if zip_info.is_dir():
                             continue
 
-                        if not matches_include_patterns(zip_info.filename, include_patterns):
+                        if not matches_include_patterns(zip_info.filename, tuple(include_patterns)):
                             continue
 
                         file_content = zip_file.read(zip_info.filename)
@@ -80,7 +80,7 @@ def attachment(
                         if tar_info.isdir():
                             continue
 
-                        if not matches_include_patterns(tar_info.name, include_patterns):
+                        if not matches_include_patterns(tar_info.name, tuple(include_patterns)):
                             continue
 
                         file_obj = tar_file.extractfile(tar_info)

smart_tests/test_runners/maven.py

@@ -169,12 +169,15 @@ def record_tests(
     )],
 ):
     client.path_builder = junit5_nested_class_path_builder(client.path_builder)
+
+    # Override report method to filter out non-JUnit report files
     original_report = client.report
+
     IGNORED_FILES = {'failsafe-summary.xml', 'testng-results.xml'}
 
     def report_with_filter(junit_report_file: str):
         if not any(junit_report_file.endswith(f) for f in IGNORED_FILES):
             original_report(junit_report_file)
 
-    client.report = report_with_filter
+    client.report = report_with_filter  # type: ignore[method-assign]
     smart_tests.CommonRecordTestImpls.load_report_files(client=client, source_roots=reports)

tests/commands/record/test_attachment.py

@@ -7,7 +7,6 @@
 
 import responses  # type: ignore
 
-from build.lib.smart_tests.utils.session import write_session
 from smart_tests.utils.http_client import get_base_url
 from tests.cli_test_case import CliTestCase
 
@@ -161,7 +160,7 @@ def test_attachment_with_include_filter(self):
             self.assertEqual(expect, result.output)
 
     @responses.activate
-    @mock.patch.dict(os.environ, {"LAUNCHABLE_TOKEN": CliTestCase.launchable_token})
+    @mock.patch.dict(os.environ, {"SMART_TESTS_TOKEN": CliTestCase.smart_tests_token})
     def test_attachment_with_identical_file_names(self):
         with tempfile.TemporaryDirectory() as temp_dir:
             # Create temporary files
@@ -216,13 +215,10 @@ def test_attachment_with_identical_file_names(self):
             self.assertEqual(expect, result.output)
 
     @responses.activate
-    @mock.patch.dict(os.environ, {"LAUNCHABLE_TOKEN": CliTestCase.launchable_token})
+    @mock.patch.dict(os.environ, {"SMART_TESTS_TOKEN": CliTestCase.smart_tests_token})
     def test_attachment_with_whitespace_in_filename(self):
         TEST_CONTENT = b"Test log content"
 
-        # emulate launchable record build & session
-        write_session(self.build_name, self.session_id)
-
         # Create a file with whitespace in the name
         with tempfile.TemporaryDirectory() as temp_dir:
             file_path = os.path.join(temp_dir, "app log file.txt")
@@ -245,7 +241,7 @@ def test_attachment_with_whitespace_in_filename(self):
             self.assertIn("✓ Recorded successfully", result.output)
 
     @responses.activate
-    @mock.patch.dict(os.environ, {"LAUNCHABLE_TOKEN": CliTestCase.launchable_token})
+    @mock.patch.dict(os.environ, {"SMART_TESTS_TOKEN": CliTestCase.smart_tests_token})
     def test_attachment_duplicate_file_paths(self):
         with tempfile.TemporaryDirectory() as temp_dir:
             # Create temporary files