fix(personfit): make ignore guard attribute-order independent

Author: seonghobae

tests/test_scientific_test_execution_contract.py

@@ -6,27 +6,52 @@
 TARGET_TEST = "mc_500_reversed_respondent_flagged_by_u3"
 
 
-def _personfit_acceptance_is_ignored(source: str) -> bool:
-    """Detect the currently guarded ignored-test spelling."""
-    return f"#[test]\n#[ignore]\nfn {TARGET_TEST}()" in source
+def _personfit_acceptance_attributes(source: str) -> tuple[str, ...]:
+    """Return Rust attributes attached to the guarded Monte Carlo test."""
+    lines = source.splitlines()
+    target = f"fn {TARGET_TEST}() {{"
+    try:
+        target_index = next(
+            index for index, line in enumerate(lines) if line.strip() == target
+        )
+    except StopIteration as exc:
+        raise AssertionError(
+            "the deterministic person-fit Monte Carlo acceptance test is missing"
+        ) from exc
+
+    attributes: list[str] = []
+    index = target_index - 1
+    seen_attribute = False
+    while index >= 0:
+        stripped = lines[index].strip()
+        if stripped.startswith("#[") and stripped.endswith("]"):
+            attributes.append(stripped)
+            seen_attribute = True
+        elif seen_attribute and (not stripped or stripped.startswith("///")):
+            pass
+        else:
+            break
+        index -= 1
+
+    return tuple(reversed(attributes))
 
 
 def test_personfit_monte_carlo_acceptance_is_not_ignored() -> None:
     """Keep deterministic person-fit Monte Carlo acceptance in the normal Rust suite."""
     source = PERSONFIT_TESTS.read_text(encoding="utf-8")
-    active = f"#[test]\nfn {TARGET_TEST}()"
+    attributes = _personfit_acceptance_attributes(source)
 
-    assert not _personfit_acceptance_is_ignored(source), (
+    assert "#[test]" in attributes, (
+        "the deterministic person-fit Monte Carlo acceptance must remain a Rust test"
+    )
+    assert "#[ignore]" not in attributes, (
         "the deterministic 500-rep person-fit Monte Carlo acceptance is skipped; "
         "scientific acceptance must run rather than rely on #[ignore]"
     )
-    assert active in source, (
-        "the deterministic person-fit Monte Carlo acceptance must remain an active Rust test"
-    )
 
 
 def test_ignore_guard_is_attribute_order_independent() -> None:
     """Reject an ignore attribute even when it precedes the test attribute."""
     reordered = f"#[ignore]\n#[test]\nfn {TARGET_TEST}() {{}}\n"
 
-    assert _personfit_acceptance_is_ignored(reordered)
+    assert "#[ignore]" in _personfit_acceptance_attributes(reordered)