[Copilot]

The project had only a placeholder test (REQUIRE(true == true)), with no coverage of the SSH login flow, connection state, or lifecycle behavior.

Build system

• CMakeLists.txt: Move enable_testing() to the top level so ctest discovers tests from the project root.
• src/CMakeLists.txt: Extract ssh_connection.cpp into a static library webssh_cpp_ssh_lib with PUBLIC include paths, shared between the main executable and test targets.
• test/CMakeLists.txt: Link all test executables against webssh_cpp_ssh_lib (transitively pulls in libssh2 + spdlog). Rename the custom target to avoid conflict with CTest's reserved test target.

Bug fix

ssh_pty_connection_manager::close_channel() was declared in the header but never implemented. Added the implementation: looks up the channel by ID, erases it from the manager's weak-pointer map, then calls close() to trigger the session cleanup chain.

New tests

File	Coverage
test/mini_buffer.test.cpp	8 cases / 42 assertions — constructors, reserve, resize, clear, append, move semantics, implicit char* conversion
test/ssh_session.test.cpp	13 cases — generate_id() variants, ssh_pty_connection_manager defaults and state (max channels, unknown ID lookups)
test/ssh_connection.test.cpp	9 integration cases — full login/auth flow, wrong-password rejection, bad-host failure, channel lifecycle, PTY+shell, resize, session pooling

Integration tests skip gracefully when TEST_SSH_HOST / TEST_SSH_USER / TEST_SSH_PASS are not set:

auto cfg = get_ssh_config();
if (!cfg.available) {
    WARN("Skipping SSH integration tests: TEST_SSH_HOST / TEST_SSH_USER / TEST_SSH_PASS not set");
    return;
}
// ... full connect → authenticate → open_channel → shell → close flow