moleske commented on a change in pull request #765:
URL: https://github.com/apache/geode-native/pull/765#discussion_r593477709
##########
File path: cppcache/test/CMakeLists.txt
##########
@@ -46,7 +50,7 @@ add_executable(apache-geode_unittests
LRUQueueTest.cpp
PdxInstanceImplTest.cpp
PdxTypeTest.cpp
- QueueConnectionRequestTest.cpp
+ ${ENABLE_QUEUECONNECTIONREQUESTTEST}
Review comment:
This test I didn't conditionally add the disable flag cause I got
confused on how to do it and it wasn't working. That would probably be better.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
