Add CTest labels for selective C++ test execution

[rgsl888prabhu]

Summary

• Add two CTest labels — routing and numopt — to all C++ test executables
• Extend ConfigureTest() CMake macro with optional LABELS keyword (backward compatible)

Label	Scope
routing	VRP, TSP, distance engine (9 tests)
numopt	LP, MIP, QP, dual simplex, MPS parser, gRPC, CLI (22 tests)

Usage

ctest -L routing    # only routing tests
ctest -L numopt     # only numerical optimization tests
ctest               # all (unchanged default)

Test plan

• cmake --build succeeds
• ctest -L routing runs only routing tests
• ctest -L numopt runs only numopt tests
• Full CI passes unchanged

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 17ad69a9-4b25-4f81-8f0c-9736a8bc21f3

📥 Commits

Reviewing files that changed from the base of the PR and between a0d4711 and ace3bb0.

📒 Files selected for processing (8)

• cpp/libmps_parser/tests/CMakeLists.txt
• cpp/tests/CMakeLists.txt
• cpp/tests/dual_simplex/CMakeLists.txt
• cpp/tests/linear_programming/CMakeLists.txt
• cpp/tests/linear_programming/grpc/CMakeLists.txt
• cpp/tests/mip/CMakeLists.txt
• cpp/tests/qp/CMakeLists.txt
• cpp/tests/utilities/CMakeLists.txt

✅ Files skipped from review due to trivial changes (1)

• cpp/libmps_parser/tests/CMakeLists.txt

🚧 Files skipped from review as they are similar to previous changes (6)

• cpp/tests/qp/CMakeLists.txt
• cpp/tests/utilities/CMakeLists.txt
• cpp/tests/linear_programming/grpc/CMakeLists.txt
• cpp/tests/dual_simplex/CMakeLists.txt
• cpp/tests/CMakeLists.txt
• cpp/tests/mip/CMakeLists.txt

---

📝 Walkthrough

Walkthrough

ConfigureTest CMake function gained optional LABELS parsing; many test registrations were updated to set CTest LABELS (mostly "numopt" or "routing"). No build/link/install behavior or source code logic was changed.

Changes

Cohort / File(s)	Summary
CMake test infrastructure cpp/tests/CMakeLists.txt	ConfigureTest() updated to parse optional LABELS keyword and use non-LABELS args as executable sources; when provided, sets CTest LABELS on the created test.
Parser tests cpp/libmps_parser/tests/CMakeLists.txt	Added set_tests_properties(MPS_PARSER_TEST PROPERTIES LABELS "numopt").
Dual simplex cpp/tests/dual_simplex/CMakeLists.txt	ConfigureTest(DUAL_SIMPLEX_TEST ...) updated to include LABELS numopt; SPDX year range updated.
Linear programming cpp/tests/linear_programming/CMakeLists.txt, cpp/tests/linear_programming/grpc/CMakeLists.txt	Multiple tests updated to include LABELS numopt; gRPC tests (GRPC_CLIENT_TEST, GRPC_PIPE_SERIALIZATION_TEST, GRPC_INTEGRATION_TEST) now have set_tests_properties(... PROPERTIES LABELS "numopt").
MIP tests cpp/tests/mip/CMakeLists.txt	All ConfigureTest(...) invocations were adjusted to add LABELS numopt.
QP tests cpp/tests/qp/CMakeLists.txt	ConfigureTest(QP_UNIT_TEST ...) modified to include LABELS numopt.
Routing & distance cpp/tests/routing/CMakeLists.txt, cpp/tests/distance_engine/CMakeLists.txt	Routing-related ConfigureTest(...) calls updated to include LABELS routing.
Utilities tests cpp/tests/utilities/CMakeLists.txt	ConfigureTest(CLI_TEST test_cli.cpp) updated to include LABELS numopt.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and concisely summarizes the main change: adding CTest labels to enable selective C++ test execution.
Description check	✅ Passed	The description clearly explains the purpose, scope, labels, usage examples, and test plan, all directly related to the changeset.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@ci/run_ctests.sh`:
- Around line 70-75: Token strings produced from CUOPT_TEST_LABELS and
test_labels (read into arrays requested/actual and iterated as req and act) may
contain leading/trailing whitespace, so trim both req and act before comparing
them to avoid false negatives; update the comparison in run_ctests.sh to
normalize (trim) whitespace from req and act (e.g., with bash parameter
expansion or a lightweight trim function) and then perform the equality check,
ensuring both sides are trimmed every loop.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 92d3fd29-a6cb-48f4-85d3-b5fb562102dd

📥 Commits

Reviewing files that changed from the base of the PR and between 9fe95f2 and 867c4c6.

📒 Files selected for processing (11)

• ci/run_ctests.sh
• cpp/libmps_parser/tests/CMakeLists.txt
• cpp/tests/CMakeLists.txt
• cpp/tests/distance_engine/CMakeLists.txt
• cpp/tests/dual_simplex/CMakeLists.txt
• cpp/tests/linear_programming/CMakeLists.txt
• cpp/tests/linear_programming/grpc/CMakeLists.txt
• cpp/tests/mip/CMakeLists.txt
• cpp/tests/qp/CMakeLists.txt
• cpp/tests/routing/CMakeLists.txt
• cpp/tests/utilities/CMakeLists.txt

[mlubin]

The label choices seem good to me. They will probably evolve a bit over time.

I'm not sure I understand the need for the extra code to generate a test_labels.txt manifest. ctest --show-only=json-v1 prints a json file with all the information we need to map tests to labels.

[rgsl888prabhu]

The label choices seem good to me. They will probably evolve a bit over time.

I'm not sure I understand the need for the extra code to generate a test_labels.txt manifest. ctest --show-only=json-v1 prints a json file with all the information we need to map tests to labels.

I was thinking to use this for testing so if we are missing any label from testing so it can be raised, but this would be helpful as well. Will remove it for now.