Skip to content

test: relocate LP rounding-probe tests to their own file#955

Merged
rasros merged 1 commit into
mainfrom
test/lp-package
Jul 3, 2026
Merged

test: relocate LP rounding-probe tests to their own file#955
rasros merged 1 commit into
mainfrom
test/lp-package

Conversation

@rasros

@rasros rasros commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

What changed

Follow-up to the LpBounding split (#953). lpRoundingProbe moved to LpPrimalHeuristics.kt, but its three tests were still in LpBoundingTest — a grab-bag mixing core-bounding and primal-heuristic tests. Relocate them into LpRoundingProbeTest, matching the LP suite's one-file-per-concern convention (ObjectiveShavingTest, FeasibilityPumpTest, LbTreeSearchTest, …).

  • LpBoundingTest keeps the core: LP bounding, node pruning, root-cut harvest, frequency policy, roundUpToResidue.
  • LpRoundingProbeTest holds the three probe tests (seed-and-preserve, conflict fallback, fractional relaxation); the small triangle()/sumObjective fixture the first one needs is copied over.

Why

Aligns the test files with the source split so the rounding-probe tests sit beside the code under test, and shrinks the LpBoundingTest grab-bag.

Testing

Tests moved verbatim; behavior unchanged. Full check lintDocs green across JVM + native + wasm.

@rasros rasros merged commit 14c2138 into main Jul 3, 2026
2 checks passed
@rasros rasros deleted the test/lp-package branch July 3, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant