[#51] Fix leading and trailing space handling in oneline_parser and add corresponding test cases

Author: patdhlk

src/sphinx_codelinks/analyse/oneline_parser.py

@@ -54,7 +54,7 @@ def oneline_parser(  # noqa: PLR0912, PLR0911 # handel warnings
 
     # extract the string wrapped by start and end
     start_idx = start_idx + len(oneline_config.start_sequence)
-    string = oneline[start_idx:end_idx]
+    string = oneline[start_idx:end_idx].strip()
 
     # numbers of needs_fields which are required
     cnt_required_fields = oneline_config.get_cnt_required_fields()

tests/test_oneline_parser.py

@@ -25,6 +25,54 @@
                 "end_column": 17,
             },
         ),
+        # Test case for leading space after start sequence (bug fix)
+        (
+            f"@ title 1, IMPL_1 {UNIX_NEWLINE}",
+            {
+                "title": "title 1",
+                "id": "IMPL_1",
+                "type": "impl",
+                "links": [],
+                "start_column": 1,
+                "end_column": 18,
+            },
+        ),
+        # Test case for multiple leading spaces after start sequence
+        (
+            f"@   title 1, IMPL_1 {UNIX_NEWLINE}",
+            {
+                "title": "title 1",
+                "id": "IMPL_1",
+                "type": "impl",
+                "links": [],
+                "start_column": 1,
+                "end_column": 20,
+            },
+        ),
+        # Test case for trailing space before end sequence
+        (
+            f"@title 1, IMPL_1  {UNIX_NEWLINE}",
+            {
+                "title": "title 1",
+                "id": "IMPL_1",
+                "type": "impl",
+                "links": [],
+                "start_column": 1,
+                "end_column": 18,
+            },
+        ),
+        # Test case for both leading and trailing spaces
+        (
+            f"@  title 1, IMPL_1   {UNIX_NEWLINE}",
+            {
+                "title": "title 1",
+                "id": "IMPL_1",
+                "type": "impl",
+                "links": [],
+                "start_column": 1,
+                "end_column": 21,
+            },
+        ),
     ],
 )
 def test_oneline_parser_default_config_positive(