Fix several small things found by copilot

Fixing assertion
Fixing wrong split in path
Fixing pytest global arguments

Author: MaximilianSoerenPollak

pyproject.toml

@@ -23,7 +23,6 @@ extend-exclude = [
     ".venv*/**",
 ]
 [tool.pytest.ini_options]
-addopts = ["-v", "-s"]
 log_cli = true
 log_cli_level = "Debug"
 log_cli_format = "[%(asctime)s.%(msecs)03d] [%(levelname)-3s] [%(name)s] %(message)s"

src/extensions/score_source_code_linker/tests/test_helpers.py

@@ -323,6 +323,7 @@ def test_get_github_link_with_real_repo(git_repo: Path) -> None:
     assert hash_from_link == actual_hash
 
 
+
 def test_complete_workflow(known_good_json: Path):
     """Test complete workflow from path to GitHub link."""
 

src/extensions/score_source_code_linker/tests/test_repo_source_link_integration.py

@@ -10,6 +10,11 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 # *******************************************************************************
+
+# ╓                                                          ╖
+# ║ Some portions generated by CoPilot                       ║
+# ╙                                                          ╜
+
 import contextlib
 import json
 import os
@@ -322,11 +327,7 @@ def test_repo_grouping_multiple_needs_per_repo(
 
         # All 3 MOD_REQ should be in this repo
         need_ids = {need.need for need in local_repo.needs}
-        assert (
-            "MOD_REQ_1" in need_ids
-            or "MOD_REQ_2" in need_ids
-            or "MOD_REQ_3" in need_ids
-        )
+        assert {"MOD_REQ_1", "MOD_REQ_2", "MOD_REQ_3"}.issubset(need_ids)
 
     finally:
         app.cleanup()

src/extensions/score_source_code_linker/tests/test_xml_parser.py

@@ -359,7 +359,7 @@ def test_clean_test_file_name_combo_path():
 def test_clean_test_file_name_tests_report_path():
     raw_path = Path("/some/path/tests-report/test-report/unit/test_module.xml")
     result = xml_parser.clean_test_file_name(raw_path)
-    assert result == Path("unit/test_module.xml")
+    assert result == Path("test-report/unit/test_module.xml")
 
 
 def test_clean_test_file_name_nested_bazel_testlogs():
@@ -371,14 +371,14 @@ def test_clean_test_file_name_nested_bazel_testlogs():
 def test_clean_test_file_name_invalid_path_raises_error():
     raw_path = Path("/invalid/path/without/markers/test.xml")
     with pytest.raises(
-        ValueError, match="Filepath does not have 'bazel-testlogs' nor 'test-report'"
+        ValueError, match="Filepath does not have 'bazel-testlogs' nor 'tests-report'"
     ):
         xml_parser.clean_test_file_name(raw_path)
 
 
 def test_clean_test_file_name_empty_path_raises_error():
     raw_path = Path("")
     with pytest.raises(
-        ValueError, match="Filepath does not have 'bazel-testlogs' nor 'test-report'"
+        ValueError, match="Filepath does not have 'bazel-testlogs' nor 'tests-report'"
     ):
         xml_parser.clean_test_file_name(raw_path)

src/extensions/score_source_code_linker/xml_parser.py

@@ -71,10 +71,10 @@ def clean_test_file_name(raw_filepath: Path) -> Path:
     if "bazel-testlogs" in str(raw_filepath):
         return Path(str(raw_filepath).split("bazel-testlogs/")[-1])
     if "tests-report" in str(raw_filepath):
-        return Path(str(raw_filepath).split("test-report/")[-1])
+        return Path(str(raw_filepath).split("tests-report/")[-1])
     raise ValueError(
         "Filepath does not have 'bazel-testlogs' nor "
-        f"'test-report'. Filepath: {raw_filepath}"
+        + f"'tests-report'. Filepath: {raw_filepath}"
     )