style: fix clang-format violations and re-enable clang-format check (#26)

Fix clang-format in build #2

Author: kkkapu

.github/workflows/build_and_test.yml

@@ -64,7 +64,6 @@ jobs:
         coverage report
 
     - name: Check C++ test coverage
-      if: false # TODO(#3): fix and then re-enable
       run: |
         # Run C++ coverage check
         echo -e "\n##### Running C++ coverage check #####"
@@ -78,7 +77,8 @@ jobs:
           --xml-pretty -o cxx-coverage.xml \
           --sort uncovered-number \
           --gcov-ignore-parse-errors negative_hits.warn \
-          --gcov-ignore-parse-errors suspicious_hits.warn
+          --gcov-ignore-parse-errors suspicious_hits.warn \
+          --gcov-ignore-errors=no_working_dir_found
 
   lint-code:
     runs-on: ubuntu-latest

src/ml_flashpoint/checkpoint_object_manager/buffer_object/test/buffer_object_test.cpp

@@ -225,7 +225,7 @@ TEST_F(BufferObjectTest, GetCapacityReturnsCorrectSize) {
   ASSERT_EQ(buffer.get_capacity(), expected_capacity);
 
   // Test with the opening constructor.
-  // The 'writer' scope ensures the file is closed before reopening.
+  // The BufferObject constructor will create the file, and its destructor will close the file (without deleting)
   {
     BufferObject writer(test_path_ + "_new", 512);
   }
@@ -308,6 +308,7 @@ TEST_F(BufferObjectTest, CloseWithTruncateIgnoresTruncateOnReadOnlyFile) {
   const size_t truncate_to_size = 512;
 
   // Setup: Create a file on disk with a known initial size.
+  // The BufferObject constructor will create the file, and its destructor will close the file (without deleting)
   {
     BufferObject writable_buffer(test_path_, initial_size);
   }