Ver código fonte

test: drop a bunch of NOLINT

Michele Caini 2 anos atrás
pai
commit
6ca1bca26c
2 arquivos alterados com 14 adições e 14 exclusões
  1. 11 11
      test/entt/container/dense_map.cpp
  2. 3 3
      test/entt/container/dense_set.cpp

+ 11 - 11
test/entt/container/dense_map.cpp

@@ -183,7 +183,7 @@ TEST(DenseMap, Iterator) {
     testing::StaticAssertTypeEq<iterator::reference, std::pair<const int &, int &>>();
 
     entt::dense_map<int, int> map;
-    map.emplace(3, 42); // NOLINT
+    map.emplace(1, 2);
 
     iterator end{map.begin()};
     iterator begin{};
@@ -221,11 +221,11 @@ TEST(DenseMap, Iterator) {
     ASSERT_GT(end, begin);
     ASSERT_GE(end, map.end());
 
-    map.emplace(42, 3); // NOLINT
+    map.emplace(3, 4);
     begin = map.begin();
 
-    ASSERT_EQ(begin[0u].first, 3);
-    ASSERT_EQ(begin[1u].second, 3);
+    ASSERT_EQ(begin[0u].first, 1);
+    ASSERT_EQ(begin[1u].second, 4);
 }
 
 TEST(DenseMap, ConstIterator) {
@@ -236,7 +236,7 @@ TEST(DenseMap, ConstIterator) {
     testing::StaticAssertTypeEq<iterator::reference, std::pair<const int &, const int &>>();
 
     entt::dense_map<int, int> map;
-    map.emplace(3, 42); // NOLINT
+    map.emplace(1, 2);
 
     iterator cend{map.cbegin()};
     iterator cbegin{};
@@ -274,16 +274,16 @@ TEST(DenseMap, ConstIterator) {
     ASSERT_GT(cend, cbegin);
     ASSERT_GE(cend, map.cend());
 
-    map.emplace(42, 3); // NOLINT
+    map.emplace(3, 4); // NOLINT
     cbegin = map.cbegin();
 
-    ASSERT_EQ(cbegin[0u].first, 3);
-    ASSERT_EQ(cbegin[1u].second, 3);
+    ASSERT_EQ(cbegin[0u].first, 1);
+    ASSERT_EQ(cbegin[1u].second, 4);
 }
 
 TEST(DenseMap, IteratorConversion) {
     entt::dense_map<int, int> map;
-    map.emplace(3, 42); // NOLINT
+    map.emplace(1, 3);
 
     const typename entt::dense_map<int, int>::iterator it = map.begin();
     typename entt::dense_map<int, int>::const_iterator cit = it;
@@ -291,8 +291,8 @@ TEST(DenseMap, IteratorConversion) {
     testing::StaticAssertTypeEq<decltype(*it), std::pair<const int &, int &>>();
     testing::StaticAssertTypeEq<decltype(*cit), std::pair<const int &, const int &>>();
 
-    ASSERT_EQ(it->first, 3);
-    ASSERT_EQ((*it).second, 42);
+    ASSERT_EQ(it->first, 1);
+    ASSERT_EQ((*it).second, 3);
     ASSERT_EQ(it->first, cit->first);
     ASSERT_EQ((*it).second, (*cit).second);
 

+ 3 - 3
test/entt/container/dense_set.cpp

@@ -27,7 +27,7 @@ TEST(DenseSet, Functionalities) {
     ASSERT_EQ(set.max_load_factor(), .875f);
     ASSERT_EQ(set.max_size(), (std::vector<std::pair<std::size_t, int>>{}.max_size()));
 
-    set.max_load_factor(.9f); // NOLINT
+    set.max_load_factor(.9f);
 
     ASSERT_EQ(set.max_load_factor(), .9f);
 
@@ -111,7 +111,7 @@ TEST(DenseSet, Constructors) {
 
 TEST(DenseSet, Copy) {
     entt::dense_set<std::size_t, entt::identity> set;
-    set.max_load_factor(set.max_load_factor() - .05f); // NOLINT
+    set.max_load_factor(set.max_load_factor() - .05f);
     set.emplace(3u);
 
     entt::dense_set<std::size_t, entt::identity> other{set};
@@ -139,7 +139,7 @@ TEST(DenseSet, Copy) {
 
 TEST(DenseSet, Move) {
     entt::dense_set<std::size_t, entt::identity> set;
-    set.max_load_factor(set.max_load_factor() - .05f); // NOLINT
+    set.max_load_factor(set.max_load_factor() - .05f);
     set.emplace(3u);
 
     entt::dense_set<std::size_t, entt::identity> other{std::move(set)};