Issue 6280 - Check that we don't return invalid entries

test/compiled/StringMap.cpp

Index: test/compiled/StringMap.cpp
===================================================================
--- a/test/compiled/StringMap.cpp
+++ b/test/compiled/StringMap.cpp
@@ -20,22 +20,27 @@
 #include "compiled/StringMap.h"
 
 ABP_NS_USING
 
 template<template <typename T> class S>
 void testStringMap()
 {
   S<std::string> map;
+
+  EXPECT_EQ(map.begin(), map.end());
+
   auto key = u"Foobar"_str;
   EXPECT_EQ(key.length(), 6);
   EXPECT_EQ(map.size(), 0);
+  EXPECT_EQ(map.begin(), map.end());

[sergei, 2018/02/12 08:50:05]

It seems this line is redundant because `map` is not modified since the same
line above.

[hub, 2018/02/12 14:20:25]

Done.

 
   map[u"Foobar"_str] = "one";
   EXPECT_EQ(map.size(), 1);
+  EXPECT_NE(map.begin(), map.end());

[sergei, 2018/02/12 08:50:05]

That's all right. In addition it would be good to test with the keys (the
implementation is missing yet though) when the first entry of mBuckets is not
used.

[sergei, 2018/02/12 08:54:28]

I missed the comment in the issue description, seems all right then.

 
   map[u""_str] = "null";
   EXPECT_EQ(map.size(), 2);
 
   auto entry = map.find(u"Foobar"_str);
   EXPECT_TRUE(entry);
 
   entry = map.find(u"Foobar2"_str);