No test target - the project only builds test executables but doesn't define the 'test' target to run tests

IronsDu / brynet

A Header-Only cross-platform C++ TCP network library . We can use vcpkg(https://github.com/Microsoft/vcpkg/tree/master/ports/brynet) install brynet.

MIT License

1.05k stars 241 forks source link

No test target - the project only builds test executables but doesn't define the 'test' target to run tests #130

Open yurivict opened 1 year ago

yurivict commented 1 year ago

Usually the 'test' target is defined by projects to simplify running tests.

IronsDu commented 1 year ago

Because some people think it will Increase build/compiler time.

yurivict commented 1 year ago

Because some people think it will Increase build/compiler time.

Tests should only be enabled by the cmake's standard BUILD_TESTS boolean value that is OFF by default. This can't possibly increase build/compile time.