## Description Fix inaccuracy of the coverage report. There three main reasons: - use llvm-cov-11 instead of the default llvm (which may be llvm 10) - increase heap and stack for the enclave of the integration test - exclude some unnecessary when capture the coverage info
The new coverage result has been uploaded to coveralls: https://coveralls.io/builds/46626151 ## Type of change (select or add applied and delete the others) - [x] Bug fix (non-breaking change which fixes an issue) ## How has this been tested? https://ci.mesalock-linux.org/mssun/incubator-teaclave/151 ## Checklist - [x] Fork the repo and create your branch from `master`. - [x] If you've added code that should be tested, add tests. - [x] If you've changed APIs, update the documentation. - [x] Ensure the tests pass (see CI results). - [x] Make sure your code lints/format. You can view, comment on, or merge this pull request online at: https://github.com/apache/incubator-teaclave/pull/627 -- Commit Summary -- * fix: fix generate testing code coverage -- File Changes -- M .drone.yml (94) M cmake/scripts/gen_cov.sh (5) M cmake/scripts/llvm-gcov.sh (2) M cmake/scripts/test.sh (6) M tests/integration/enclave/Enclave.config.xml (4) -- Patch Links -- https://github.com/apache/incubator-teaclave/pull/627.patch https://github.com/apache/incubator-teaclave/pull/627.diff -- Reply to this email directly or view it on GitHub: https://github.com/apache/incubator-teaclave/pull/627 You are receiving this because you are subscribed to this thread. Message ID: <apache/incubator-teaclave/pull/6...@github.com>