Assert should not be used in tests
The C assert
macro is used to check contracts. It might fail if the assertion is false (it does when the macro NDEBUG is not defined).
The tests should always fail whether compiled in debug or release mode, hence a plain if is better suited to the task.
We should remove basically all assert from the tests in PDI.