unit-test: Remove `UBSAN_OPTIONS=halt_on_error=1`

Setting `UBSAN_OPTIONS=halt_on_error=1` here makes adding the
`UBSAN_OPTION` environment variable ineffective (like
`suppressions=ubsan.supp` or `print_summary=1`). This environment
variable is always overwritten.

The mechanism that makes UBSAN kill tests is no longer needed when using
meson version 1.3.0 or higher
https://github.com/mesonbuild/meson/commit/7b7d2e060b where default
```
UBSAN_OPTION=halt_on_error=1:abort_on_error=1:print_summary=1:print_stacktrace=1
```

Resolves: openbmc/openbmc-build-scripts#43

Tested:
- Run `run-unit-test-docker.sh` without setting env variable, with
`invalid vptr` error - tests failed with more info in logs (stack trace,
summary)
- Run `run-unit-test-docker.sh` with setting env variable
`UBSAN_OPTION=suppressions=ubsan.supp` with suppression for vptr check
and with bug location - tests passed

Change-Id: I64eb799645dbc9cde4b7ee75098b641ebe30a5a4
Signed-off-by: Ewelina Walkusz <ewelinax.walkusz@intel.com>
1 file changed