[native] Fix incorrect exclusion of PrestoC++ unit tests

The regex used int he CTest command to run tests excludes tests
with “velox” in the name leading to skipped tests such as
presto_to_velox_query_plan_test and others.
This results in missing coverage and non-detection of
introduced issues.

Author: czentgr

.github/workflows/prestocpp-linux-build-and-unit-test.yml

@@ -117,7 +117,7 @@ jobs:
           # Ensure transitive dependency libboost-iostreams is found.
           ldconfig /usr/local/lib
           cd presto-native-execution/_build/release
-          ctest -j 4 -VV --output-on-failure --exclude-regex velox.*
+          ctest -j 4 -VV --output-on-failure --exclude-regex ^velox.*
 
       - name: Upload artifacts
         if: |

presto-native-execution/Makefile

@@ -119,7 +119,7 @@ cmake-and-build:			#: cmake and build without updating submodules which requires
 	cmake --build $(BUILD_BASE_DIR)/$(BUILD_DIR) -j $(NUM_THREADS)
 
 unittest: debug			#: Build with debugging and run unit tests
-	cd $(BUILD_BASE_DIR)/debug && ctest -j $(NUM_THREADS) -VV --output-on-failure --exclude-regex velox.*
+	cd $(BUILD_BASE_DIR)/debug && ctest -j $(NUM_THREADS) -VV --output-on-failure --exclude-regex ^velox.*
 
 presto_protocol:		#: Build the presto_protocol serde library
 	cd presto_cpp/presto_protocol; $(MAKE) presto_protocol