[riot-commits] [RIOT-OS/RIOT] cc729b: tests: deactivate DEVELHELP for unittests

Sebastian Meiling s at mlng.net
Fri Jan 26 16:03:42 CET 2018


  Branch: refs/heads/master
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: cc729bd26a4c0e06831d03d3df6037a2fbbc3b87
      https://github.com/RIOT-OS/RIOT/commit/cc729bd26a4c0e06831d03d3df6037a2fbbc3b87
  Author: Martine Lenders <m.lenders at fu-berlin.de>
  Date:   2018-01-25 (Thu, 25 Jan 2018)

  Changed paths:
    M tests/unittests/Makefile

  Log Message:
  -----------
  tests: deactivate DEVELHELP for unittests

In my opinion it doesn't make sense to activate DEVELHELP for unittests
because it also causes the evaluation of `assert()`. This causes the
tests to crash for values the tests test intentionally (because it is
the point of the unittests to test also some unexpected values and how
this influences the behavior).


  Commit: c2400083aedd699f1c995148db29f29531a2b019
      https://github.com/RIOT-OS/RIOT/commit/c2400083aedd699f1c995148db29f29531a2b019
  Author: Sebastian Meiling <s at mlng.net>
  Date:   2018-01-26 (Fri, 26 Jan 2018)

  Changed paths:
    M tests/unittests/Makefile

  Log Message:
  -----------
  Merge pull request #8440 from miri64/tests/fix/deactivate-DEVELHELP

tests: deactivate DEVELHELP for unittests


Compare: https://github.com/RIOT-OS/RIOT/compare/a360fdeddacb...c2400083aedd


More information about the commits mailing list