fix: adapt tests and fix bugs from enhancements rebase onto main

- Fix transport_tcp.c: initialize server->ssl_ctx to NULL
  (prevents SSL_CTX_free on garbage when server_create_tls fails)
- Fix test_transport_tcp.c: client_create now sets fd=-1, family=AF_UNSPEC
- Fix test_transport_tcp.c: server address family may be AF_INET or AF_INET6
- Fix test_file_sendfile.c: send_path=false protocol includes file_type prefix

src/shared/transport_tcp.c

@@ -60,6 +60,7 @@ Server* server_create(int port) {
   }
 
   server->file_descriptor = fd;
+  server->ssl_ctx = NULL;
   int opt = 1;
   if (setsockopt(server->file_descriptor, SOL_SOCKET, SO_REUSEADDR, &opt, sizeof(opt))) {
     perror("Error setting a socket option!");
@@ -285,6 +286,7 @@ void client_disconnect(Client* client) {
 void client_delete(Client* client) {
   if (client == NULL)
     return;
+  client_disconnect(client);
   if (client->ssl_ctx) {
     SSL_CTX_free(client->ssl_ctx);
     client->ssl_ctx = NULL;

tests/test_file_sendfile.c

@@ -224,8 +224,9 @@ static void test_sendfile_no_path() {
   pid_t pid = fork();
   if (pid == 0) {
     close(p[1]);
-    /* When send_path is false, the sender sends raw data (size + bytes) only.
-     * We need to receive just the Data, not a File. */
+    /* When send_path is false, the sender still sends file_type + data */
+    int file_type;
+    receive_int(p[0], &file_type);
     Data* received = receive_data(p[0]);
     close(p[0]);
 

tests/test_transport_tcp.c

@@ -8,8 +8,8 @@
 static void test_client_create_delete() {
   Client* client = client_create();
   EXPECT_NOT_NULL(client);
-  EXPECT_TRUE(client->file_descriptor >= 0);
-  EXPECT_EQ_INT(client->address.sin_family, AF_INET);
+  EXPECT_EQ_INT(client->file_descriptor, -1);
+  EXPECT_EQ_INT(client->address.ss_family, AF_UNSPEC);
   EXPECT_EQ_INT(client->ssh_child_pid, -1);
   EXPECT_NULL(client->ssl);
   EXPECT_NULL(client->ssl_ctx);
@@ -30,7 +30,7 @@ static void test_server_create_delete() {
   Server* server = server_create(0);
   EXPECT_NOT_NULL(server);
   EXPECT_TRUE(server->file_descriptor >= 0);
-  EXPECT_EQ_INT(server->address.sin_family, AF_INET);
+  EXPECT_TRUE(server->address.ss_family == AF_INET || server->address.ss_family == AF_INET6);
   EXPECT_NULL(server->ssl_ctx);
 
   /* Clean up */
@@ -57,9 +57,8 @@ static void test_client_create_multiple() {
   Client* c2 = client_create();
   EXPECT_NOT_NULL(c1);
   EXPECT_NOT_NULL(c2);
-  EXPECT_TRUE(c1->file_descriptor >= 0);
-  EXPECT_TRUE(c2->file_descriptor >= 0);
-  EXPECT_TRUE(c1->file_descriptor != c2->file_descriptor);
+  EXPECT_EQ_INT(c1->file_descriptor, -1);
+  EXPECT_EQ_INT(c2->file_descriptor, -1);
 
   client_delete(c1);
   client_delete(c2);