fixed bug in clearr function and added more tests

Author: vinitjames

circular_buffer.h

@@ -269,7 +269,7 @@ template<typename T>
 inline 
 void  CircularBuffer<T>::clear(){
 	std::lock_guard<std::mutex> _lck(_mtx);
-	_head = _tail = _size = _max_size = 0;
+	_head = _tail = _size = 0;
 }
 
 template<typename T>

tests/string_buffer.cpp

@@ -5,6 +5,7 @@
 #include "gtest/gtest.h"
 
 #define TEST_BUFFER_SIZE 100
+#define REDUCE_SIZE 100
 
 class CircularBufferTest : public ::testing::Test{
 
@@ -72,6 +73,9 @@ TEST_F(CircularBufferTest, ClearTest){
 	//check size and empty flags after clearing
 	EXPECT_TRUE(test_buff.empty());
 	EXPECT_EQ(test_buff.size(), 0);
+	//fill buffer after clear
+	for(int i=0; i<TEST_BUFFER_SIZE; i++)
+		test_buff.push_back("string" + std::to_string(i));
 
 }
 
@@ -347,6 +351,53 @@ TEST_F(CircularBufferTest, EndIteratorTest){
 
 }
 
+TEST_F(CircularBufferTest, IteratorBasedLoopTest){	
+	//create full buffer
+	for(int i=0; i<TEST_BUFFER_SIZE; i++)
+		test_buff.push_back("string" + std::to_string(i));
+    int i = 0;
+	for(auto it = test_buff.begin(); it!=test_buff.end(); it++)
+		EXPECT_EQ(*it, "string" + std::to_string(i++));
+	//partially fill buffers
+	test_buff.clear();
+	for(int i=0; i<TEST_BUFFER_SIZE/2; i++)
+		test_buff.push_back("string" + std::to_string(i));
+	//test begin and end on partially full buffer
+	i = 0;
+	for(auto it = test_buff.begin(); it!=test_buff.end(); it++)
+		EXPECT_EQ(*it, "string" + std::to_string(i++));
+	//test size with increment variable
+	EXPECT_EQ(i, TEST_BUFFER_SIZE/2);
+
+}
+
+TEST_F(CircularBufferTest, RangeBasedLoopTest){	
+	//create full buffer
+	for(int i=0; i<TEST_BUFFER_SIZE; i++)
+		test_buff.push_back("string" + std::to_string(i));
+	int i = 0;
+	//copied elements 
+    for(auto buff_elem : test_buff)
+		EXPECT_EQ(buff_elem, "string" + std::to_string(i++));
+	//auto reference
+	i = 0;
+    for(auto& buff_elem : test_buff)
+		EXPECT_EQ(buff_elem, "string" + std::to_string(i++));	
+	//auto const reference
+	i = 0;
+    for(const auto& buff_elem : test_buff)
+		EXPECT_EQ(buff_elem, "string" + std::to_string(i++));
+	//check iterators after modifications
+	for(int i = 0; i<REDUCE_SIZE; i++)
+		test_buff.pop_front();
+	i = 0;
+    for(const auto& buff_elem : test_buff)
+		EXPECT_EQ(buff_elem, "string" + std::to_string(i++));
+	EXPECT_EQ(i, TEST_BUFFER_SIZE - REDUCE_SIZE);
+}
+
+
+