From: Michael Altizer (mialtize) <mialtize@cisco.com>
Date: Fri, 3 Apr 2020 18:55:21 +0000 (+0000)
Subject: Merge pull request #2128 in SNORT/snort3 from ~MASHASAN/snort3:wrap_enable_shell... 
X-Git-Tag: 3.0.1-2~40
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f24dd3cd201ae212b52135033bb4798923c88624;p=thirdparty%2Fsnort3.git

Merge pull request #2128 in SNORT/snort3 from ~MASHASAN/snort3:wrap_enable_shell to master

Squashed commit of the following:

commit 9ccb24f001e6699d02a40b5198108773f3a5dcb1
Author: Masud Hasan <mashasan@cisco.com>
Date:   Fri Apr 3 07:18:43 2020 -0400

    control: Disable request unit-test in cmake if shell is disabled
---

diff --git a/src/main/test/CMakeLists.txt b/src/main/test/CMakeLists.txt
index 4c11cd6fa..fe5aff786 100644
--- a/src/main/test/CMakeLists.txt
+++ b/src/main/test/CMakeLists.txt
@@ -1,4 +1,6 @@
-add_cpputest(request_test
-    SOURCES
-        ../request.cc
-)
+if ( ENABLE_SHELL )
+    add_cpputest(request_test
+        SOURCES
+            ../request.cc
+    )
+endif ( ENABLE_SHELL )
diff --git a/src/main/test/request_test.cc b/src/main/test/request_test.cc
index faaf5f9ad..119472b60 100644
--- a/src/main/test/request_test.cc
+++ b/src/main/test/request_test.cc
@@ -45,7 +45,6 @@ TEST_GROUP(request_tests)
 //--------------------------------------------------------------------------
 // Make sure multiple responses are queued
 //--------------------------------------------------------------------------
-#ifdef SHELL
 TEST(request_tests, queued_response_test)
 {
     Request request(STDOUT_FILENO);
@@ -57,7 +56,7 @@ TEST(request_tests, queued_response_test)
     CHECK(request.send_queued_response() == true);
     CHECK(request.send_queued_response() == false); // empty queue after being written
 }
-#endif
+
 //-------------------------------------------------------------------------
 // main
 //-------------------------------------------------------------------------