Added const to begin and end of Vector

Change-Id: If62bf53aee68dadcd51491e0950126ace109a636

modules/capu/include/capu/container/Vector.h

@@ -53,24 +53,24 @@ namespace capu
             /**
              * Dereferences the iterator to access the internal data
              */
-            T& operator*();
+            T const& operator*();
 
             /**
              * Allows access to methods and member of the internal data
              */
-            T* operator->();
+            T const* operator->();
         protected:
         private:
             /**
              * Creates a Iterator for the Vector
              * @param start pointer for the iterator
              */
-            Iterator(T* start);
+            Iterator(const T* start);
 
             /**
              * Pointer to the current data
              */
-            T* m_current;
+            T const* m_current;
         };
 
         /**
@@ -130,13 +130,13 @@ namespace capu
          * Returns a new Iterator to the start of the Vector
          * @return a new Iterator to the start of the Vector
          */
-        Iterator begin();
+        Iterator begin() const;
 
         /**
          * Returns a new Iterator to the end of the Vector
          * @return a new Iterator to the end of the Vector
          */
-        Iterator end();
+        Iterator end() const;
 
     protected:
     private:
@@ -253,22 +253,22 @@ namespace capu
     template<typename T>
     inline
     typename Vector<T>::Iterator
-    Vector<T>::begin()
+    Vector<T>::begin() const
     {
         return Iterator(m_data.getRawData());
     }
 
     template<typename T>
     inline
     typename Vector<T>::Iterator
-    Vector<T>::end()
+    Vector<T>::end() const
     {
         return Iterator(m_data.getRawData() + m_size);
     }
 
     template<typename T>
     inline
-    Vector<T>::Iterator::Iterator(T* start)
+    Vector<T>::Iterator::Iterator(const T* start)
         : m_current(start)
     {
 
@@ -292,15 +292,15 @@ namespace capu
 
     template<typename T>
     inline
-    T&
+    T const&
     Vector<T>::Iterator::operator*()
     {
         return *m_current;
     }
 
     template<typename T>
     inline
-    T*
+    T const*
     Vector<T>::Iterator::operator->()
     {
         return &** this;

modules/capu/test/container/VectorTest.cpp

@@ -99,6 +99,40 @@ namespace capu
         EXPECT_EQ(47u, vector2[1]);
     }
 
+    TEST_F(VectorTest, IteratorOnConstVector)
+    {
+        Vector<uint32_t> vector;
+
+        const Vector<uint32_t>& constVector = vector;
+
+        vector.push_back(42u);
+        vector.push_back(47u);
+
+        Vector<uint32_t> vector2;
+
+        for (Vector<uint32_t>::Iterator iter = constVector.begin(); iter != constVector.end(); ++iter)
+        {
+            vector2.push_back(*iter);
+        }
+
+        EXPECT_EQ(42u, vector2[0]);
+        EXPECT_EQ(47u, vector2[1]);
+    }
+
+    TEST_F(VectorTest, AccessOperator)
+    {
+        Vector<uint32_t> vector;
+
+        vector.push_back(42u);
+        vector.push_back(47u);
+
+        vector[0] = 47u;
+        vector[1] = 42u;
+
+        EXPECT_EQ(47u, vector[0]);
+        EXPECT_EQ(42u, vector[1]);
+    }
+
     struct TestStruct
     {
         uint32_t value1;