You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Update the vcpkg consumer integration test to use REQUIRED for all find_package() calls so CI fails when a package cannot be found, instead of silently skipping.
Current: Only common_system uses REQUIRED; other 7 systems use optional find_package() that silently skip on failure
Expected: All 8 systems use REQUIRED so CI catches find_package() naming mismatches
The current consumer test created a false positive in CI: 4 systems (logger, container, database, network) could not be found via snake_case find_package(), but CI still passed
What
Update the vcpkg consumer integration test to use
REQUIREDfor allfind_package()calls so CI fails when a package cannot be found, instead of silently skipping.common_systemusesREQUIRED; other 7 systems use optionalfind_package()that silently skip on failureREQUIREDso CI catches find_package() naming mismatchesintegration_tests/vcpkg_consumer/CMakeLists.txtWhy
find_package(), but CI still passedWhere
integration_tests/vcpkg_consumer/CMakeLists.txt— lines 20-91How
Technical Approach
Change all
find_package({name} CONFIG)calls tofind_package({name} CONFIG REQUIRED)and remove the conditionalif({name}_FOUND)blocks.Acceptance Criteria
find_package()calls useREQUIRED