add gtest for ConcreteBuffer, SimpleArray and BufferExpander

tigercosmos commented 3 weeks ago

This PR aims to close https://github.com/solvcon/modmesh/issues/306.

After reading the source code of ConcreteBuffer, I think we don't need to align the iterator implementation between ConcreteBuffer and BufferExpander. In ConcreteBuffer, there are no m_begin and m_end, and only has a std::unique_ptr<int8_t, data_deleter_type> type m_data, so I think we can keep it as it is.
Added iterator tests in gtest for ConcreteBuffer, SimpleArray, and BufferExpander.

p.s. Seems that SimpleCollector.hpp also misses the implementation of iterator, but let's make it in another PR.

yungyuc commented 3 weeks ago

This PR aims to close #306.

I am not sure using the PR directives like "close", "fix" is a good idea. But let's try. If we need to do a lot of manual reopen, maybe it's not good.

p.s. Seems that SimpleCollector.hpp also misses the implementation of iterator, but let's make it in another PR.

Makes sense.

tigercosmos commented 3 weeks ago

@yungyuc do you think the issue is not ready to close after this PR? also, please check my comment.

solvcon / modmesh

add gtest for ConcreteBuffer, SimpleArray and BufferExpander #358