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

Sebastian Meiling s at mlng.net
Fri Jan 26 19:33:57 CET 2018


  Branch: refs/heads/2018.01-branch
  Home:   https://github.com/RIOT-OS/RIOT
  Commit: d40cb1398ae9c9ced29f7c676132872481f11a01
      https://github.com/RIOT-OS/RIOT/commit/d40cb1398ae9c9ced29f7c676132872481f11a01
  Author: Martine Lenders <m.lenders at fu-berlin.de>
  Date:   2018-01-26 (Fri, 26 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: 101578d7a19aca689cbf118aa907a3aa518ce45f
      https://github.com/RIOT-OS/RIOT/commit/101578d7a19aca689cbf118aa907a3aa518ce45f
  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 #8454 from miri64/2018.01/tests/fix/deactivate-DEVELHELP

tests: deactivate DEVELHELP for unittests [2018.01 backport]


Compare: https://github.com/RIOT-OS/RIOT/compare/eca57516ab9f...101578d7a19a


More information about the commits mailing list