Radium Engine  1.5.14
Loading...
Searching...
No Matches
How to contribute to Radium Libraries

Radium libraries

Direct contributions to master are closed. We defer merge to master after our radium based projects (public or private) CI tests. To this end, please select release_candidate as a base branch of your pull requests.

Pre commit hook

We provide a configuration file for pre-commit. Please install and use it, e.g.

pip install pre-commit
pre-commit install

This hook will check several formatting convention and cleanup your files' style on each commit. If some checks fail, it might have update automatically your code (hence just add again the modified file to your commit) or needs manual intervention. These script are also added as CI github check (to show any problems in case you have troubles to install the hook script).

Radium test suite

Preliminaries

Radium testing is done using two tools:

  • CTest: the test system by CMake. It provides compilation, running and basic reporting mechanisms for tests.
  • Continuous Integration system: github action and codecov.io. We use these services to call CTest on several platforms (windows, ubuntu and MacOs) with several compilers, and validate any change made to the main repository.

As a Radium contributor, and in most cases, you will have to work only with CTest, and let the CI system run your tests automatically. In this section we describe how to write a new test, how to insert it in the testing suite, and how to validate your results on your computer and the CI systems.

How to write tests

There is two kind of tests, integration test and unit test. Tests are run with the check target and it's variant (check_verbose, test, run_unittests, run_unittests_verbose) or directly with ctest. Calling make test or running ctest only perform testing, without build anything (e.g. if the code base is changed), due to CMake way of working.

  • integration test are based on program return values, using the ctest suite. The tests are set in tests/integration/CMakeLists.txt, see add_test documentation. Integration tests typically run some program, catch and analyze the output, compare to reference output etc.
  • unit test use Catch2 framework see tutorial

Note that we target Unit Tests, which can be defined as:

‍The purpose of a unit test in software engineering is to verify the behavior of a relatively small piece of software, independently from other parts. Unit tests are narrow in scope, and allow us to cover all cases, ensuring that every single part works correctly.

Source: https://www.toptal.com/qa/how-to-write-testable-code-and-why-it-matters

To add a unit test, create the test file in tests/unittest/LIB/mytestfile.cpp where LIB correspond to the library you are testing (Core, Engine, Gui, IO). Edit tests/unittest/CMakeLists.txt and add your test to the list of sources of add_executable(unittests .... If your test needs data, add them to tests/unittest/data. unittest working dir is tests/unittest so you can refer your data with a relative path starting with data. No output file are allowed in unittest. TEST_CASE first argument is the unique test name, second argument is a list of optional tags. You can add independent TEST_SECTIONs to your test, please refer to catch2 documentation.

Here is an example:

#include <Core/Geometry/Obb.hpp>
#include <Core/Math/Math.hpp>
#include <Core/Types.hpp>
#include <catch2/catch.hpp>
TEST_CASE( "Core/Geometry/Obb", "[Core][Core/Geometry][Obb]" ) {
using namespace Ra::Core;
SECTION( "Simple tests" ) {
// Check Obb and empty aabb
Aabb aabb;
Geometry::Obb obb( aabb, Transform::Identity() );
REQUIRE( obb.toAabb().isEmpty() );
REQUIRE( obb.toAabb().isApprox( aabb ) );
obb.transform().rotate( AngleAxis( Math::Pi, Vector3::UnitX() ) );
REQUIRE( obb.toAabb().isEmpty() );
REQUIRE( obb.toAabb().isApprox( aabb ) );
}
}
An oriented bounding box.
Definition Obb.hpp:12
This namespace contains everything "low level", related to data, datastuctures, and computation.
Definition Cage.cpp:4

Code coverage

On PR, code coverage analysis is performed through a github action with help of codecov.io report.

You can run such analysis on your own (on linux system, not tested on other plateform) using lcov. First install lcov, sed and xmllint for the tests. It's probably on your system repositories, e.g.:

  • for Debian: sudo apt install lcov libxml2-utils
  • on MacOS: brew install lcov libxml2

You can generate lcov output files to check tests coverage. To this end RADIUM_ENABLE_COVERAGE=ON have to be passed to cmake, works on Linux in Debug. Then the target coverage builds and run all the necessary steps and produce a file total.info in your build dir. Please check CMakeLists.txt for the detailed steps, and lcov documentation for further processing of total.info to generate an html report.

Here are the example commands to run test and generate report.

#some vars
RADIUM_CONFIG=path/to/file/with/radium-options.cmake #like where external are built ... it's optionnal
RADIUM_SOURCE=path/to/radium/main/CMakeLists.txt
# in you build path
cmake -DCMAKE_BUILD_TYPE=Debug -DRADIUM_ENABLE_COVERAGE=ON -C ${RADIUM_CONFIG} ${RADIUM_SOURCE}
cmake --build . --target coverage #build everything, with coverage enable, run test, perform analysis

output report is ./coverage/index.html.

lcov is quite slow to process files, fastcov is a parallel alternative. To install fastcov: pip3 install fastcov.

Note: fastcov is currently supported by GNU gcov only, so we enable it only on UNIX systems, excluding MacOS. See more details at: https://github.com/RPGillespie6/fastcov/issues/36.

and to run the analysis:

# in you build path
cmake -DCMAKE_BUILD_TYPE=Debug -DRADIUM_ENABLE_COVERAGE=ON -C RADIUM_CONFIG RADIUM_SOURCE
cmake --build . --target coverag_fastcov #build everything, with coverage enable, run test, perform analysis

report is ./coverage_fastcov/index.html

For comparison on a Intel(R) Xeon(R) Silver 4210 CPU @ 2.20GHz (with radium build cmake --build . already done).

  • time cmake --build . --target coverage -> 2:08.60 total
  • time cmake --build . --target coverage_fastcov -> 48.178 total