summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGabe Black <gabeblack@google.com>2017-12-03 01:42:22 -0800
committerGabe Black <gabeblack@google.com>2017-12-12 19:46:16 +0000
commit3297cd0f6ed9d6b5ad3bbbf477b9b74dfba5b472 (patch)
treee5894a7ad096612d45e669000957f6ebf8e59794
parent364a80fbf78dbecb02565c367f96720e907cd8f2 (diff)
downloadgem5-3297cd0f6ed9d6b5ad3bbbf477b9b74dfba5b472.tar.xz
tests: Convert the trie unit test into a gtest.
Change-Id: Idcf60260d9bda1b8ef5b6f5d59b74ca218395f0c Reviewed-on: https://gem5-review.googlesource.com/6265 Maintainer: Andreas Sandberg <andreas.sandberg@arm.com> Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com> Reviewed-by: Brandon Potter <Brandon.Potter@amd.com>
-rw-r--r--src/unittest/SConscript2
-rw-r--r--src/unittest/trietest.cc165
2 files changed, 86 insertions, 81 deletions
diff --git a/src/unittest/SConscript b/src/unittest/SConscript
index 9892df845..b497ddbfd 100644
--- a/src/unittest/SConscript
+++ b/src/unittest/SConscript
@@ -41,7 +41,7 @@ UnitTest('nmtest', 'nmtest.cc')
UnitTest('rangemaptest', 'rangemaptest.cc')
UnitTest('refcnttest', 'refcnttest.cc')
UnitTest('strnumtest', 'strnumtest.cc')
-UnitTest('trietest', 'trietest.cc')
+GTest('trietest', 'trietest.cc')
stattest_py = PySource('m5', 'stattestmain.py', tags='stattest')
UnitTest('stattest', 'stattest.cc', stattest_py, main=True)
diff --git a/src/unittest/trietest.cc b/src/unittest/trietest.cc
index 12d434fe6..4738ebd64 100644
--- a/src/unittest/trietest.cc
+++ b/src/unittest/trietest.cc
@@ -28,15 +28,12 @@
* Authors: Gabe Black
*/
+#include <gtest/gtest.h>
+
#include <iostream>
#include "base/trie.hh"
#include "base/types.hh"
-#include "unittest/unittest.hh"
-
-using UnitTest::setCase;
-
-typedef Trie<Addr, uint32_t> TestTrie;
namespace {
@@ -47,80 +44,88 @@ static inline uint32_t *ptr(uintptr_t val)
} // anonymous namespace
-int
-main()
+class TrieTestData : public testing::Test
+{
+ protected:
+ typedef Trie<Addr, uint32_t> TrieType;
+ TrieType trie;
+};
+
+TEST_F(TrieTestData, Empty)
+{
+ EXPECT_EQ(trie.lookup(0x123456701234567), nullptr);
+}
+
+TEST_F(TrieTestData, SingleEntry)
+{
+ trie.insert(0x0123456789abcdef, 40, ptr(1));
+ EXPECT_EQ(trie.lookup(0x123456701234567), nullptr);
+ EXPECT_EQ(trie.lookup(0x123456789ab0000), ptr(1));
+}
+
+TEST_F(TrieTestData, TwoOverlappingEntries)
+{
+ trie.insert(0x0123456789abcdef, 40, ptr(1));
+ trie.insert(0x0123456789abcdef, 36, ptr(2));
+ EXPECT_EQ(trie.lookup(0x123456700000000), nullptr);
+ EXPECT_EQ(trie.lookup(0x123456789ab0000), ptr(2));
+}
+
+TEST_F(TrieTestData, TwoOverlappingEntriesReversed)
+{
+ trie.insert(0x0123456789abcdef, 36, ptr(2));
+ trie.insert(0x0123456789abcdef, 40, ptr(1));
+ EXPECT_EQ(trie.lookup(0x123456700000000), nullptr);
+ EXPECT_EQ(trie.lookup(0x123456789ab0000), ptr(2));
+}
+
+TEST_F(TrieTestData, TwoIndependentEntries)
+{
+ trie.insert(0x0123456789abcdef, 40, ptr(2));
+ trie.insert(0x0123456776543210, 40, ptr(1));
+ EXPECT_EQ(trie.lookup(0x0123456789000000), ptr(2));
+ EXPECT_EQ(trie.lookup(0x0123456776000000), ptr(1));
+ EXPECT_EQ(trie.lookup(0x0123456700000000), nullptr);
+}
+
+TEST_F(TrieTestData, TwoEntries)
+{
+ trie.insert(0x0123456789000000, 40, ptr(4));
+ trie.insert(0x0123000000000000, 40, ptr(1));
+ trie.insert(0x0123456780000000, 40, ptr(3));
+ trie.insert(0x0123456700000000, 40, ptr(2));
+
+ EXPECT_EQ(trie.lookup(0x0123000000000000), ptr(1));
+ EXPECT_EQ(trie.lookup(0x0123456700000000), ptr(2));
+ EXPECT_EQ(trie.lookup(0x0123456780000000), ptr(3));
+ EXPECT_EQ(trie.lookup(0x0123456789000000), ptr(4));
+}
+
+TEST_F(TrieTestData, RemovingEntries)
{
- // Create an empty Ptr and verify it's data pointer is NULL.
- setCase("An empty trie.");
- TestTrie trie1;
- EXPECT_EQ(trie1.lookup(0x123456701234567), NULL);
-
- setCase("A single entry.");
- trie1.insert(0x0123456789abcdef, 40, ptr(1));
- EXPECT_EQ(trie1.lookup(0x123456701234567), NULL);
- EXPECT_EQ(trie1.lookup(0x123456789ab0000), ptr(1));
-
- setCase("Two entries, one on the way to the other.");
- TestTrie trie2;
- trie2.insert(0x0123456789abcdef, 40, ptr(1));
- trie2.insert(0x0123456789abcdef, 36, ptr(2));
- EXPECT_EQ(trie2.lookup(0x123456700000000), NULL);
- EXPECT_EQ(trie2.lookup(0x123456789ab0000), ptr(2));
-
- TestTrie trie3;
- trie3.insert(0x0123456789abcdef, 36, ptr(2));
- trie3.insert(0x0123456789abcdef, 40, ptr(1));
- EXPECT_EQ(trie3.lookup(0x123456700000000), NULL);
- EXPECT_EQ(trie3.lookup(0x123456789ab0000), ptr(2));
-
- setCase("Two entries on different paths.");
- TestTrie trie4;
- trie4.insert(0x0123456789abcdef, 40, ptr(2));
- trie4.insert(0x0123456776543210, 40, ptr(1));
- EXPECT_EQ(trie4.lookup(0x0123456789000000), ptr(2));
- EXPECT_EQ(trie4.lookup(0x0123456776000000), ptr(1));
- EXPECT_EQ(trie4.lookup(0x0123456700000000), NULL);
-
- setCase("Skipping past an entry but not two.");
- TestTrie trie5;
- trie5.insert(0x0123456789000000, 40, ptr(4));
- trie5.insert(0x0123000000000000, 40, ptr(1));
- trie5.insert(0x0123456780000000, 40, ptr(3));
- trie5.insert(0x0123456700000000, 40, ptr(2));
-
- setCase("Looking things up.");
- EXPECT_EQ(trie5.lookup(0x0123000000000000), ptr(1));
- EXPECT_EQ(trie5.lookup(0x0123456700000000), ptr(2));
- EXPECT_EQ(trie5.lookup(0x0123456780000000), ptr(3));
- EXPECT_EQ(trie5.lookup(0x0123456789000000), ptr(4));
-
- setCase("Removing entries.");
- TestTrie trie6;
- TestTrie::Handle node1, node2;
- trie6.insert(0x0123456789000000, 40, ptr(4));
- trie6.insert(0x0123000000000000, 40, ptr(1));
- trie6.insert(0x0123456780000000, 40, ptr(3));
- node1 = trie6.insert(0x0123456700000000, 40, ptr(2));
- node2 = trie6.insert(0x0123456700000000, 32, ptr(10));
-
- EXPECT_EQ(trie6.lookup(0x0123000000000000), ptr(1));
- EXPECT_EQ(trie6.lookup(0x0123456700000000), ptr(10));
- EXPECT_EQ(trie6.lookup(0x0123456780000000), ptr(10));
- EXPECT_EQ(trie6.lookup(0x0123456789000000), ptr(10));
-
- trie6.remove(node2);
-
- EXPECT_EQ(trie6.lookup(0x0123000000000000), ptr(1));
- EXPECT_EQ(trie6.lookup(0x0123456700000000), ptr(2));
- EXPECT_EQ(trie6.lookup(0x0123456780000000), ptr(3));
- EXPECT_EQ(trie6.lookup(0x0123456789000000), ptr(4));
-
- trie6.remove(node1);
-
- EXPECT_EQ(trie6.lookup(0x0123000000000000), ptr(1));
- EXPECT_EQ(trie6.lookup(0x0123456700000000), NULL);
- EXPECT_EQ(trie6.lookup(0x0123456780000000), ptr(3));
- EXPECT_EQ(trie6.lookup(0x0123456789000000), ptr(4));
-
- return UnitTest::printResults();
+ TrieType::Handle node1, node2;
+ trie.insert(0x0123456789000000, 40, ptr(4));
+ trie.insert(0x0123000000000000, 40, ptr(1));
+ trie.insert(0x0123456780000000, 40, ptr(3));
+ node1 = trie.insert(0x0123456700000000, 40, ptr(2));
+ node2 = trie.insert(0x0123456700000000, 32, ptr(10));
+
+ EXPECT_EQ(trie.lookup(0x0123000000000000), ptr(1));
+ EXPECT_EQ(trie.lookup(0x0123456700000000), ptr(10));
+ EXPECT_EQ(trie.lookup(0x0123456780000000), ptr(10));
+ EXPECT_EQ(trie.lookup(0x0123456789000000), ptr(10));
+
+ trie.remove(node2);
+
+ EXPECT_EQ(trie.lookup(0x0123000000000000), ptr(1));
+ EXPECT_EQ(trie.lookup(0x0123456700000000), ptr(2));
+ EXPECT_EQ(trie.lookup(0x0123456780000000), ptr(3));
+ EXPECT_EQ(trie.lookup(0x0123456789000000), ptr(4));
+
+ trie.remove(node1);
+
+ EXPECT_EQ(trie.lookup(0x0123000000000000), ptr(1));
+ EXPECT_EQ(trie.lookup(0x0123456700000000), nullptr);
+ EXPECT_EQ(trie.lookup(0x0123456780000000), ptr(3));
+ EXPECT_EQ(trie.lookup(0x0123456789000000), ptr(4));
}