Add more comments on tests

Author: winner245

libcxx/test/std/containers/sequences/vector.bool/append_range_exceptions.pass.cpp

@@ -26,7 +26,7 @@ int main(int, char**) {
   // Note: `test_append_range_exception_safety_throwing_copy` doesn't apply because copying booleans cannot throw.
   test_append_range_exception_safety_throwing_allocator<std::vector, bool>();
 
-  {
+  { // Attempt to append more elements to a non-empty vector<bool> than its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size() - 2, true);
     bool a[] = {false, true, false};
@@ -39,16 +39,14 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
-    std::vector<bool, limited_allocator<bool, 10> > v(5, true);
-    bool a[v.max_size()] = {}; // A large enough array to trigger a length_error when appended
+  { // Attempt to append more elements to an empty vector<bool> than its maximum possible size
+    std::vector<bool, limited_allocator<bool, 10> > v;
+    bool a[v.max_size() + 1] = {}; // A large enough array to trigger a length_error when appended
     try {
       v.append_range(a);
       assert(false);
     } catch (const std::length_error&) {
-      assert(v.size() == 5);
-      for (std::size_t i = 0; i != v.size(); ++i)
-        assert(v[i] == true);
+      assert(v.empty());
     }
   }
 

libcxx/test/std/containers/sequences/vector.bool/emplace_back_exceptions.pass.cpp

@@ -24,6 +24,8 @@
 int main(int, char**) {
   std::vector<bool, limited_allocator<bool, 10> > v;
   v.resize(v.max_size(), true);
+
+  // Attempt to append one more element to a vector<bool> that is already at its maximum possible size
   try {
     v.emplace_back(true);
     assert(false);

libcxx/test/std/containers/sequences/vector.bool/emplace_exceptions.pass.cpp

@@ -21,7 +21,7 @@
 #include "test_allocator.h"
 
 int main(int, char**) {
-  {
+  { // Attempt to insert an element at the beginning of a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), true);
     try {
@@ -33,7 +33,7 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
+  { // Attempt to insert an element at the end of a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), true);
     try {
@@ -45,7 +45,7 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
+  { // Attempt to insert an element in the middle of a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), true);
     try {

libcxx/test/std/containers/sequences/vector.bool/insert_exceptions.pass.cpp

@@ -25,7 +25,7 @@
 #include "test_macros.h"
 
 int main(int, char**) {
-  {
+  { // Attempt to insert an element at the beginning of a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), true);
     try {
@@ -37,7 +37,7 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
+  { // Attempt to insert an element at the end of a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), false);
     try {
@@ -49,7 +49,7 @@ int main(int, char**) {
         assert(v[i] == false);
     }
   }
-  {
+  { // Attempt to insert an element in the middle of a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), true);
     try {
@@ -61,19 +61,7 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
-    std::vector<bool, limited_allocator<bool, 10> > v;
-    v.resize(v.max_size(), true);
-    try {
-      v.insert(v.begin(), 1, false);
-      assert(false);
-    } catch (const std::length_error&) {
-      assert(v.size() == v.max_size());
-      for (std::size_t i = 0; i != v.size(); ++i)
-        assert(v[i] == true);
-    }
-  }
-  {
+  { // Attempt to insert an iterator range to a vector<bool> that would exceed its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size() - 2, true);
     bool a[] = {true, false, true};
@@ -87,7 +75,7 @@ int main(int, char**) {
     }
   }
 #if TEST_STD_VER >= 11
-  {
+  { // Attempt to insert an initializer_list to a vector<bool> that would exceed its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size() - 2, true);
     try {

libcxx/test/std/containers/sequences/vector.bool/insert_range_exceptions.pass.cpp

@@ -26,7 +26,7 @@ int main(int, char**) {
   // Note: `test_insert_range_exception_safety_throwing_copy` doesn't apply because copying booleans cannot throw.
   test_insert_range_exception_safety_throwing_allocator<std::vector, bool>();
 
-  {
+  { // Attempt to insert a range at the end of a vector<bool> that would exceed its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size() - 2, true);
     bool a[] = {true, false, true};
@@ -39,7 +39,7 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
+  { // Attempt to insert a range at the beginning of a vector<bool> that would exceed its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size() - 2, false);
     bool a[] = {true, false, true};
@@ -52,7 +52,7 @@ int main(int, char**) {
         assert(v[i] == false);
     }
   }
-  {
+  { // Attempt to insert a range in the middle of a vector<bool> that would exceed its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v(5, true);
     bool a[v.max_size()] = {};
     try {

libcxx/test/std/containers/sequences/vector.bool/push_back_exceptions.pass.cpp

@@ -20,6 +20,7 @@
 #include "test_allocator.h"
 
 int main(int, char**) {
+  // Attempt to push back an element into a vector<bool> that is already at its maximum possible size
   std::vector<bool, limited_allocator<bool, 10> > v;
   v.resize(v.max_size(), true);
   try {

libcxx/test/std/containers/sequences/vector.bool/reserve_exceptions.pass.cpp

@@ -20,7 +20,7 @@
 #include "test_allocator.h"
 
 int main(int, char**) {
-  {
+  { // Attempt to reserve more space than the maximum possible size of a vector<bool>
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.reserve(5);
     try {
@@ -35,7 +35,7 @@ int main(int, char**) {
       assert(v.capacity() >= 5);
     }
   }
-  {
+  { // Attempt to reserve more space for a vector<bool> that is already at its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
     v.resize(v.max_size(), true);
     try {
@@ -47,7 +47,7 @@ int main(int, char**) {
         assert(v[i] == true);
     }
   }
-  {
+  { // Attempt to reserve 1 more space than the maximum possible size of a vector<bool>
     bool a[] = {true, false, true, false, true};
     std::vector<bool> v(std::begin(a), std::end(a));
     try {

libcxx/test/std/containers/sequences/vector.bool/resize_size_exceptions.pass.cpp

@@ -21,7 +21,7 @@
 #include "test_allocator.h"
 
 int main(int, char**) {
-  {
+  { // Attempt to resize a non-empty vector<bool> to a size greater than its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v(5, false);
     try {
       v.resize(v.max_size() + 1);
@@ -33,19 +33,16 @@ int main(int, char**) {
         assert(!v[i]);
     }
   }
-  {
+  { // Attempt to resize an empty vector<bool> to a size greater than its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
-    v.resize(v.max_size() / 2);
     try {
       v.resize(v.max_size() + 1);
       assert(false);
     } catch (const std::length_error&) {
-      assert(v.size() == v.max_size() / 2);
-      for (std::size_t i = 0; i < v.size(); ++i)
-        assert(v[i] == false);
+      assert(v.empty());
     }
   }
-  {
+  { // Attempt to resize a vector<bool> with standard allocator to a size greater than its maximum possible size
     bool a[] = {true, false, true, false, true};
     std::vector<bool> v(std::begin(a), std::end(a));
     try {

libcxx/test/std/containers/sequences/vector.bool/resize_size_value_exceptions.pass.cpp

@@ -23,7 +23,7 @@
 #include "test_allocator.h"
 
 int main(int, char**) {
-  {
+  { // Attempt to resize a non-empty vector<bool> to a size greater than its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v(5, false);
     try {
       v.resize(v.max_size() + 1, true);
@@ -35,19 +35,16 @@ int main(int, char**) {
         assert(!v[i]);
     }
   }
-  {
+  { // Attempt to resize an empty vector<bool> to a size greater than its maximum possible size
     std::vector<bool, limited_allocator<bool, 10> > v;
-    v.resize(v.max_size() / 2);
     try {
       v.resize(v.max_size() + 1, true);
       assert(false);
     } catch (const std::length_error&) {
-      assert(v.size() == v.max_size() / 2);
-      for (std::size_t i = 0; i < v.size(); ++i)
-        assert(v[i] == false);
+      assert(v.empty());
     }
   }
-  {
+  { // Attempt to resize a vector<bool> with standard allocator to a size greater than its maximum possible size
     bool a[] = {true, false, true, false, true};
     std::vector<bool> v(std::begin(a), std::end(a));
     try {

libcxx/test/std/containers/sequences/vector/vector.modifiers/append_range_exceptions.pass.cpp

@@ -25,6 +25,7 @@ int main(int, char**) {
   test_append_range_exception_safety_throwing_copy<std::vector>();
   test_append_range_exception_safety_throwing_allocator<std::vector, int>();
 
+  // Attempt to append a range to a vector that would exceed its maximum possible size
   {
     std::vector<int, limited_allocator<int, 10> > v(8, 42);
     int a[] = {1, 2, 3};