[Kartikey1306]

master does not compile

tests/test_net.c:212:5: error: call to undeclared function 'assert';
  ISO C99 and later do not support implicit function declarations
tests/test_net.c:232:5: error: ...
tests/test_net.c:249:5: error: ...

test_net.c makes ten assert() calls and includes stdio.h, stdlib.h and string.h — but never assert.h.

It built while every test target was compiled with NDEBUG. #104 removed that from test targets so the assertions in these suites are real again, and that is what turned the missing include into a hard error. The omission was always there; #104 made the compiler care.

Fix

Add the include. Nothing else changes.

I swept the rest of tests/*.c — this is the only file that calls assert() without including the header:

$ for f in $(grep -rl "assert(" tests/*.c); do
    grep -q "include <assert.h>" $f || echo "MISSING: $f"; done
MISSING: tests/test_net.c

Verified

$ cmake --build build -j8      # was: 3 errors
BUILD OK
$ ctest
100% tests passed, 34 of 34

and the assertions are genuinely live rather than compiled away — the binary still carries its test_net.c assertion strings, which is the guarantee #104 exists to provide.

[Kartikey1306]

Merge-order note: this fixes the test build, but CI — eos stays red on its own — the default non-test build fails separately in net/src/net_posix.c. #113 fixes that and is green. Merging both, or just #113 (it contains this commit), returns master to green.

[Kartikey1306]

Closing in favour of #113, which contains this commit (9d2f7ea) verbatim as the first of its two, and is green on all five jobs.

This PR cannot go green on its own: it fixes Build & Test (Linux x86_64), but the same workflow's Default configuration (no tests, no product) job fails separately in net/src/net_posix.c, which is what the second commit in #113 fixes. Merging #113 lands both and returns master to green in one step.