Fix according review

Author: Alex-PLACET

.github/copilot-instructions.md

@@ -51,10 +51,3 @@ See `integration_tests/CMakeLists.txt` for examples. Missing this causes "cannot
 - `m_` prefix for members
 - Namespace: `sparrow_ipc`
 - Format: `cmake --build build --target clang-format` (requires `ACTIVATE_LINTER=ON`)
-
-## Common Issues
-
-1. Schema mismatches in stream → `std::invalid_argument`
-2. Deallocating source buffer while arrays in use → undefined behavior
-3. Missing RPATH on Linux → runtime linking errors
-4. Only LZ4 compression supported (not ZSTD yet)

.github/workflows/integration_tests.yaml

@@ -46,10 +46,6 @@ jobs:
         working-directory: build
         run: cmake --build . --config Release --target arrow_validate
 
-      - name: List all folders and subfolders
-        run: |
-          echo "Listing all folders and subfolders:"
-          find . -type d
 
       - name: Build Docker image
         run: docker build -t sparrow/integration-tests -f ci/docker/integration.dockerfile .

.github/workflows/linux.yml

@@ -103,6 +103,14 @@ jobs:
         working-directory: build
         run: cmake --build . --target run_example
 
+      - name: Build test_integration_tools
+        working-directory: build
+        run: cmake --build . --target test_integration_tools
+
+      - name: Run test_integration_tools
+        working-directory: build
+        run: cmake --build . --target run_test_integration_tools
+
       - name: Install
         working-directory: build
         run: sudo cmake --install .

.github/workflows/osx.yml

@@ -64,6 +64,14 @@ jobs:
         working-directory: build
         run: cmake --build . --target run_example
 
+      - name: Build test_integration_tools
+        working-directory: build
+        run: cmake --build . --target test_integration_tools
+
+      - name: Run test_integration_tools
+        working-directory: build
+        run: cmake --build . --target run_test_integration_tools
+
       - name: Install
         working-directory: build
         run: cmake --install .

.github/workflows/windows.yml

@@ -64,6 +64,13 @@ jobs:
         working-directory: build
         run: cmake --build . --config ${{ matrix.build_type }} --target run_example
 
+      - name: Build test_integration_tools
+        working-directory: build
+        run: cmake --build . --config ${{ matrix.build_type }} --target test_integration_tools
+
+      - name: Run test_integration_tools
+        working-directory: build
+        run: cmake --build . --config ${{ matrix.build_type }} --target run_test_integration_tools
       - name: Install
         working-directory: build
         run: cmake --install . --config ${{ matrix.build_type }}

integration_tests/CMakeLists.txt

@@ -157,6 +157,16 @@ target_include_directories(test_integration_tools
         ${CMAKE_BINARY_DIR}/generated
 )
 
+add_custom_target(run_test_integration_tools
+    COMMAND test_integration_tools
+    DEPENDS 
+        test_integration_tools
+    COMMENT "Running tests"
+    USES_TERMINAL
+)
+
+set_target_properties(run_test_integration_tools PROPERTIES FOLDER "Tests utilities")
+
 # Register with CTest
 enable_testing()
 add_test(NAME integration_tools_test COMMAND test_integration_tools)

src/serialize.cpp

@@ -47,7 +47,7 @@ namespace sparrow_ipc
         //
         // Arrow's WriteMessage returns metadata_length = align_to_8(8 + flatbuffer_size)
         // which INCLUDES the continuation bytes.
-        
+
         // Write metadata
         common_serialize(builder, stream);