From: Tom Peters (thopeter) <thopeter@cisco.com>
Date: Tue, 28 Jun 2022 17:05:00 +0000 (+0000)
Subject: Pull request #3487: http2_inspect: unit tests depending on REG_TEST
X-Git-Tag: 3.1.33.0~4
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f76e4a76628ff31939d4c02c982ff8585eb879db;p=thirdparty%2Fsnort3.git

Pull request #3487: http2_inspect: unit tests depending on REG_TEST

Merge in SNORT/snort3 from ~THOPETER/snort3:nhttp165 to master

Squashed commit of the following:

commit d3b038594dc5118c6114f29a998f8c71aa4518f1
Author: Tom Peters <thopeter@cisco.com>
Date:   Mon Jun 27 15:57:42 2022 -0400

    http2_inspect: unit tests depending on REG_TEST
---

diff --git a/src/service_inspectors/http2_inspect/test/http2_hpack_int_decode_test.cc b/src/service_inspectors/http2_inspect/test/http2_hpack_int_decode_test.cc
index 975a3c090..a0455f054 100644
--- a/src/service_inspectors/http2_inspect/test/http2_hpack_int_decode_test.cc
+++ b/src/service_inspectors/http2_inspect/test/http2_hpack_int_decode_test.cc
@@ -51,7 +51,6 @@ TEST_GROUP(http2_hpack_int_decode_success)
     void teardown() override
     {
         CHECK(inf.none_found() == true);
-        CHECK(events.none_found() == true);
         delete decode;
     }
 };
@@ -319,7 +318,6 @@ TEST(http2_hpack_int_decode_leading_zeros, leading_zeros)
     CHECK(res == 31);
     CHECK(bytes_processed == 3);
     CHECK(local_inf.get_raw(0) == (1<<INF_INT_LEADING_ZEROS));
-    CHECK(local_events.get_raw(0) == (1<<(EVENT_INT_LEADING_ZEROS-1)));
 }
 
 TEST(http2_hpack_int_decode_leading_zeros, leading_0_byte_11)
@@ -341,7 +339,6 @@ TEST(http2_hpack_int_decode_leading_zeros, leading_0_byte_11)
     CHECK(res == 0x7FFFFFFFFFFFFFFF);
     CHECK(bytes_processed == 11);
     CHECK(local_inf.get_raw(0) == (1<<INF_INT_LEADING_ZEROS));
-    CHECK(local_events.get_raw(0) == (1<<(EVENT_INT_LEADING_ZEROS-1)));
 }
 
 int main(int argc, char** argv)
diff --git a/src/service_inspectors/http2_inspect/test/http2_hpack_string_decode_test.cc b/src/service_inspectors/http2_inspect/test/http2_hpack_string_decode_test.cc
index 45ba8dc65..8e03fb991 100644
--- a/src/service_inspectors/http2_inspect/test/http2_hpack_string_decode_test.cc
+++ b/src/service_inspectors/http2_inspect/test/http2_hpack_string_decode_test.cc
@@ -58,7 +58,6 @@ TEST_GROUP(http2_hpack_string_decode_success)
     void teardown() override
     {
         CHECK(inf.none_found() == true);
-        CHECK(events.none_found() == true);
         delete decode;
     }
 };
diff --git a/src/utils/infractions.h b/src/utils/infractions.h
index 95039276b..c20aeb570 100644
--- a/src/utils/infractions.h
+++ b/src/utils/infractions.h
@@ -46,12 +46,10 @@ public:
     friend bool operator&(const Infractions& lhs, const Infractions& rhs)
         { return (lhs.infractions & rhs.infractions) != 0; }
 
-#ifdef REG_TEST
     bool none_found() const { return infractions == 0; }
 
     uint64_t get_raw(unsigned base) const { return
         ((infractions >> base) & std::bitset<MAX>(0xFFFFFFFFFFFFFFFF)).to_ulong(); }
-#endif
 
 private:
     std::bitset<MAX> infractions = 0;