Skip to content

Replace 'return 1' with '::exit( EINVAL )' in test plugins

Marco Clemencic requested to merge fix-test-checks into master

Tests run with geoPluginRun should not just return 1 in case of failures because the return value is ignored. The only ways to get geoPluginRun to fail is to throw an exception or exit immediately with ::exit() (or assert if we make sure we do not define NDEBUG for tests).

Note that this MR actually causes some tests to explicitly fail (instead of ignoring the failure).

Merge request reports