Merge pull request #1211 in SNORT/snort3 from proto_ref_test to master

Squashed commit of the following:

commit f13afb877073cc8851a9847bce9185a931c62bd5
Author: Victor Roemer <viroemer@cisco.com>
Date:   Tue May 1 11:40:52 2018 -0400

    target_based: 100% coverage on snort_protocols.cc

diff --git a/src/target_based/test/proto_ref_test.cc b/src/target_based/test/proto_ref_test.cc
index a3013227532ab90e818d6f8e823382ed36564d64..977c32568a8d71733a83fabe26b2ae36a63122d1 100644 (file)
--- a/src/target_based/test/proto_ref_test.cc
+++ b/src/target_based/test/proto_ref_test.cc
@@ -186,6 +186,13 @@ TEST(protocol_reference, constructors)
     CHECK( refs.find("unknown") == unknown );
 }
 
+// Verify calling get_name(id) where id > id_map.size() returns "unknown".
+TEST(protocol_reference, getname_gt_mapsize)
+{
+    ProtocolReference refs;
+    CHECK( std::string(refs.get_name(100)) == "unknown"  );
+}
+
 int main(int argc, char** argv)
 {
     return CommandLineTestRunner::RunAllTests(argc, argv);