fix: add assertions for RpcServer start() and fix buffer overflow

- raft_manager_tests: Add ASSERT_TRUE for RpcServer::start() in SetUp()
- storage_manager_tests: Fix buffer overflow by using PAGE_SIZE instead of 512

Author: poyrazK

tests/raft_manager_tests.cpp

@@ -31,7 +31,7 @@ class RaftManagerTests : public ::testing::Test {
         config_.cluster_port = TEST_PORT;
         cm_ = std::make_unique<ClusterManager>(&config_);
         rpc_ = std::make_unique<RpcServer>(TEST_PORT);
-        rpc_->start();
+        ASSERT_TRUE(rpc_->start()) << "RpcServer failed to start - port may be in use";
         manager_ = std::make_unique<RaftManager>("node1", *cm_, *rpc_);
     }
 

tests/storage_manager_tests.cpp

@@ -110,8 +110,8 @@ TEST_F(StorageManagerTests, PartialReadReturnsFalse) {
     ASSERT_TRUE(sm_->open_file(filename));
 
     // Write a small amount of data
-    char write_buf[512];
-    std::memset(write_buf, 0xAB, 512);
+    char write_buf[StorageManager::PAGE_SIZE];
+    std::memset(write_buf, 0xAB, StorageManager::PAGE_SIZE);
     ASSERT_TRUE(sm_->write_page(filename, 0, write_buf));
 
     // Try to read the small write as a full page should succeed (EOF handling fills zeros)