search

CesiumGS / cesium-omniverse

Bringing the 3D geospatial ecosystem to Omniverse
https://cesium.com/platform/cesium-for-omniverse/
Apache License 2.0
55 stars 7 forks source link

Get coverage working with tests extension #471

Open mattelser opened 11 months ago

mattelser commented 11 months ago

This may be as simple as adding extern/nvidia/_build/target-deps/kit-sdk/kit ./apps/cesium.omniverse.cpp.tests.runner.kit as a command run by the coverage cmake target (in cmake/GenerateCoverage.cmake) but I'm not actually sure how that target works at the moment and I wouldn't be surprised if it were more involved than that.

mattelser commented 10 months ago

This is currently blocked by an issue nvidia is looking into where omniverse does not exit cleanly when the tests extension is enabled. Killed processes don't generate the necessary GCDA files, so there is no way to generate coverage until this is fixed.

In talking with @lilleyse, once this is unblocked we'll likely want to set coverage up to be generated in two steps. Step one is to manually run then manually exit the tests extension in such a way that it generates GCDA files. Step two is to point gcov at those GCDA files so it gathers what lines were executed during step one.