Simplify force_allocation_failure

Author: cschreib

tests/runtime_tests.cpp

@@ -9,10 +9,10 @@
 constexpr std::size_t max_allocations = 20'000;
 void*                 allocations[max_allocations];
 void*                 allocations_array[max_allocations];
-std::size_t           num_allocations          = 0u;
-std::size_t           double_delete            = 0u;
-bool                  memory_tracking          = false;
-bool                  force_allocation_failure = false;
+std::size_t           num_allocations               = 0u;
+std::size_t           double_delete                 = 0u;
+bool                  memory_tracking               = false;
+bool                  force_next_allocation_failure = false;
 
 #define CHECK_MEMORY_LEAKS 1
 
@@ -22,7 +22,8 @@ void* allocate(std::size_t size, bool array) {
         throw std::bad_alloc();
     }
 
-    if (force_allocation_failure) {
+    if (force_next_allocation_failure) {
+        force_next_allocation_failure = false;
         throw std::bad_alloc();
     }
 
@@ -286,11 +287,10 @@ TEST_CASE("owner acquiring constructor bad alloc", "[owner_construction]") {
     {
         test_object* raw_ptr = new test_object;
         try {
-            force_allocation_failure = true;
+            force_next_allocation_failure = true;
             test_ptr{raw_ptr};
         } catch (...) {
         }
-        force_allocation_failure = false;
     }
 
     REQUIRE(instances == 0);
@@ -321,11 +321,10 @@ TEST_CASE("owner acquiring constructor with deleter bad alloc", "[owner_construc
     {
         test_object* raw_ptr = new test_object;
         try {
-            force_allocation_failure = true;
+            force_next_allocation_failure = true;
             test_ptr_with_deleter{raw_ptr, test_deleter{42}};
         } catch (...) {
         }
-        force_allocation_failure = false;
     }
 
     REQUIRE(instances == 0);
@@ -1325,11 +1324,10 @@ TEST_CASE("owner reset to new bad alloc", "[owner_utility]") {
         test_object* raw_ptr2 = new test_object;
         test_ptr     ptr(raw_ptr1);
         try {
-            force_allocation_failure = true;
+            force_next_allocation_failure = true;
             ptr.reset(raw_ptr2);
         } catch (...) {
         }
-        force_allocation_failure = false;
         REQUIRE(instances == 1);
         REQUIRE(ptr.get() == raw_ptr1);
     }
@@ -3547,12 +3545,11 @@ TEST_CASE("observer from this maybe no block reset to new bad alloc", "[observer
         test_ptr_from_this_maybe_no_block ptr{raw_ptr1};
 
         try {
-            force_allocation_failure = true;
+            force_next_allocation_failure = true;
             ptr.reset(raw_ptr2);
         } catch (...) {
         }
 
-        force_allocation_failure = false;
         REQUIRE(instances == 1);
         REQUIRE(ptr.get() == raw_ptr1);
     }