Merge remote-tracking branch 'origin/main' into merge-all-v2

# Conflicts:
#	src/client/client_cli.c
#	tests/test_transport_ssh.c

tests/test_transport_ssh.c

@@ -4,14 +4,39 @@
 
 static void test_ssh_connect_invalid_dest_no_colon() {
   /* cppcheck-suppress constVariablePointer */
-  Client* c = client_connect_ssh("/path/to/dest", 22);
-  EXPECT_NULL(c);
+  Client* client = client_connect_ssh("invalid-destination-no-colon", 22, NULL);
+  EXPECT_NULL(client);
 }
 
 static void test_ssh_connect_invalid_dest_empty() {
   /* cppcheck-suppress constVariablePointer */
-  Client* c = client_connect_ssh("", 22);
-  EXPECT_NULL(c);
+  Client* client = client_connect_ssh("", 22, NULL);
+  EXPECT_NULL(client);
+}
+
+/* Test client_connect_ssh with malformed destination (just a colon).
+ * parse_remote_dest succeeds, ssh is exec'd and fails, but the function
+ * creates a Client that must be cleaned up. */
+static void test_ssh_connect_malformed() {
+  Client* client = client_connect_ssh(":", 22, NULL);
+  /* ssh binary exists, so exec succeeds; the function returns a Client.
+   * We just verify it doesn't crash and clean up properly. */
+  if (client != NULL) {
+    client_disconnect(client);
+    client_delete(client);
+  }
+  EXPECT_TRUE(true);
+}
+
+/* Test client_connect_ssh with valid format but unreachable host.
+ * The function launches ssh which will fail to connect, returns a Client. */
+static void test_ssh_connect_unreachable() {
+  Client* client = client_connect_ssh("nonexistent.invalid:/remote/path", 22, NULL);
+  if (client != NULL) {
+    client_disconnect(client);
+    client_delete(client);
+  }
+  EXPECT_TRUE(true);
 }
 
 void test_transport_ssh() {