fix: prj.03: make allocator copy constructible

j0tunn · Oct 26, 2022 · b60b57e · b60b57e
1 parent 247aad2
commit b60b57e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/projects/03/allocator_test.cc b/projects/03/allocator_test.cc
@@ -112,3 +112,10 @@ TEST_F(BulkAllocatorTest, OnRemoveAllElements_ShouldDeallocateWholeMemoryOnce) {
 
     Mock::VerifyAndClearExpectations(&MemoryMgrMock::getInstance());
 }
+
+TEST_F(BulkAllocatorTest, CopyConstructor_ShuldInitiateContainerCopyWithClearAllocator) {
+    EXPECT_CALL(MemoryMgrMock::getInstance(), Allocate).Times(Exactly(2));
+
+    Map<2> m{{1, 1}, {2, 2}};
+    auto m2 = m;
+}
diff --git a/projects/03/bulk_allocator.h b/projects/03/bulk_allocator.h
@@ -23,7 +23,9 @@ class BulkAllocator {
     ~BulkAllocator() = default;
 
     template <typename U>
-    BulkAllocator(const BulkAllocator<U, N>&) {}
+    BulkAllocator(const BulkAllocator<U, N>&) : memoryBlocks_() {}
+
+    BulkAllocator(const BulkAllocator<T, N>&) : memoryBlocks_() {}
 
     T* allocate(std::size_t n) {
         T* p = nullptr;