test(barrier): cap qplib_8515_ruiz_forced_off at the iteration limit

[mlubin]

The test was flaky because sometimes it never converged. Enforce an iteration limit instead.

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[mlubin]

/ok to test e8d6a57

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: d4099b66-7e56-4d5f-b4b3-7c595e555343

📥 Commits

Reviewing files that changed from the base of the PR and between 74e3866 and e8d6a57.

📒 Files selected for processing (1)

• cpp/tests/dual_simplex/unit_tests/solve_barrier.cu

---

📝 Walkthrough

Walkthrough

Changes

Barrier solver test behavior

Layer / File(s)	Summary
Enforce and assert iteration limit cpp/tests/dual_simplex/unit_tests/solve_barrier.cu	The test disables Ruiz equilibration, sets iteration_limit to 50, and asserts IterationLimit termination instead of checking the step count.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: aliceb-nv, yuwenchen95, chris-maes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the test change to cap qplib_8515_ruiz_forced_off at the iteration limit.
Description check	✅ Passed	The description is directly related to the change and explains the test flakiness and iteration-limit fix.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[Bubullzz]

clean fix. thanks Miles

[github-actions]

CI Test Summary

✅ All 31 test job(s) passed.

[mlubin]

/merge