Remove include(ctest)

[wusatosi]

Fixes #60 .

I am not really understanding if this is desired, so reject this if this change is not wanted.

Explicitly asking @camio @bretbrownjr and @ClausKlein for input.

[steve-downey]

👍

[camio]

I think we can drop include(CTest) entirely. Also, the code was simpler when FetchContent wasn't conditionally included.

[wusatosi]

[we can drop include(CTest) entirely]

Done in CMakeLists, the CI isn't failing so I guess this would work.

Will this impact commands such as ctest --test-dir build --build-config Release?

The code was simpler when FetchContent wasn't conditionally included.

Well, this is such a "when we are arguing we are losing" thing, just let me know if I should revert the change with FetchContent.

[camio]

[we can drop include(CTest) entirely]

Will this impact commands such as ctest --test-dir build --build-config Release?

Nope, we no longer need anything this module provides.