Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: test/compiled/StringMap.cpp

Issue 29680720: Issue 6221 - Add native tests. (Closed) Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Created Jan. 26, 2018, 8:49 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1
2 #include <string>
sergei 2018/01/26 22:13:36 Could you please remove the empty lines and add th
hub 2018/01/27 03:33:37 Done.
3
4 #include "gtest/gtest.h"
5
6 #include "compiled/StringMap.h"
7
8 template<template <typename T> class S>
9 void testStringMap()
10 {
11 S<std::string> map;
12 auto key = u"Foobar"_str;
13 EXPECT_EQ(key.length(), 6);
14 EXPECT_EQ(map.size(), 0);
15
16 map[u"Foobar"_str] = std::string("one");
17 EXPECT_EQ(map.size(), 1);
18
19 map[u""_str] = std::string("null");
20 EXPECT_EQ(map.size(), 2);
21
22 auto entry = map.find(u"Foobar"_str);
23 EXPECT_TRUE(entry);
24
25 entry = map.find(u"Foobar2"_str);
26 EXPECT_FALSE(entry);
27
28 map[u"Foobar2"_str] = std::string("two");
29 entry = map.find(u"Foobar2"_str);
30 EXPECT_TRUE(entry);
31
32 map[u"Foobar3"_str] = std::string("three");
33 entry = map.find(u"Foobar3"_str);
34 EXPECT_TRUE(entry);
35
36 EXPECT_EQ(map.size(), 4);
37
38 map.erase(u"Foobar2"_str);
39
40 // DISABLED. This should be true, but it isn't
41 //EXPECT_EQ(map.size(), 2);
42
43 entry = map.find(u"Foobar2"_str);
44 EXPECT_FALSE(entry);
45
46 int i = 0;
47 for (const auto& e : map)
48 {
49 EXPECT_FALSE(e.is_invalid());
50 // DISABLED entries that are deleted shouldn't be returned.
51 // See issue #6281
52 //EXPECT_FALSE(e.is_deleted());
53 i++;
54 }
55
56 EXPECT_EQ(i, 4); // SHOULD be 3. See issue #6281
57 EXPECT_EQ(i, map.size());
58 }
59
60 TEST(TestStringMap, stringMap)
61 {
62 testStringMap<StringMap>();
63 }
64
65 TEST(TestStringMap, ownedStringMap)
66 {
67 testStringMap<OwnedStringMap>();
68 }
OLDNEW

Powered by Google App Engine
This is Rietveld