Enable barrier lp test

[hlinsen]

Tests should be enabled after closing: #519

Summary by CodeRabbit

• Tests
  • Re-enabled a previously skipped test related to barrier solver options, increasing test coverage and ensuring those options are validated during CI.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

A skip decorator was removed from the test_barrier_solver_options test in the test suite, so the test now executes with the same parameterization and assertions; no other logic or behavior was changed.

Changes

Cohort / File(s)	Summary
Test skip decorator removal python/cuopt_server/cuopt_server/tests/test_lp.py	Removed skip decorator from test_barrier_solver_options, enabling the test to run with existing parameterization and assertions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Quick review recommended: python/cuopt_server/cuopt_server/tests/test_lp.py to confirm no other decorators or conditional skips remain.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly and specifically describes the main change: enabling a previously skipped barrier solver options test in the LP test suite.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ae5be31 and 5c28385.

📒 Files selected for processing (1)

• python/cuopt_server/cuopt_server/tests/test_lp.py (0 hunks)

💤 Files with no reviewable changes (1)

• python/cuopt_server/cuopt_server/tests/test_lp.py

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (9)

• GitHub Check: wheel-build-cuopt-sh-client / 13.0.2, 3.10, amd64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.12, arm64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.13, arm64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.13, amd64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.10, arm64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.11, arm64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.11, amd64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.12, amd64, rockylinux8
• GitHub Check: wheel-build-cuopt-mps-parser / 13.0.2, 3.10, amd64, rockylinux8

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[anandhkb]

@hlinsen @tmckayus Please correct me if I am wrong, this is adding a new test which does not need to gate 25.12 release

@chris-maes @rg20 for viz

[hlinsen]

@hlinsen @tmckayus Please correct me if I am wrong, this is adding a new test which does not need to gate 25.12 release

@chris-maes @rg20 for viz

We are looking to evaluate if the issue is closed but it would not gate the release.

[github-actions]

🔔 Hi @anandhkb, this pull request has had no activity for 7 days. Please update or let us know if it can be closed. Thank you!

If this is an "epic" issue, then please add the "epic" label to this issue.
If it is a PR and not ready for review, then please convert this to draft.
If you just want to switch off this notification, then use the "skip inactivity reminder" label.

[github-actions]

🔔 Hi @anandhkb, this pull request has had no activity for 7 days. Please update or let us know if it can be closed. Thank you!

If this is an "epic" issue, then please add the "epic" label to this issue.
If it is a PR and not ready for review, then please convert this to draft.
If you just want to switch off this notification, then use the "skip inactivity reminder" label.